← Back to issue list

executor: take permissions into account for collisions

View original Github issue

Metadata

Project
craft-parts
Number
#305
Type
pull request
State
merged
Author
tigarmo
Labels
Created
Updated
Closed

Current evaluation

Merged. Updates the executor to flag path collisions when both paths have permission definitions. Uses a simplified check without evaluating matching definitions. Approved by one reviewer, passed CI, and changed six files.

Suggested action:

No scores available.

Issue body

Two paths are considered to be colliding if they both have permissions definitions. The definitions are not "evaluated" to see if they possibly match (that is, if both sets of definitions would generate the same result) for simplicity, but this can be improved in the future. - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged. Updates the executor to flag path collisions when both paths have permission definitions. Uses a simplified check without evaluating matching definitions. Approved by one reviewer, passed CI, and changed six files.
qwen3.6-35b-a3b-mtp-q6 Merged executor update that flags paths with permissions definitions as collisions. Definitions are not evaluated for matching results to maintain simplicity, with potential future improvements noted.

Update history

No update history recorded yet.

Related issues

No related issues found above the similarity threshold.