[MERGED] libosmo-sccp[master]: jenkins.sh: use --enable-werror configure flag, not CFLAGS

2018-03-12 Thread Neels Hofmeyr 
Neels Hofmeyr has submitted this change and it was merged.

Change subject: jenkins.sh: use --enable-werror configure flag, not CFLAGS
..


jenkins.sh: use --enable-werror configure flag, not CFLAGS

Change-Id: I5c3f11586d48a076479eb19ed80a11caad4251d8
---
M contrib/jenkins.sh
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Neels Hofmeyr: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh
index ecaa1fe..d15cebe 100755
--- a/contrib/jenkins.sh
+++ b/contrib/jenkins.sh
@@ -36,7 +36,7 @@
 set -x
 
 autoreconf --install --force
-./configure --enable-sanitize CFLAGS="-Werror" CPPFLAGS="-Werror"
+./configure --enable-sanitize --enable-werror
 $MAKE $PARALLEL_MAKE
 $MAKE distcheck \
   || cat-testlogs.sh

-- 
To view, visit https://gerrit.osmocom.org/7103
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I5c3f11586d48a076479eb19ed80a11caad4251d8
Gerrit-PatchSet: 1
Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Neels Hofmeyr

libosmo-sccp[master]: jenkins.sh: use --enable-werror configure flag, not CFLAGS

2018-03-12 Thread Neels Hofmeyr 

Patch Set 1:

marking +2 because I'm (barely) allowed to by the comment at 
https://gerrit.osmocom.org/7096 (03-09 13:04 CET)

-- 
To view, visit https://gerrit.osmocom.org/7103
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c3f11586d48a076479eb19ed80a11caad4251d8
Gerrit-PatchSet: 1
Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Neels Hofmeyr 
Gerrit-HasComments: No

libosmo-sccp[master]: jenkins.sh: use --enable-werror configure flag, not CFLAGS

2018-03-12 Thread Neels Hofmeyr 

Patch Set 1: Code-Review+2

-- 
To view, visit https://gerrit.osmocom.org/7103
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c3f11586d48a076479eb19ed80a11caad4251d8
Gerrit-PatchSet: 1
Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Neels Hofmeyr 
Gerrit-HasComments: No

[PATCH] libosmo-sccp[master]: jenkins.sh: use --enable-werror configure flag, not CFLAGS

2018-03-05 Thread Neels Hofmeyr 

Review at  https://gerrit.osmocom.org/7103

jenkins.sh: use --enable-werror configure flag, not CFLAGS

Change-Id: I5c3f11586d48a076479eb19ed80a11caad4251d8
---
M contrib/jenkins.sh
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.osmocom.org:29418/libosmo-sccp refs/changes/03/7103/1

diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh
index ecaa1fe..d15cebe 100755
--- a/contrib/jenkins.sh
+++ b/contrib/jenkins.sh
@@ -36,7 +36,7 @@
 set -x
 
 autoreconf --install --force
-./configure --enable-sanitize CFLAGS="-Werror" CPPFLAGS="-Werror"
+./configure --enable-sanitize --enable-werror
 $MAKE $PARALLEL_MAKE
 $MAKE distcheck \
   || cat-testlogs.sh

-- 
To view, visit https://gerrit.osmocom.org/7103
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5c3f11586d48a076479eb19ed80a11caad4251d8
Gerrit-PatchSet: 1
Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr