SNAP_CONFINE_DEBUG should not trigger on empty strings
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| snap-confine |
Fix Released
|
Wishlist
|
Zygmunt Krynicki | ||
| snap-confine (Ubuntu) |
New
|
Undecided
|
Unassigned | ||
| Xenial |
Fix Committed
|
Undecided
|
Unassigned | ||
| Yakkety |
Fix Committed
|
Undecided
|
Unassigned | ||
Bug Description
[Impact]
snap-confine respects SNAP_CONFINE_DEBUG environment variable. The presence of this variable, regardless of what the value is, enables diagnostic output. To have more predictable and sane behaviour only specific values will trigger diagnostic output. The values are "yes", "no", "0" and "1".
[Test Case]
The test case can be found here:
https:/
The test case is ran automatically for each pull request and for each final release. It can be reproduced manually by executing the shell commands listed in the prepare/
The commands there assume that snapd and snap-confine are installed.
No other additional setup is necessary.
[Regression Potential]
* Regression potential is almost non-existent as the debug mode is not enabled by default and it was even easier to enable previously.
[Other Info]
* snap-confine is technically an integral part of snapd which has an SRU exception and is allowed to introduce new features and take advantage of accelerated procedure. For more information see https:/
== # Pre-SRU bug description follows # ==
The environment variable SNAP_CONFINE_DEBUG should respond to values like "yes" "no" "1" and "0" instead to just being defined (even if empty).
| Changed in snap-confine: | |
| status: | In Progress → Fix Released |
| description: | updated |
| tags: | added: verification-needed |
| Andy Whitcroft (apw) wrote Please test proposed package | #1 |
| Changed in snap-confine (Ubuntu Yakkety): | |
| status: | New → Fix Committed |
| Andy Whitcroft (apw) wrote | #2 |
| Changed in snap-confine (Ubuntu Xenial): | |
| status: | New → Fix Committed |
| Federico Gimenez (fgimenez) wrote | #3 |
| tags: |
added: verification-done removed: verification-needed |
Hello Zygmunt, or anyone else affected,
Accepted snap-confine into yakkety-proposed. The package will build now and be available at https:/
Please help us by testing this new package. See https:/
If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-
Further information regarding the verification process can be found at https:/