debian/control: Add patchelf to Depends:
Metadata
Current evaluation
Closed unmerged; the apt dependency change was superseded by #1888 (commit 7c947f5ec), which added Patchelf to Snapcraft's deb and snap delivery mechanisms.
Resolution: closed superseded
Reason: PR #1864 was closed unmerged after maintainers noted Xenial lacked required Patchelf 0.9. PR #1888 superseded its packaging approach and merged as 7c947f5ec, making Patchelf a dependency through supported delivery paths.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [y] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [y] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [y] If this is a bugfix. https://bugs.launchpad.net/snapcraft/+bug/1742707
- [n] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)?
- [y] Have you successfully run `./runtests.sh static`?
- [y] Have you successfully run `./runtests.sh unit`?
-----
Since snapcraft 2.38 `patchelf` is required but it is not listed in `Depends:` or `Recommends:` This pull request fixes that.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed superseded | Closed unmerged; the apt dependency change was superseded by #1888 (commit 7c947f5ec), which added Patchelf to Snapcraft's deb and snap delivery mechanisms. | |
| stealth/space-bunny-alpha | — | closed not a bug | Closed unmerged: Xenial lacked Patchelf 0.9, so maintainers rejected the dependency change. Later PR #1888 landed commit 7c947f5ec, providing Patchelf through Snapcraft's supported packaging paths. | |
| stealth/space-bunny-alpha | — | closed stale | Closed unmerged; Xenial lacked patchelf 0.9, so the narrow Debian dependency change was abandoned. Later PR #1888 addressed patchelf delivery in commit 7c947f5ec. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed without merging. Maintainers declined adding patchelf to Debian dependencies due to Xenial version constraints, noting it is supplied via build snaps or Docker images. Abandoned in favor of existing distribution methods. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Closed without merging. Maintainers declined adding patchelf to Debian dependencies, noting it is provided via build snaps or Docker images, and Xenial lacks the required version. The request was abandoned. |
Update history
No update history recorded yet.
Related work
-
Superseded By:
snapcraft#1888
(confidence 98%)
Merged commit 7c947f5ec implemented Patchelf dependency handling for Snapcraft's deb and snap delivery paths.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1742707 patchelf is missing from snapcraft Depends: | snapcraft (launchpad) | closed | Reported missing patchelf apt dependency since snapcraft 2.38. Fixed by PR #1888 (commit 7c947f5ec, "elf: make patchelf a dependency"), released in snapcraft 2.40; issue closed by sergiusens. | |
| #1888 elf: make patchelf a dependency | snapcraft | merged | Merged PR #1888 via commit 7c947f5ec, moving Patchelf out of project_loader build tools and into the deb and snap delivery mechanisms. | |
| #1873 elf: cleaner patchelf experience | snapcraft | merged | Merged as df6992224: Snapcraft now prefers system Patchelf outside snaps and copies target files to avoid hard-link patching conflicts. | |
| #3277 pluginhandler: support using patchelf on strict snaps | snapcraft | merged | Merged PR #3277 as fd8e95c3d, adding per-part enable-patchelf support to patch ELF rpaths and interpreters in strictly confined snaps. | |
| #2528 packaging: use our patchelf branch | snapcraft | merged | Merged (commit bd26cafd6, squashed merge of #2528): snapcraft.yaml now points patchelf sources at Snapcore's own patchelf branch with the failing s390x test disabled, unblocking s390x package builds; later reverted by #2531. | |
| #1794 Apply patchelf work to non classic | snapcraft | closed | Patchelf support was extended from classic confinement to strict snaps by merged PR #3277 (commit fd8e95c3d), resolving #1794. | |
| #3589 snap: correct patch apply for patchelf | snapcraft | merged | Merged PR #3589 fixed PatchELF patch application in the snap build via commit d9484faf9 (merge a636d26b5). | |
| #5135 fix(test): add patchelf to dev dependencies for testing | snapcraft | closed | Closed unmerged as obsolete after #5225 superseded the proposed patchelf test dependency; the Starflow migration merged in commit 7a1a94e46. | |
| #1983 snap: update revision of patchelf to use | snapcraft | closed | Closed unmerged; the bundled PatchELF revision bump in #1983 was superseded by #2006, which adopted packaging-based acquisition (commit aa2a40322). |