fix: do not load system libraries for classic core24 snaps
Metadata
Current evaluation
PR #4807 was closed in favor of #4809, which merged commit 5493a5f91 and shipped in Snapcraft 8.2.8; classic core24 snaps now use patchelf --no-default-libs.
Resolution: closed superseded
Reason: The change was carried out by merged PR canonical/snapcraft#4809. Commit 5493a5f91 implemented the fix and was released in Snapcraft 8.2.8.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [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] Have you successfully run `tox run -m lint`?
- [x] Have you successfully run `tox run -e test-py310`? (supported versions: `py39`, `py310`, `py311`, `py312`)
-----
On `core24`, snapcraft should pass `--no-default-libs` to `patchelf` to not load libraries from default locations.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed superseded | PR #4807 was closed in favor of #4809, which merged commit 5493a5f91 and shipped in Snapcraft 8.2.8; classic core24 snaps now use patchelf --no-default-libs. | |
| stealth/space-bunny-alpha | — | closed superseded | Superseded by merged #4809 (commit 5493a5f91), released in Snapcraft 8.2.8; classic core24 builds now disable patchelf's default system-library search. | |
| stealth/space-bunny-alpha | — | closed superseded | Superseded by #4809, which merged commit 5493a5f91 and shipped in Snapcraft 8.2.8; classic core24 builds now prevent patchelf from loading system libraries. | |
| stealth/space-bunny-alpha | — | closed superseded | Superseded by canonical/snapcraft#4809, merged as 5493a5f91 and released in 8.2.8; classic core24 builds now pass --no-default-libs to patchelf. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in canonical/snapcraft#4809 (commit 5493a5f91) and released in Snapcraft 8.2.8; classic core24 builds now pass --no-default-libs to patchelf to prevent system-library loading. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged as canonical/snapcraft#4809 in Snapcraft 8.2.8. Classic core24 builds now pass --no-default-libs to patchelf, preventing libraries from being loaded from system default locations. | |
| stealth/space-bunny-alpha | — | closed superseded | Closed unmerged; its --no-default-libs fix was incorporated through merged PR canonical/snapcraft#4809 and released in snapcraft 8.2.8. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed without merging. The patchelf configuration change for core24 snaps was abandoned due to zero review comments and no CI checks. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Passes --no-default-libs to patchelf for core24 snaps to prevent loading system libraries from default locations. |
Update history
No update history recorded yet.
Related work
-
Superseded By:
canonical/snapcraft#4809
(confidence 99%)
Merged replacement PR containing the fix from #4807.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #4809 do not load system libraries for classic core24 snaps, bump craft-parts, unregister dotnet | snapcraft | merged | Merged for Snapcraft 8.2.8: classic core24 builds use patchelf --no-default-libs, craft-parts is bumped, and core24 dotnet is unregistered; replacing #4807. | |
| #3087 meta: remove snapd workaround for classic for core20 onwards | snapcraft | merged | Merged canonical/snapcraft#3087 as commit 244b997ca, removing the automatic classic snapcraft-runner workaround for core20 and newer while retaining it for legacy bases. | |
| #1647699 Support classic building in snapcraft | snapcraft (launchpad) | closed | Classic-snap builds were implemented by merged snapcore/snapcraft#949 (commit 058f5626d), resolving libraries from the core/base snap. | |
| #1656187 Support core path in classic snaps for library paths | snapcraft (launchpad) | closed | Classic-snap core library path support was delivered by merged PR #1047 in commit 6276bca73, including wrapper LD_LIBRARY_PATH handling using /snap/core/current. |