current HEAD is broken on old gcc (and warnings on current clang)

[Gary E. Miller]

Yo Fred!

On Mon, 13 Feb 2017 13:26:31 -0800 (PST)
Fred Wright <fw at fwright.net> wrote:

> On Mon, 13 Feb 2017, Gary E. Miller wrote:
> > On Mon, 13 Feb 2017 13:02:49 -0800 (PST)
> > Fred Wright <fw at fwright.net> wrote:
> >  
> > > I suggest reverting 287a00b9a until a more judicious way to apply
> > > the appropriate options can be worked out.  
> >
> > Sorry, can't do that.  Can only move forward, not back.  
> 
> I contend that any properly conditional means of adding those flags
> will look completely different from 287a00b9a, so the change as it
> stands has no value (not to mention negative value on some
> platforms).  I've already locally reverted it just to get something
> that isn't broken.

Feel free to do what you will in your private branch.  Better yet, just
wait a day.

> > > Note that one can always get
> > > the same effect temporarily for testing just by specifying
> > > CFLAGS/LDFLAGS on the command line.  
> >
> > Except doing that with buildbot is a PITA.  
> 
> Don't the buildbots have a way to specify environment variables?
> Travis certainly does.
> 
> It would also be useful to have the buildbots feed from a testing
> branch, so that the master branch doesn't get used as a testing
> sandbox (e.g., 016eda025/4ca79909a).

I inherited a pretty cranky buildbot structure.  In theory what you
say is true, in practice it is a PTIA.  And a pointless PITA to
customize buildbot for each worker since that is the purpose of waf.

RGDS
GARY
---------------------------------------------------------------------------
Gary E. Miller Rellim 109 NW Wilmington Ave., Suite E, Bend, OR 97703
	gem at rellim.com  Tel:+1 541 382 8588

	    Veritas liberabit vos. -- Quid est veritas?
    "If you can’t measure it, you can’t improve it." - Lord Kelvin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 455 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ntpsec.org/pipermail/devel/attachments/20170213/bd9af4fd/attachment.bin>