On 2013-08-27, at 17:05 , Pekka Jääskeläinen <[email protected]> wrote:

> On 08/27/2013 08:00 PM, Erik Schnetter wrote:
>> I thus suggest to use -Werror by default, which forces people to notice 
>> warnings and act on them.
> 
> If it now passes with that then I suggest we turn it on by
> default during the development, but not in the release.
> 
> I have had some headaches when compiling pedantic projects
> which use -Werror and then my newer/different compiler than
> they used for development find new warnings that then "break
> the build".


Yes, same here. That's why I emphasized that we are not using -Wall, so that 
-Werror will complain only about the really iffy cases.

Alternatively, we can introduce a rule that commits must not introduce 
warnings, except in really special cases. A buildbot can then complain about 
warnings and point fingers (or whatever buildbots have as equivalent).

-erik

-- 
Erik Schnetter <[email protected]>
http://www.perimeterinstitute.ca/personal/eschnetter/

My email is as private as my paper mail. I therefore support encrypting
and signing email messages. Get my PGP key from http://pgp.mit.edu/.

Attachment: signature.asc
Description: Message signed with OpenPGP using GPGMail

------------------------------------------------------------------------------
Learn the latest--Visual Studio 2012, SharePoint 2013, SQL 2012, more!
Discover the easy way to master current and previous Microsoft technologies
and advance your career. Get an incredible 1,500+ hours of step-by-step
tutorial videos with LearnDevNow. Subscribe today and save!
http://pubads.g.doubleclick.net/gampad/clk?id=58040911&iu=/4140/ostg.clktrk
_______________________________________________
pocl-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/pocl-devel

Reply via email to