← Back to issue list

fix(gnome): update cmake prefix path

View original Github issue

Metadata

Project
snapcraft
Number
#5589
Type
pull request
State
merged
Author
mr-cal
Labels
Created
Updated
Closed

Current evaluation

Merged fix updating the cmake prefix path to usr for GNOME. Follows up on #5272 and complements the KDE fix in #5585. Approved by three reviewers, passed CI checks, and integrated into the main branch.

Suggested action:

No scores available.

Issue body

- [x] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `make lint`? - [x] Have you successfully run `make test`? --- This is a follow-up fix to #5272, where the cmake prefix path needs to be look in `usr`. This is fixed for KDE in #5585.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged fix updating the cmake prefix path to usr for GNOME. Follows up on #5272 and complements the KDE fix in #5585. Approved by three reviewers, passed CI checks, and integrated into the main branch.
qwen3.6-35b-a3b-mtp-q6 Merged a fix updating the cmake prefix path for GNOME to search in usr. This follow-up to #5272 aligns with the KDE fix in #5585, resolving build configuration inconsistencies.
qwen3.6-35b-a3b-mtp-q6 Merged fix updating the cmake prefix path to search in usr for GNOME. Resolves a follow-up to #5272 and aligns with the KDE adjustment in #5585.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#5461 feat(gnome): add cmake prefix path snapcraft merged Merged to resolve #5272. Injects CMAKE_PREFIX_PATH into the GNOME extension environment to prevent conflicts with part-level cmake parameters, fixing build failures for affected snaps like Remmina.
79%
#5585 fix: kde-neon: Append usr to cmake prefix path #5272 snapcraft merged Merged a fix for KDE Neon that appends usr to the CMake prefix path. Approved by two reviewers and passed integration and security checks. A conventional commits policy check failed but did not block the merge.
74%
#5310 fix: Fix cmake prefix path and remove invalid cmake root path. snapcraft merged Merged fix for cmake prefix path and removal of an invalid cmake root path. Resolves #5310. Approved by reviewers, passed CI, and targeted for Snapcraft 8.8 instead of 8.7 due to newer code dependencies.
71%