snap: revert to patchelf 0.9 with local patches
Metadata
Current evaluation
Merged PR #2531 as commit 44be372e3, reverting from Patchelf 0.10 to locally patched 0.9 after architecture tests exposed unreliable binaries.
Resolution: closed resolved
Reason: Commit 44be372e3 restored Patchelf 0.9 with local patches because 0.10 produced unreliable binaries on i386, ppc64el, and s390x.
Evaluated by: mr-cal-bot
No scores available.
Issue body
After extensive tests with patchelf 0.10 on different architectures,
it seems that it's not producing reliable binaries on (at least) i386,
ppc64el and s390x. Revert back to 0.9 with local patches until 0.10
issues are properly addressed.
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@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/)?
- [ ] Have you successfully run `./runtests.sh static`?
- [ ] Have you successfully run `./runtests.sh tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #2531 as commit 44be372e3, reverting from Patchelf 0.10 to locally patched 0.9 after architecture tests exposed unreliable binaries. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #2531 in 44be372e3, reverting snapcraft to locally patched patchelf 0.9 after 0.10 produced unreliable binaries on i386, ppc64el, and s390x. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #2531 as commit 44be372e3, reverting bundled patchelf from 0.10 to locally patched 0.9 after cross-architecture tests found unreliable binaries. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in commit 44be372e3, reverting Snapcraft from PatchELF 0.10 to locally patched 0.9 after cross-architecture reliability failures. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #2531, restoring locally patched patchelf 0.9 after 0.10 produced unreliable binaries on i386, ppc64el, and s390x (commit 44be372e3). | |
| qwen/qwen3.6-35b-a3b | — | — | Merged a revert to patchelf 0.9 with local patches after version 0.10 generated unreliable binaries on i386, ppc64el, and s390x. The change restores build stability pending upstream fixes. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a revert of patchelf from 0.10 to 0.9 with local patches after 0.10 generated unreliable binaries on i386, ppc64el, and s390x. The change was merged into master as a temporary workaround until upstream issues are resolved. |
Update history
No update history recorded yet.
Related work
-
Caused By:
snapcraft#2519
(confidence 96%)
Introduced Patchelf 0.10, whose architecture-specific reliability problems prompted the rollback.
-
Caused By:
snapcraft#2528
(confidence 94%)
Switched Patchelf sources to the Snapcore 0.10 branch immediately before this rollback.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2519 packaging: use local patchelf | snapcraft | merged | Merged as e39d2404c, adopting Patchelf 0.10 and relinking dynamic Go executables; later reverted by #2531 after architecture tests exposed unreliable binaries. | |
| #2530 Patchelf test branch | snapcraft | closed | Patchelf test-branch PR #2530 was closed unmerged after years without review or CI; PR #2531 shipped the alternative locally patched Patchelf 0.9 approach in 44be372e3. | |
| #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. | |
| #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. | |
| #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). | |
| #6511 build(deps): update patchelf to 0.19 | snapcraft | open | Updates bundled patchelf from 0.9 to 0.19 to fix ELF corruption bugs with Qt 6 plugins. Blocked by external PR canonical/patchelf#4. Has 1 unresolved review comment and one failing CI check (OSV-scanner). | |
| #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). | |
| #2081 elf: patch everything instead of a subset of elf files | snapcraft | merged | Merged PR #2081 restored Snapcraft 2.39 behavior by patching every ELF file; commit 77770452b partially reverted 64f78f762a05. | |
| #1967 project_loader: improve the logic to install patchelf as a build tool | snapcraft | merged | Merged PR #1967 as commit 84aaac591, improving patchelf build-tool installation and excluding broken Xenial armhf/s390x packages. | |
| #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. |