Merge lp://qastaging/~mvo/snap-confine/environment-file into lp://qastaging/~snappy-dev/snap-confine/trunk
Proposed by
Michael Vogt
Status: | Needs review |
---|---|
Proposed branch: | lp://qastaging/~mvo/snap-confine/environment-file |
Merge into: | lp://qastaging/~snappy-dev/snap-confine/trunk |
Diff against target: |
203 lines (+147/-0) 6 files modified
debian/usr.bin.ubuntu-core-launcher (+3/-0) src/environment.c (+85/-0) src/environment.h (+23/-0) src/main.c (+4/-0) tests/common.sh (+4/-0) tests/test_environment (+28/-0) |
To merge this branch: | bzr merge lp://qastaging/~mvo/snap-confine/environment-file |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Zygmunt Krynicki | Needs Fixing | ||
Review via email: mp+295237@code.qastaging.launchpad.net |
Description of the change
This implements passing environment variables that got defined via the snapd daemon. This is useful to avoid having to manually write wrappers in snapcraft. Instead the environment is part of meta/snap.yaml and will get generated automatically on install by snapd.
To post a comment you must log in.
Unmerged revisions
- 139. By Michael Vogt
-
fix missing dereference (thanks @zyga)
- 138. By Michael Vogt
-
kill final \n
- 137. By Michael Vogt
-
fix confinment, improve debug
- 136. By Michael Vogt
-
add support for adding an environment from a special environment file
Note that this does not yet work for LD_LIBRARY_PATH, the strong suspicion is that the apparmor profile in debian/ usr.bin. ubuntu- core-launcher prevents that via the apparmor environment scrubbing functionality.