laforge has submitted this change. (
https://gerrit.osmocom.org/c/libgtpnl/+/38474?usp=email )
Change subject: build: fix bashims in configure.ac
......................................................................
build: fix bashims in configure.ac
$ foo=bar
$ foo+=zzz
dash: 2: foo+=zzz: not found
Fixes: 1.2.5-6-g0bd1e0d
Change-Id: I2c0a62ab80f480502dd2a1bcf4aef71e70152ea6
---
M configure.ac
1 file changed, 3 insertions(+), 3 deletions(-)
Approvals:
osmith: Looks good to me, but someone else must approve
laforge: Looks good to me, approved
Jenkins Builder: Verified
pespin: Looks good to me, but someone else must approve
diff --git a/configure.ac b/configure.ac
index 6eabd32..24658a4 100644
--- a/configure.ac
+++ b/configure.ac
@@ -71,9 +71,9 @@
if test x"$werror" = x"yes"
then
WERROR_FLAGS="-Werror"
- WERROR_FLAGS+=" -Werror=implicit-int -Werror=int-conversion
-Werror=old-style-definition"
- WERROR_FLAGS+=" -Wno-error=deprecated
-Wno-error=deprecated-declarations"
- WERROR_FLAGS+=" -Wno-error=cpp" # "#warning"
+ WERROR_FLAGS="$WERROR_FLAGS -Werror=implicit-int -Werror=int-conversion
-Werror=old-style-definition"
+ WERROR_FLAGS="$WERROR_FLAGS -Wno-error=deprecated
-Wno-error=deprecated-declarations"
+ WERROR_FLAGS="$WERROR_FLAGS -Wno-error=cpp" # "#warning"
regular_CFLAGS="$WERROR_FLAGS"
regular_CPPFLAGS="$WERROR_FLAGS"
fi
--
To view, visit https://gerrit.osmocom.org/c/libgtpnl/+/38474?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: libgtpnl
Gerrit-Branch: master
Gerrit-Change-Id: I2c0a62ab80f480502dd2a1bcf4aef71e70152ea6
Gerrit-Change-Number: 38474
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
