On Wed, 6 Oct 2004, Leopold Toetsch via RT wrote:

> Andy Dougherty (via RT) wrote:

> I don't like the change, it just hides all warnings.

Odd.  I don't see that.  Hiding all warnings would certainly be
bad, but I still seem to get warnings.  I'll go back and look at that more
closely.

>  Further "make -s" is just broken now.

Oops.  I hadn't thought of that.  I didn't even know it existed -- I had
to go look it up!  Good catch.

If 'make -s' is indeed useful, then I agree we oughtn't be doing an end
run around it.  Hmm.  It might be possible and sensible to have
tools/dev/cc_flags.pl only print the command if $ENV{MAKEFLAGS} doesn't
contain 's'.  Or perhaps it's simpler and better to just retract the
patch.

> But I see of course the point of beeing able to see the current build
> command. What about that: cc_flags.pl prints the build command if it
> changes, i.e. if there are any changes except the filename(s) involved.

That's probably not that useful -- in the usual case, there are no
changes, so the 'Compiling with ....' line might be a couple of thousand
lines above the current executing command.

-- 
    Andy Dougherty              [EMAIL PROTECTED]

Reply via email to