diff options
author | Kelvin M. Klann <kmk3.code@protonmail.com> | 2022-09-27 05:48:52 -0300 |
---|---|---|
committer | Kelvin M. Klann <kmk3.code@protonmail.com> | 2022-11-28 04:50:16 -0300 |
commit | c693ea183aaf2ef9009e89046a6a213762441fdc (patch) | |
tree | 11663048bc732e71a970714dc612fced7bbfa95a /configure | |
parent | configure*: fix typo of "been" (diff) | |
download | firejail-c693ea183aaf2ef9009e89046a6a213762441fdc.tar.gz firejail-c693ea183aaf2ef9009e89046a6a213762441fdc.tar.zst firejail-c693ea183aaf2ef9009e89046a6a213762441fdc.zip |
configure*: fix trailing comma in HAVE_CONTRIB_INSTALL
Consider the current code:
AS_IF([test "x$enable_lts" = "xyes"], [
# ...
HAVE_CONTRIB_INSTALL="no",
Result of testing the value:
$ HAVE_CONTRIB_INSTALL="no",
$ printf '%s\n' "$HAVE_CONTRIB_INSTALL"
no,
$ test "x$HAVE_CONTRIB_INSTALL" = "xno" &&
echo equal || echo 'not equal'
not equal
This means that whenever HAVE_LTS is enabled, HAVE_CONTRIB_INSTALL is
always considered enabled when testing against "no".
But luckily, in the current code the latter variable is only tested
against "yes", so nothing should be affected:
$ git grep HAVE_CONTRIB_INSTALL |
grep -v -e '^configure:' -e '^configure.ac:'
Makefile:ifeq ($(HAVE_CONTRIB_INSTALL),yes)
config.mk.in:HAVE_CONTRIB_INSTALL=@HAVE_CONTRIB_INSTALL@
Added on commit d1acb31c9 ("compile time: enable LTS", 2021-02-28).
Diffstat (limited to 'configure')
-rwxr-xr-x | configure | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -3670,7 +3670,7 @@ if test "x$enable_lts" = "xyes"; then : | |||
3670 | HAVE_FILE_TRANSFER="" | 3670 | HAVE_FILE_TRANSFER="" |
3671 | HAVE_SUID="-DHAVE_SUID" | 3671 | HAVE_SUID="-DHAVE_SUID" |
3672 | BUSYBOX_WORKAROUND="no" | 3672 | BUSYBOX_WORKAROUND="no" |
3673 | HAVE_CONTRIB_INSTALL="no", | 3673 | HAVE_CONTRIB_INSTALL="no" |
3674 | 3674 | ||
3675 | fi | 3675 | fi |
3676 | 3676 | ||