← Back to issue list

Add SNAP_LIBRARY_PATH to LD_LIBRARY_PATH.

View original Github issue

Metadata

Project
snapcraft
Number
#462
Type
pull request
State
merged
Author
kyrofa
Labels
—
Created
Updated
Closed

Current evaluation

PR #462 by kyrofa merged as commit a983e470b, prefixing the generated LD_LIBRARY_PATH with ${SNAP_LIBRARY_PATH}${LD_LIBRARY_PATH:+:$LD_LIBRARY_PATH} so snaps pick up host OS snap libraries. Resolves LP bug #1570945.

Resolution: closed resolved

Reason: Merged into snapcraft's mainline as commit a983e470b ("Add SNAP_LIBRARY_PATH to LD_LIBRARY_PATH. (#462)"), updating snapcraft/utils.py get_ld_library_paths() to emit ${SNAP_LIBRARY_PATH}${LD_LIBRARY_PATH:+:$LD_LIBRARY_PATH} plus unit/spread test expectations; it closed LP bug #1570945 and superseded the earlier attempt PR #460.

Evaluated by: mr-cal-bot

No scores available.

Issue body

This PR resolves LP: [#1570945](https://bugs.launchpad.net/snapcraft/+bug/1570945)

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved PR #462 by kyrofa merged as commit a983e470b, prefixing the generated LD_LIBRARY_PATH with ${SNAP_LIBRARY_PATH}${LD_LIBRARY_PATH:+:$LD_LIBRARY_PATH} so snaps pick up host OS snap libraries. Resolves LP bug #1570945.
stealth/space-bunny-alpha — closed resolved Merged PR #462 as commit a983e470b, adding SNAP_LIBRARY_PATH to LD_LIBRARY_PATH and resolving LP #1570945.
qwen/qwen3.6-35b-a3b — — Merged changes adding SNAP_LIBRARY_PATH to LD_LIBRARY_PATH, resolving Launchpad bug #1570945. The three-file update received maintainer approval and was merged.
qwen3.6-35b-a3b-mtp-q6 — — Merged to add SNAP_LIBRARY_PATH to LD_LIBRARY_PATH, resolving Launchpad bug #1570945. Maintainer kyrofa authored the fix, which passed retests and received reviewer approval before integration.

Update history

No update history recorded yet.

Related work

  • Related To: snapcraft (launchpad)#1570945 (confidence 92%)

    Launchpad bug #1570945 cited in the PR body and closed by the merge.

  • Related To: canonical/snapcraft#460 (confidence 90%)

    Earlier 'add SNAP_LIBRARY_PATH' PR closed unmerged; #462 is the completed version by the same maintainer.

  • Related To: snapcraft#204 (confidence 45%)

    Follow-on work (LP #1531620) that added Mesa/libGL directories to the same default LD_LIBRARY_PATH built by get_ld_library_paths().

Related issues

Issue Project State Summary Similarity
#1570945 Add SNAP_LIBRARY_PATH to LD_LIBRARY_PATH snapcraft (launchpad) closed Fixed in canonical/snapcraft#462, merged as commit a983e470b, by prepending SNAP_LIBRARY_PATH to generated LD_LIBRARY_PATH values so binaries use the OS snap's OpenGL libraries.
79%
#460 add SNAP_LIBRARY_PATH snapcraft closed PR #460 (mvo5, one-line SNAP_LIBRARY_PATH addition) was closed unmerged by kyrofa in favour of #462, which made the same change plus test fixes and was merged as commit a983e470b.
78%