libosmocore[master]: configure.ac: Enable -Werror by default

2017-06-22 Thread Neels Hofmeyr
Patch Set 7: Code-Review-1 after discussion in chat, we should not have -Werror on by default, instead add it in the contrib/jenkins script, by means of 'make CFLAGS+="-Werror" CPPFLAGS+="-Werror"' -- To view, visit https://gerrit.osmocom.org/2954 To unsubscribe, visit https://gerrit.osmocom.

libosmocore[master]: configure.ac: Enable -Werror by default

2017-06-21 Thread Neels Hofmeyr
Patch Set 5: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/2954 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1752cad1e34cb6e5f36edf3f538d803eff29d18c Gerrit-PatchSet: 5 Gerrit-Project: libosmocore Gerrit-Branch: master Ger

libosmocore[master]: configure.ac: Enable -Werror by default

2017-06-19 Thread Pau Espin Pedrol
Patch Set 5: I didn't. Can you give me some info on how to test that? Do I need a debian based system? -- To view, visit https://gerrit.osmocom.org/2954 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1752cad1e34cb6e5f36edf3f538d803eff2

libosmocore[master]: configure.ac: Enable -Werror by default

2017-06-19 Thread Max
Patch Set 5: Have this been tested with .deb package building? -- To view, visit https://gerrit.osmocom.org/2954 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1752cad1e34cb6e5f36edf3f538d803eff29d18c Gerrit-PatchSet: 5 Gerrit-Project:

[PATCH] libosmocore[master]: configure.ac: Enable -Werror by default

2017-06-18 Thread Pau Espin Pedrol
Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/2954 to look at the new patch set (#4). configure.ac: Enable -Werror by default Change-Id: I1752cad1e34cb6e5f36edf3f538d803eff29d18c --- M configure.ac 1 file changed, 2 insertions(+), 2 del

[PATCH] libosmocore[master]: configure.ac: Enable -Werror by default

2017-06-18 Thread Pau Espin Pedrol
Review at https://gerrit.osmocom.org/2954 configure.ac: Enable -Werror by default Change-Id: I1752cad1e34cb6e5f36edf3f538d803eff29d18c --- M configure.ac 1 file changed, 2 insertions(+), 2 deletions(-) git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/54/2954/1 diff --git a/