← 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 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.

Suggested action:

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
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.
qwen3.6-35b-a3b-mtp-q6 Merged into hotfix/7.2. The linter was updated to exclude the dynamic linker from missing library checks, resolving false positive warnings.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#3978 linters: fix classic linter interpreter warning snapcraft merged Merged into hotfix/7.2. Resolves a classic linter interpreter warning by conditionally verifying the interpreter section. Approved, passed CI, and achieved 100% diff coverage.
77%
#3872 linters: add missing library linter snapcraft merged Merged changes add a missing library linter to list ELF dependencies in the payload directory and fix filename filtering in the classic linter. Approved, passed CI, and increased coverage by 0.02%.
75%