hooks should be made executable instead of erroring out
Metadata
Current evaluation
Automatic executable permission for hooks was implemented via merged pull request #2440. Snapcraft now sets hook permissions automatically instead of raising an error.
Suggested action: —
No scores available.
Issue body
Related to bug #1792078 in some way.
snapcraft today complains if a hook isn't executable. It should just make it executable instead.
PR up here: https://github.com/snapcore/snapcraft/pull/2440
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Automatic executable permission for hooks was implemented via merged pull request #2440. Snapcraft now sets hook permissions automatically instead of raising an error. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed after merging PR #2440. Snapcraft now automatically applies the executable permission to hooks instead of failing with an error. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed after merging PR #2440. Snapcraft now automatically applies executable permissions to hooks instead of raising an error. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | snapcraft automatically sets hooks as executable instead of throwing an error. The fix was merged via PR #2440, resolving the issue. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2440 meta: make hooks executable instead of complaining they're not | snapcraft | merged | Merged into master. Hooks are now made executable instead of triggering an error, resolving bug #1812003. The change passed CI, received approval, and achieved 100% diff coverage. Whitelisting was deferred to a follow-up PR. | |
| #2441 legacy/meta: make hooks executable instead of complaining they're not | snapcraft | merged | Merged cherry-pick of #2440 that fixes bug #1812003 by making hooks executable instead of raising an error. Approved by one reviewer, passed CI, and modified 2 files. | |
| #1026 Add support for hooks. | snapcraft | merged | Merged into master to add hook support to snapcraft. Hooks are now supported via convention by placing executables in the snap/hooks directory or installing them from parts. Approved and passed CI checks. |