linters: fix classic linter interpreter warning
Metadata
Current evaluation
Merged #3978 into hotfix/7.2 via 06ca48442, limiting the classic-linter interpreter check to files that contain an interpreter section.
Resolution: closed resolved
Reason: PR #3978 was merged into hotfix/7.2 as resolving commit 06ca48442. It fixed spurious interpreter warnings by validating the interpreter only when an interpreter section is present.
Evaluated by: mr-cal-bot
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 |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged #3978 into hotfix/7.2 via 06ca48442, limiting the classic-linter interpreter check to files that contain an interpreter section. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged into hotfix/7.2. Fixed the classic linter warning by checking the interpreter only when an interpreter section exists; resolving commit 06ca48442. | |
| 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 |
|---|---|---|---|---|
| #3849 linters: add classic confinement linter | snapcraft | merged | Merged #3849 in commit 82cf4c1d4, adding a classic-confinement linter for ELF interpreter, rpath, and staged-libc issues. | |
| #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. |