Yo Hal!

On Wed, 05 Apr 2017 13:57:59 -0700
Hal Murray <[email protected]> wrote:

> [email protected] said:
> > And Matt Selsky reports lots of warnings on Solaris.  Seems like
> > Solaris is not very POSIX...  I'm gonna remove the sign-conversion
> > check, no way to make it silent for all targets.   
> 
> You could make it conditional on the systems where it works.

There used to be a lot of OS conditionals in the waf,  Most got ripped
out as host detection got better.

In the specific case of -Wsign-conversion, it illuminates a bug in
Bison, so prolly not a good idea to have on by default.  It will just
annoy and confuse people.
 
> Or make it a configure option so we can easily work on fixing things.

I thinking -Wsign-comparison would be good with 'waf --enable-debug, 
but not for normal builds.

The debug part of waf needs works, in many places.

> >> Is it reasonable to tweak buildbot each time something sneaks
> >> through?  
> > Lost me?   
> 
> If buildbot was supposed to reject non-clean commits but missed
> something, is it practical to add something to buildbot to catch that
> case?

Define clean?  If something does not build, then buildbot fails.  
otherwise it passes.  There is the known bug that test failures do
not fail cause buildbot fail.  I think someone already filed an issue
on it.

RGDS
GARY
---------------------------------------------------------------------------
Gary E. Miller Rellim 109 NW Wilmington Ave., Suite E, Bend, OR 97703
        [email protected]  Tel:+1 541 382 8588

            Veritas liberabit vos. -- Quid est veritas?
    "If you can’t measure it, you can’t improve it." - Lord Kelvin

Attachment: pgpvsM0WPF8HD.pgp
Description: OpenPGP digital signature

_______________________________________________
devel mailing list
[email protected]
http://lists.ntpsec.org/mailman/listinfo/devel

Reply via email to