← Back to issue list

build(deps): update patchelf to 0.19

View original Github issue

Metadata

Project
snapcraft
Number
#6511
Type
pull request
State
open
Author
mr-cal
Labels
—
Created
Updated
Closed
—

Current evaluation

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).

Suggested action: keep open

Reason: This is a routine dependency update (patchelf 0.9→0.19) with a small diff (+15/-31 lines, 3 files) that fixes ELF corruption bugs for Qt 6 plugins (#6410). However, it is blocked by the external PR canonical/patchelf#4, has 1 unresolved review comment, and one failing CI check (OSV-scanner). The PR should remain open until the external dependency lands and review feedback is addressed.

Evaluated by: mr-cal-bot

Impact: 55 Quick Win: 21 Complexity: 25 Confidence: 75 Actionability: 50

Issue body

Bumps patchelf to 0.19. For a detailed report on the changes and validation, see the report in #6509. Blocked by https://github.com/canonical/patchelf/pull/4 Fixes https://github.com/canonical/snapcraft/pull/6350#discussion_r3596366277 Fixes https://github.com/canonical/snapcraft/issues/6410 Fixes #6509 (SNAPCRAFT-1400) --- - [x] I've followed the [contribution guidelines](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md). - [x] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/). - [x] I've successfully run `make lint && make test`. - [ ] I've added or updated any relevant documentation. - [ ] In documents I changed, I [added a meta description](https://canonical-starflow.readthedocs-hosted.com/how-to/add-a-page-meta-description/) if one was missing. - [ ] I've updated the relevant release notes.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b
Impact: 55
Quick Win: 21
Complexity: 25
Confidence: 75
Actionability: 50
keep 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).
qwen/qwen3.6-35b-a3b
Impact: 50
Quick Win: 13
Complexity: 15
Confidence: 75
Actionability: 30
needs review Dependency update bumping patchelf to 0.19 to fix ELF corruption regressions. Has 3 unresolved review comments, no reviewers yet, and a failing OSV-scanner CI check. Blocked by external PR canonical/patchelf#4.

Update history

Date Change
updated
created

Related work

  • Related To: snapcraft#6410 (confidence 95%)

    The patchelf update fixes the Qt 6 plugin ELF corruption bug described in this issue.

  • Related To: snapcraft#6509 (confidence 95%)

    Tracking issue for the patchelf update with test plan and validation results.

  • Blocked By: patchelf#4 (confidence 90%)

    PR explicitly states it is blocked by this external PR which must land first.

Related issues

Issue Project State Summary Similarity
#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.
80%
#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.
72%
#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.
72%
#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.
71%
#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.
71%
#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).
70%