snap: update revision of patchelf to use
Metadata
Current evaluation
Closed unmerged; the bundled PatchELF revision bump in #1983 was superseded by #2006, which adopted packaging-based acquisition (commit aa2a40322).
Resolution: closed superseded
Reason: PR #2006 was merged as commit aa2a40322 and replaced the revision-specific snap build change by using packaging logic to obtain PatchELF.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)?
- [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)?
- [x] Have you successfully run `./runtests.sh static`?
- [x] Have you successfully run `./runtests.sh unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed superseded | Closed unmerged; the bundled PatchELF revision bump in #1983 was superseded by #2006, which adopted packaging-based acquisition (commit aa2a40322). | |
| stealth/space-bunny-alpha | — | closed stale | canonical/snapcraft#1983 updated the patchelf revision but was closed unmerged with no reviews, comments, or CI checks, leaving the change abandoned. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed without merging. The patchelf revision update received no reviews, comments, or CI checks, indicating it was abandoned or superseded by a later change. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Closed without comments. The pull request updated the patchelf revision in the snap configuration. |
Update history
No update history recorded yet.
Related work
-
Superseded By:
snapcraft#2006
(confidence 93%)
Its merged packaging-based approach replaced the bundled PatchELF revision update proposed in #1983.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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). | |
| #2006 many: use packaging logic to get patchelf | snapcraft | merged | Merged PR #2006 in commit aa2a40322, replacing separate patchelf retrieval with shared packaging logic so snap and deb builds use the same version. | |
| #1666 Refactor to support elf patching | snapcraft | closed | ELF patching support was refactored by merged PR #1744 in commit ec7ba948a, converting the ELF implementation into reusable libraries. | |
| #2531 snap: revert to patchelf 0.9 with local patches | snapcraft | merged | Merged PR #2531 as commit 44be372e3, reverting from Patchelf 0.10 to locally patched 0.9 after architecture tests exposed unreliable binaries. | |
| #3841 elf: add elf file patcher | snapcraft | merged | Merged PR #3841 in commit e03a971b3, adding a patchelf-based ELF file patcher ported from Snapcraft 6.x for CRAFT-1191. | |
| #3588 snap: patch patchelf on riscv64 (CRAFT-566) | snapcraft | merged | Merged PR #3588 via commit bf99dbdb2, conditionally patching PatchELF on riscv64 while leaving other architectures unchanged. | |
| #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. | |
| #6509 Update patchelf | snapcraft | open | Maintainer proposes updating bundled patchelf from 0.9 to 0.19.2 to fix ELF corruption bugs and improve performance; test plan and results included. | |
| #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. | |
| #1798 elf: strip the .note.go.buildid to make room for patching elf | snapcraft | merged | Merged #1798 in fa8b7b174, stripping Go ELF .note.go.buildid sections before patching and requiring newer patchelf header-layout support. |