sources: add 7z source handler
Metadata
Current evaluation
Added a 7z source handler for parts. Supersedes #232, uses upstream 7zip, switches linting to Ruff, and updates subprocess calls. Merged after approval by three reviewers and CI fixes.
Suggested action: —
No scores available.
Issue body
- [X] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
Allow parts to specify 7z sources. Based on the implementation from Snapcraft.
Supersedes #232. The changes from #232 include:
- Use better maintained upstream 7zip version instead of defunct `p7zip` package
- Switch linter annotations from Pylint to Ruff
- Use `-o` path option instead of using `cwd=` option in `subprocess.run`
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Added a 7z source handler for parts. Supersedes #232, uses upstream 7zip, switches linting to Ruff, and updates subprocess calls. Merged after approval by three reviewers and CI fixes. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Successfully merged. Adds a 7z source handler for parts, replacing p7zip with upstream 7zip, switching linting to Ruff, and fixing subprocess path handling. Supersedes #232. CI errors were resolved prior to merge. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #232 sources: add 7z source handler | craft-parts | closed | Adding a 7z source handler was closed and superseded by PR #740. The original implementation was rebased on main but never merged, as a replacement PR handles the feature. | |
| #1515 fix: 7z(ip) sources support (#1443) | craft-parts | merged | Merged PR #1443 to add and document 7z/ip source type support. Approved by two reviewers and merged despite failing CI checks on select architectures and the OSV-scanner. | |
| #1168 sources: add support for 7-zip files | snapcraft | merged | Merged. Adds support for unpacking 7-zip files using libarchive. Contributor signed CLA, resolved conflicts, and passed tests following reviewer approval. | |
| #212 sources: add zip file source handler | craft-parts | merged | Merged after approval and CI checks. Adds a zip file source handler allowing parts to specify zip archives as sources. Changes span six files with 261 additions and 7 deletions. Co-authored by Kyle Fazzari, Sergio Schvezov, and Marius Gripsgard. |