spread tests: fix classic patchelf linker regex to match all arches
Metadata
Current evaluation
Merged a fix to spread tests that broadened the patchelf linker regex from ld-linux to ld.*. This ensures compatibility with architectures like ppc64el using alternative linker names. Approved by reviewer and passed CI checks.
Suggested action: —
No scores available.
Issue body
ppc64el and others may not follow the `ld-linux` convention, e.g.:
/snap/core18/current/lib64/ld64.so.2
Fix by broadening the match to ld.*.
Signed-off-by: Chris Patterson <chris.patterson@canonical.com>
- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `./runtests.sh static`?
- [ ] Have you successfully run `./runtests.sh tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged a fix to spread tests that broadened the patchelf linker regex from ld-linux to ld.*. This ensures compatibility with architectures like ppc64el using alternative linker names. Approved by reviewer and passed CI checks. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix for the classic patchelf linker regex in spread tests. The pattern was broadened to ld.* to support architectures like ppc64el that use non-standard linker naming conventions instead of ld-linux. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix broadening the patchelf linker regex in spread tests from ld-linux to ld.*. This resolves architecture-specific path mismatches, enabling correct test execution on ppc64el and other non-standard linker configurations. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #3242 tests: migrate legacy classic patchelf tests to spread | snapcraft | merged | Merged after reviewer approval and passing CI. The change migrated legacy classic patchelf tests to the spread framework, updating 13 files with a net reduction of 186 lines. | |
| #6350 test: fix spread tests | snapcraft | merged | Merged after fixing non-store spread tests and the OSV scanner. Pinned black to 26.3.1 to resolve patchelf segfaults and silent failures. Approved by two reviewers and passed CI checks. | |
| #13 test(spread): add more spread tests | snapcraft-rocks | merged | Successfully merged spread tests covering architectures, chisel slices, craftctl, package-repositories, and patchelf. Approved by two reviewers across 15 files (+206/-5 lines). |