linters: fix classic linter interpreter warning
Metadata
Current evaluation
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.
Suggested action: —
No scores available.
Issue body
Only verify if the interpreter is correctly set in files that have an interpreter section.
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`?
- [ ] Have you successfully run `pytest tests/unit`?
-----
CRAFT-1470
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into hotfix/7.2 to fix a classic linter interpreter warning. The change restricts interpreter checks to files with an interpreter section, maintaining 100% diff coverage without affecting overall metrics. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #3989 linters: don't list dynamic linker as missing library | snapcraft | merged | 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. | |
| #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%. | |
| #3849 linters: add classic confinement linter | snapcraft | merged | Merged addition of a classic confinement linter warning about incorrect elf binary interpreter or rpath values and staged libc. Adjusts existing elf infrastructure, passed review and CI, was rebased on main, and increased code coverage. | |
| #166 linters: re-enable pylint fixme info | craft-parts | merged | Merged a one-line change to re-enable pylint fixme info in the linters configuration. Approved by one reviewer, passed all CI checks, and resolved with zero outstanding comments. |