Re: [PATCH (?)] add -Werror to autotools and cmake ?

2012-10-01 Thread Pavel Sanda
Richard Heck wrote: > On 10/01/2012 03:45 AM, Lars Gullik Bj?nnes wrote: >> But we should perhaps have a configure switch for -Werror. Then we can >> argue later on what should be the default for it. >> > Perhaps for --enable-build-type=dev, it should be on, and for > --enable-build-type-rel, off

Re: [PATCH (?)] add -Werror to autotools and cmake ?

2012-10-01 Thread Jean-Marc Lasgouttes
Le 01/10/2012 15:46, Kornel Benko a écrit : Do all compiler have such a switch? And if yes, is it the same? I would say "yes" and then "no". Usually for autotools many such special settings are only done for gcc. JMarc

Re: Re: [PATCH (?)] add -Werror to autotools and cmake ?

2012-10-01 Thread Kornel Benko
Am Montag, 1. Oktober 2012 um 09:41:50, schrieb Richard Heck > On 10/01/2012 03:45 AM, Lars Gullik Bjønnes wrote: > > But we should perhaps have a configure switch for -Werror. Then we can > > argue later on what should be the default for it. > > > Perhaps for --enable-build-type=dev, it should b

Re: [PATCH (?)] add -Werror to autotools and cmake ?

2012-10-01 Thread Richard Heck
On 10/01/2012 03:45 AM, Lars Gullik Bjønnes wrote: But we should perhaps have a configure switch for -Werror. Then we can argue later on what should be the default for it. Perhaps for --enable-build-type=dev, it should be on, and for --enable-build-type-rel, off. rh

Re: [PATCH (?)] add -Werror to autotools and cmake ?

2012-10-01 Thread Lars Gullik Bjønnes
André Pönitz writes: | On Sat, Sep 29, 2012 at 04:53:00AM -0400, Scott Kostyshak wrote: >> I have been thinking about compiler warnings because of the recent >> discussion here: >> http://www.mail-archive.com/lyx-devel@lists.lyx.org/msg175099.html >> >> >From that thread, I have the understandin

Re: [PATCH (?)] add -Werror to autotools and cmake ?

2012-09-29 Thread Scott Kostyshak
On Sat, Sep 29, 2012 at 6:37 AM, André Pönitz wrote: > On Sat, Sep 29, 2012 at 04:53:00AM -0400, Scott Kostyshak wrote: >> I have been thinking about compiler warnings because of the recent >> discussion here: >> http://www.mail-archive.com/lyx-devel@lists.lyx.org/msg175099.html >> >> >From that t

Re: [PATCH (?)] add -Werror to autotools and cmake ?

2012-09-29 Thread André Pönitz
On Sat, Sep 29, 2012 at 04:53:00AM -0400, Scott Kostyshak wrote: > I have been thinking about compiler warnings because of the recent > discussion here: > http://www.mail-archive.com/lyx-devel@lists.lyx.org/msg175099.html > > >From that thread, I have the understanding that some developers think >

[PATCH (?)] add -Werror to autotools and cmake ?

2012-09-29 Thread Scott Kostyshak
I have been thinking about compiler warnings because of the recent discussion here: http://www.mail-archive.com/lyx-devel@lists.lyx.org/msg175099.html >From that thread, I have the understanding that some developers think it's important to fix warnings, although not if it is risky. I don't have mu