← Back to issue list

fix(kde-neon-6): don't modify LD_LIBRARY_PATH

View original Github issue

Metadata

Project
snapcraft
Number
#6366
Type
pull request
State
merged
Author
mr-cal
Labels
PR: Run Manual Spread
Created
Updated
Closed

Current evaluation

Merged PR #6366 reapplies #6016 onto main: KDE Neon 6 no longer sets LD_LIBRARY_PATH/QT_PLUGIN_PATH in snapcraft.yaml (moved to the launcher) and lxqt-support is updated, fixing #6327 (SNAPCRAFT-1365).

Resolution: closed resolved

Reason: The PR itself was merged with 2 approvals, reverting the 8.14 hotfix regression introduced by the merge in #6104; issue #6327 is now closed as fixed by #6366, restoring commit 3fc9abd0b (#6016) on main.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Correctly applies #6016 onto `main`. #6016 made a few fixes for the KDE extensions for Snapcraft 8: - setting `LD_LIBRARY_PATH` and `QT_PLUGIN_PATH` in the launcher rather than in the `snapcraft.yaml` - updating the lxqt-support content interface These fixes weren't fully applied when merging from `hotfix/8.14` into `main` (https://github.com/canonical/snapcraft/pull/6104), so Snapcraft 9 never received the fixes. Source: https://forum.snapcraft.io/t/call-for-testing-snapcraft-9-0/51348/23?u=mr_cal Fixes #6327 (SNAPCRAFT-1365) --- - [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
stealth/space-bunny-alpha — closed resolved Merged PR #6366 reapplies #6016 onto main: KDE Neon 6 no longer sets LD_LIBRARY_PATH/QT_PLUGIN_PATH in snapcraft.yaml (moved to the launcher) and lxqt-support is updated, fixing #6327 (SNAPCRAFT-1365).
stealth/space-bunny-alpha — closed resolved Merged #6366 restores #6016 on main: KDE Neon 6 configures LD_LIBRARY_PATH and QT_PLUGIN_PATH in its launcher, and lxqt-support is updated, fixing #6327 in Snapcraft 9.
qwen3.6-35b-a3b-mtp-q6 — — Merged into main. Applies KDE Neon 6 fixes by moving LD_LIBRARY_PATH and QT_PLUGIN_PATH configuration to the launcher instead of snapcraft.yaml, resolving issue #6327.
qwen/qwen3.6-35b-a3b
Staleness: 0
Complexity: 10
Confidence: 95
needs review Applies missed KDE extension fixes from #6016 to main, correcting LD_LIBRARY_PATH handling in the Snapcraft launcher. Approved, CI failures noted as unrelated, ready for merge.
qwen3.6-35b-a3b-mtp-q6
Staleness: 5
Complexity: 15
Confidence: 90
needs review Applies missing KDE extension fixes to main by moving LD_LIBRARY_PATH and QT_PLUGIN_PATH configuration to the launcher instead of snapcraft.yaml, resolving issue #6327. Awaiting maintainer review and CI approval.

Update history

Date Change
updated
updated
updated
updated
updated
updated
updated
updated
created

Related work

  • Duplicate Of: canonical/snapcraft#6016 (confidence 95%)

    Original merged fix (commit 3fc9abd0b) on the 8.14 hotfix branch, reapplied here to main.

  • Related To: canonical/snapcraft#6104 (confidence 88%)

    The merge/8.14.4 into main that failed to fully apply the fixes and caused the Snapcraft 9 regression.

Related issues

Issue Project State Summary Similarity
#6016 fix(kde): don't modify LD_LIBRARY_PATH snapcraft merged Merged in PR #6016 (commit 3fc9abd0b), removing Snapcraft-added LD_LIBRARY_PATH and QT_LIBRARY_PATH from KDE Neon 6 in favor of launcher configuration.
92%
#6002 Fix kde-neon-6 LD_LIBRARY_PATH snapcraft closed Superseded by merged PR canonical/snapcraft#6016 (commit 3fc9abd0b), which corrected KDE Neon 6 environment handling by relying on launcher configuration.
88%
#6327 Snapcraft 9 incorrectly sets LD_LIBRARY_PATH and QT_LIBRARY_PATH in the extensions snapcraft closed Fixed in merged PR #6366, restoring #6016 on main: KDE Neon 6 configures library paths in its launcher, and lxqt-support symlinks are corrected.
83%
#5904 Add the lxqt-support plug when creating Snap apps that use the kde-neon-6 extension. snapcraft closed Fixed by merged PR #5917 (commit 70275c88a): the kde-neon-6 extension now adds the lxqt-support-core24 content plug, LD_LIBRARY_PATH/QT_PLUGIN_PATH env setup and Kvantum layout; shipped in Snapcraft 8.14.
75%
#5302 kde-neon: invalid CMAKE_PREFIX_PATH snapcraft closed Fixed KDE Neon CMAKE_PREFIX_PATH handling in merged PR #5310 (commit 9cb3f5155), released in Snapcraft 8.8; the maintainer marked #5302 completed.
74%
#5585 fix: kde-neon: Append usr to cmake prefix path #5272 snapcraft merged Merged as e2daf4cbe: PR #5585 updates KDE Neon CMAKE_PREFIX_PATH entries to use SDK usr subdirectories, fixing #5272; released in Snapcraft 8.10.1.
73%
#5104 extensions/kde-neon-6: fix XDG_CONFIG_DIRS snapcraft merged Merged PR #5104, adding $SNAP_REAL_HOME/.config to XDG_CONFIG_DIRS so KDE Neon 6 can locate kdeglobals theming settings.
73%
#4884 kde-neon-6: Fix long standing theming issues with Qt snaps. snapcraft merged Merged as fb1834ce8, adding qt-common-themes interfaces to kde-neon-6. Reverted from the 8.4 hotfix by #5013 but retained on main.
72%
#5917 feat(kde-neon-6): lxqt and kvantum theme support snapcraft merged Merged PR #5917 at commit 70275c88a, adding lxqt-support content snaps and LXQt/Kvantum theme support to kde-neon-6; it closed #5904.
71%
#1244 repo: enable KDE Neon. snapcraft merged Merged in canonical/snapcraft#1244 via commit 0b70903dc, adding the neon distribution ID so Snapcraft recognizes KDE Neon.
71%