fix(application): support snap/craft environment variables
Metadata
Current evaluation
Merged PR #4607 as b41ebe6e4, adding SNAP/CRAFT environment-variable support to the application and tests.
Resolution: closed resolved
Reason: PR #4607 was approved and merged into Snapcraft as commit b41ebe6e4, resolving support for both Snapcraft-specific and Craft environment variables.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [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 `tox run -m lint`?
- [x] Have you successfully run `tox run -e test-py310`? (supported versions: `py39`, `py310`, `py311`, `py312`)
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #4607 as b41ebe6e4, adding SNAP/CRAFT environment-variable support to the application and tests. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged as commit b41ebe6e4, adding SNAP/CRAFT environment-variable support to the application and its tests after three approvals. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged after three approvals and passing CI. The PR adds snap/craft environment variable support across two files. Initial spread test failures and merge conflicts from a concurrent PR were resolved before final integration. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged after fixing spread tests and updating commit formatting. Conflicts arose when a related PR merged first, requiring rebasing. The fix adds snap/craft environment variable support with passing tests and coverage. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #3645 meta: parse environment variables (CRAFT-835) | snapcraft | merged | Merged #3645 into snapcraft/7.0 via 7e7089fb1, adding parsing of root- and app-level environment dictionaries from snapcraft.yaml. | |
| #4560 feat(craft-application): initial support | snapcraft | merged | Merged PR #4560 as 98550856e, adding initial craft-application support with fallback to Snapcraft's existing logic. | |
| #1583259 Snappy needs to influence environment variables in applications | snapcraft (launchpad) | closed | Snapcraft gained top-level and per-app environment keys in snap.yaml, merged in snapcraft#1103 (c66f55656). The review-tool SRU 0.44~16.04.1 was verified and released. | |
| #1533552 Please update snapcraft for new environment variables | snapcraft (launchpad) | closed | Updated examples for Ubuntu 16.04 snap environment variables in merged PR #234 (commit 04b1a13bc), replacing legacy names and removing SNAP_APP_TEMPDIR. |