← Back to issue list

executor: standardize environment variable prefix

View original Github issue

Metadata

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

Current evaluation

Merged. Standardized part scriptlet environment variables to a common CRAFT_ prefix, replacing application-specific prefixes. Added CRAFT_PROJECT_DIR, rebased on main, and approved by reviewer.

Suggested action:

No scores available.

Issue body

Instead of using application-specific lifecycle context clients such as snapcraftctl or charmcraftctl, craft-parts provides craftctl as a generic client to be used by all applications. Following the same motivation, use a common `CRAFT_` prefix for environment variables used inside part scriptlets. Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com> - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? ----- CRAFT-936

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged. Standardized part scriptlet environment variables to a common CRAFT_ prefix, replacing application-specific prefixes. Added CRAFT_PROJECT_DIR, rebased on main, and approved by reviewer.
qwen3.6-35b-a3b-mtp-q6 Merged changes standardize environment variable prefixes to CRAFT_ across craft-parts. Replaces app-specific prefixes with a generic craftctl client, preserving Snapcraft naming for easier migration. Adds CRAFT_PROJECT_DIR.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#82 executor: set step environment variables (CRAFT-272) craft-parts merged Merged after two approvals and passing CI. Adds project and part environment variables to step execution. Implementation validates names and supports custom prefixes instead of automatic transformation.
77%
#336 fix global environment expansion rockcraft merged Merged fix for global environment expansion preventing CRAFT_ variable setup before parts lifecycle. Updates tests, adds CRAFT_PROJECT_VERSION to spread tests, and resolves issue #333 after local verification and approval.
72%