← Back to issue list

linters: don't list dynamic linker as missing library

View original Github issue

Metadata

Project
snapcraft
Number
#3989
Type
pull request
State
merged
Author
cmatsuoka
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #3989 fixed the library linter so it no longer reports the dynamic linker as a missing library; landed as commit 97ce08c66.

Resolution: closed resolved

Reason: PR #3989 was approved and merged. Commit 97ce08c66 adds logic to skip the dynamic linker when generating missing-library warnings.

Evaluated by: mr-cal-bot

No scores available.

Issue body

When creating library linker warnings, don't consider the dynamic linker when checking for missing libraries. Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com> - [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/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 `pytest tests/unit`? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #3989 fixed the library linter so it no longer reports the dynamic linker as a missing library; landed as commit 97ce08c66.
stealth/space-bunny-alpha — closed resolved Merged #3989 to exclude the ELF dynamic linker from missing-library warnings; resolution recorded in commit 97ce08c66.
stealth/space-bunny-alpha — closed resolved Merged PR #3989 into hotfix/7.2, stopping the linter from incorrectly reporting the dynamic linker as a missing library.
qwen/qwen3.6-35b-a3b — — Merged into hotfix/7.2. Updated the linter to exclude the dynamic linker from missing library warnings, resolving false positive linker errors. Approved and passed CI.
qwen3.6-35b-a3b-mtp-q6 — — Merged into hotfix/7.2. The linter now excludes the dynamic linker from missing library checks, eliminating false positive linker warnings.

Update history

No update history recorded yet.

Related work

  • Related To: snapcore/snapcraft#3872 (confidence 90%)

    Commit c5f4d8b53 introduced the missing-library linter later corrected by #3989.

Related issues

Issue Project State Summary Similarity
#3872 linters: add missing library linter snapcraft merged Merged PR #3872 as c5f4d8b53, adding a linter for missing ELF payload dependencies and fixing filename filtering in the classic linter.
80%
#4293 linters: specify which package offers a missing library snapcraft closed Package ownership details for missing-library warnings landed via merged PR #4430 (commit 09edca316), replacing the unmerged implementation in #4293.
72%