diff options
| author |  Kelvin M. Klann <kmk3.code@protonmail.com> | 2022-11-03 13:04:45 -0300 |
|---|---|---|
| committer | Kelvin M. Klann <kmk3.code@protonmail.com> | 2022-11-04 15:57:17 -0300 |
| commit | f03d98649788243d4f47b340858a08e8f0d0dc8c (patch) | |
| tree | e4be24e51842ca8abdcaed579a446587292fe357 /platform | |
| parent | RELNOTES: add build: Fix musl warnings (diff) | |
| download | firejail-f03d98649788243d4f47b340858a08e8f0d0dc8c.tar.gz firejail-f03d98649788243d4f47b340858a08e8f0d0dc8c.tar.zst firejail-f03d98649788243d4f47b340858a08e8f0d0dc8c.zip | |
docs: clarify that --appimage should appear before --profile
And fix the argument order in the examples to reflect that.
Background: The order in which these options appeared in the
documentation was inconsistent. src/man/firejail.txt used --appimage
before --profile and src/man/firejail-profile.txt used --profile before
--appimage. Then commit 44fefcac0 ("Make appimage examples consistent
with --appimage option short description", 2022-10-05) / PR #5402 was
made, which standardized on --profile before --appimage in both places.
But as mentioned by @rusty-snake[1], --appimage has be specified before
--profile in order for any `?HAS_APPIMAGE` conditionals inside of the
profile to evaluate to true.
So change the documentation to use and recommend the latter form.
Also, add --quiet to one example to make it clear that --appimage does
not have to be the first option (nor the last option before --profile).
[1] https://github.com/netblue30/firejail/pull/5402#issuecomment-1274889618
Diffstat (limited to 'platform')
0 files changed, 0 insertions, 0 deletions