← Back to issue list

project loader: better error message for classic

View original Github issue

Metadata

Project
snapcraft
Number
#1025
Type
pull request
State
merged
Author
sergiusens
Labels
—
Created
Updated
Closed

Current evaluation

Merged to improve the error message when classic confinement requires the core snap. Resolved Launchpad bug #1650946 after passing CI and review.

Suggested action: —

No scores available.

Issue body

When compiling using classic confinement the core snap is needed, the check is correct but the error message is confusing. LP: #1650946 Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Merged to improve the error message when classic confinement requires the core snap. Resolved Launchpad bug #1650946 after passing CI and review.
qwen3.6-35b-a3b-mtp-q6 — — Merged into master. Updated the project loader to display a clearer error message when classic confinement requires the core snap, resolving LP #1650946. Coverage remained at 96.39%.
qwen3.6-35b-a3b-mtp-q6 — — Merged into master. Improved the project loader error message for classic confinement compilation to clarify the core snap requirement, resolving LP #1650946. Code coverage remained at 96.39%.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1189 core: resolve ld link first. snapcraft merged Merged a fix for classic confinement builds that checks for symbolic links on the ld executable before verifying its existence. Integrated despite unrelated CI failures and pending reviews.
73%
#2485 project loader: remove special LD_LIBRARY_FLAGS handling for classic snapcraft merged Merged removal of special LD_LIBRARY_FLAGS handling for classic snaps to prevent libc6 mismatches and undefined behavior. Improved library crawling made the logic obsolete. Travis CI updated to edge. Approved, CI passed, 7 files changed.
72%
#4216 meta: omit LD_LIBRARY_PATH and PATH in classic confinement snapcraft merged Merged to omit default LD_LIBRARY_PATH and PATH in classic confinement, resolving #4187. Post-merge comments warn the change may break existing classic snaps relying on the previous PATH behavior.
70%
#946 meta: support classic confinment snapcraft merged Merged. Adds support for the confinement property in classic confinement mode. Approved by two reviewers, passed CI, and modified three files with a +5/-4 line change. Resolves Launchpad bug #1647422.
70%