core: resolve ld link first.
Metadata
Current evaluation
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.
Suggested action: —
No scores available.
Issue body
When building with classic confinement the logic was to check
if what was defined as ld existed, but it wouldn't if it was
a symbolic link so check for that in the first check as well.
LP: #1672495
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix to resolve symbolic links for the ld linker before existence checks, preventing classic confinement build failures. Unrelated CI errors did not block the merge. Resolves LP #1672495. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix for classic confinement builds that failed to detect the ld linker when it was a symbolic link. The change adds symlink resolution to the initial check. CI failures were unrelated and addressed separately. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1170 core: fix symlink resolution in get_core_dynamic_linker | snapcraft | merged | Merged fix for symlink resolution in get_core_dynamic_linker. Relative symlinks were incorrectly expanded relative to /snap/core/current, breaking classic snap builds on architectures like i386. Approved by reviewers, passed CI, and merged. Verified via integration tests. | |
| #1094 core: switch to using rpath for clasic confinement. | snapcraft | merged | Merged after reviewer approval and CI checks. The change switches the core module to use RPATH for classic confinement, resolving LP #1657504. The minimal two-file diff adjusts ELF dynamic linker path resolution behavior. | |
| #1025 project loader: better error message for classic | snapcraft | merged | Merged to improve the error message when classic confinement requires the core snap. Resolved Launchpad bug #1650946 after passing CI and review. | |
| #1759 many: ensure classic confined binaries use the correct interpreter | snapcraft | merged | Merged. Resolves #1662 by correcting the ELF interpreter path for classic confined binaries during the prime step. Approved by reviewers and passed CI checks. |