Re: Do not use -Werror by default in your modules without joining #testable

2017-04-03 Thread Emmanuele Bassi
On 3 April 2017 at 19:38, Nicolas Dufresne wrote: > Le dimanche 02 avril 2017 à 14:59 +0100, Emmanuele Bassi a écrit : >> Yes, I know: this would be slightly more easier if Continuous warned >> about build breakages via email (though I'm pretty sure email would >> still be a

Re: Do not use -Werror by default in your modules without joining #testable

2017-04-03 Thread Nicolas Dufresne
Le dimanche 02 avril 2017 à 14:59 +0100, Emmanuele Bassi a écrit : > Yes, I know: this would be slightly more easier if Continuous warned > about build breakages via email (though I'm pretty sure email would > still be a high latency medium that tends to be ignored); > nevertheless, joining the

Re: Do not use -Werror by default in your modules without joining #testable

2017-04-03 Thread Michael Catanzaro
On Sun, 2017-04-02 at 20:22 +0200, Sébastien Wilmet wrote: > To disable -Werror by default it's better to set the IS-RELEASE > parameter of AX_COMPILER_FLAGS to "yes": > > AX_COMPILER_FLAGS([WARN_CFLAGS], [WARN_LDFLAGS], [yes]) Yeah I think you're right, that's what Emmanuele wants... OTOH, this

Re: Do not use -Werror by default in your modules without joining #testable

2017-04-03 Thread Michael Catanzaro
On Mon, 2017-04-03 at 06:42 +0200, Sébastien Wilmet wrote: > Maybe this works: > module_autogenargs['epiphany'] = '--enable-Werror' Actually it does work. No clue what I was doing wrong when I tested this previously. :) module_autogenargs['epiphany'] = 'CFLAGS="-Werror"' works too. I didn't try