← Back to issue list

core: resolve ld link first.

View original Github issue

Metadata

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

Current evaluation

Merged PR #1189 as commit 78d1883ca, resolving ld symlinks before existence checks in classic-confinement builds and fixing LP #1672495.

Resolution: closed resolved

Reason: PR #1189 was merged as commit 78d1883ca. It changed classic-confinement handling to resolve the dynamic linker path first, allowing symlinked ld binaries to be recognized.

Evaluated by: mr-cal-bot

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
stealth/space-bunny-alpha — closed resolved Merged PR #1189 as commit 78d1883ca, resolving ld symlinks before existence checks in classic-confinement builds and fixing LP #1672495.
stealth/space-bunny-alpha — closed resolved Merged PR #1189 (78d1883ca) to resolve ld symlinks before checking existence in classic-confinement builds, addressing LP #1672495.
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.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/snapcraft#1187 (confidence 95%)

    A separate zesty failure noted during testing was fixed by PR #1187; it was unrelated to the ld-symlink change.

Related issues

Issue Project State Summary Similarity
#1094 core: switch to using rpath for clasic confinement. snapcraft merged Merged PR #1094 (commit e174e46e6) switched core classic confinement from RUNPATH to RPATH, addressing LP #1657504.
76%
#1170 core: fix symlink resolution in get_core_dynamic_linker snapcraft merged Merged canonical/snapcraft#1170 as commit b19e9cf68, fixing relative symlink resolution in get_core_dynamic_linker so classic snap builds work on affected architectures.
73%
#1759 many: ensure classic confined binaries use the correct interpreter snapcraft merged Merged PR #1759 as commit 426fc8207, fixing #1662 by setting ELF .interp entries to the linker provided by core during prime.
70%