Re: Remove debug echo

2011-12-15 Thread Max Khon
Garrett, On Thu, Dec 15, 2011 at 10:39 AM, Garrett Cooper wrote: >> If someone would please, PLEASE commit this.. I will give you beer, or > >> wine, or a copy of Skyrim, or a few months subscription to WoW, or > >> something else of value to you that we could negotiate :)... I'm quite > >> fran

Re: Remove debug echo

2011-12-14 Thread Garrett Cooper
On Wed, Dec 14, 2011 at 7:27 PM, Max Khon wrote: > Garrett, > > On Thu, Dec 1, 2011 at 2:15 PM, Garrett Cooper wrote: > >> >> I've attached a patch that makes make do what I would like it to do; >> there are some other items that require cleanup to achieve the `argv0' >> prefixing that's availabl

Re: Remove debug echo

2011-12-14 Thread Max Khon
Garrett, On Thu, Dec 1, 2011 at 2:15 PM, Garrett Cooper wrote: > I've attached a patch that makes make do what I would like it to do; > there are some other items that require cleanup to achieve the `argv0' > prefixing that's available in gmake, but this is good enough for a > meaningful traceb

Re: Remove debug echo

2011-12-01 Thread Garrett Cooper
On Thu, Dec 1, 2011 at 6:08 PM, David O'Brien wrote: > On Thu, Dec 01, 2011 at 10:04:08AM -0500, John Baldwin wrote: >> I think this is useful, perhaps send it to harti@ or jilles@ for review? > > I'd like to get some NetBSD bmake maintainers POV too. > We should reduce the needless diversion betw

Re: Remove debug echo

2011-12-01 Thread David O'Brien
On Thu, Dec 01, 2011 at 10:04:08AM -0500, John Baldwin wrote: > I think this is useful, perhaps send it to harti@ or jilles@ for review? I'd like to get some NetBSD bmake maintainers POV too. We should reduce the needless diversion between the two makes. -- -- David (obr...@freebsd.org) ___

Re: Remove debug echo

2011-12-01 Thread David O'Brien
On Wed, Nov 30, 2011 at 05:59:33PM -0800, Garrett Cooper wrote: > On Wed, Nov 30, 2011 at 5:43 PM, Alexander Best wrote: > > On Wed Nov 30 11, Garrett Cooper wrote: > >> On Wed, Nov 30, 2011 at 4:25 PM, Alexander Best > >> wrote: > >> ? ? pmake sucks as far as diagnostic output is concerned when

Re: Remove debug echo

2011-12-01 Thread Garrett Cooper
On Thu, Dec 1, 2011 at 9:17 AM, Freddie Cash wrote: > So, now that you've improved the default diagnostic output of make, how > about the OP's original request: >    make -s truly silent by removing unnecessary diagnostic messages when -s > is used?  :) > > [Thought I'd bring the thread back aroun

Re: Remove debug echo

2011-12-01 Thread Max Khon
Garrett, On Thu, Dec 1, 2011 at 2:15 PM, Garrett Cooper wrote: > What I really want is this: > > > > $ cat Makefile > > all: foo bar baz yadda > > > > foo bar yadda: > > > > baz: > >false > > $ gmake > > false > > gmake: *** [baz] Error 1 > > > > $ make all > > false > >

Re: Remove debug echo

2011-12-01 Thread Freddie Cash
So, now that you've improved the default diagnostic output of make, how about the OP's original request: make -s truly silent by removing unnecessary diagnostic messages when -s is used? :) [Thought I'd bring the thread back around to it's original purpose.] -- Freddie Cash fjwc...@gmail.com

Re: Remove debug echo

2011-12-01 Thread John Baldwin
On Thursday, December 01, 2011 2:15:11 am Garrett Cooper wrote: > On Wed, Nov 30, 2011 at 5:59 PM, Garrett Cooper wrote: > > On Wed, Nov 30, 2011 at 5:43 PM, Alexander Best wrote: > >> On Wed Nov 30 11, Garrett Cooper wrote: > >>> On Wed, Nov 30, 2011 at 4:25 PM, Alexander Best wrote: > >>> >

Re: Remove debug echo

2011-11-30 Thread Garrett Cooper
On Wed, Nov 30, 2011 at 5:59 PM, Garrett Cooper wrote: > On Wed, Nov 30, 2011 at 5:43 PM, Alexander Best wrote: >> On Wed Nov 30 11, Garrett Cooper wrote: >>> On Wed, Nov 30, 2011 at 4:25 PM, Alexander Best wrote: >>> > On Tue Nov 29 11, Warner Losh wrote: >>> >> kill it. >>> >> >>> >> Warner >>

Re: Remove debug echo

2011-11-30 Thread Garrett Cooper
On Wed, Nov 30, 2011 at 5:43 PM, Alexander Best wrote: > On Wed Nov 30 11, Garrett Cooper wrote: >> On Wed, Nov 30, 2011 at 4:25 PM, Alexander Best wrote: >> > On Tue Nov 29 11, Warner Losh wrote: >> >> kill it. >> >> >> >> Warner >> >> On Nov 29, 2011, at 2:07 PM, John Baldwin wrote: >> >> >> >>

Re: Remove debug echo

2011-11-30 Thread Alexander Best
On Wed Nov 30 11, Garrett Cooper wrote: > On Wed, Nov 30, 2011 at 4:25 PM, Alexander Best wrote: > > On Tue Nov 29 11, Warner Losh wrote: > >> kill it. > >> > >> Warner > >> On Nov 29, 2011, at 2:07 PM, John Baldwin wrote: > >> > >> > Any objections to this?  It removes a weird line during 'make -

Re: Remove debug echo

2011-11-30 Thread Garrett Cooper
On Wed, Nov 30, 2011 at 4:25 PM, Alexander Best wrote: > On Tue Nov 29 11, Warner Losh wrote: >> kill it. >> >> Warner >> On Nov 29, 2011, at 2:07 PM, John Baldwin wrote: >> >> > Any objections to this?  It removes a weird line during 'make -s >> > buildworld' >> > output and I think it was debug

Re: Remove debug echo

2011-11-30 Thread Alexander Best
On Tue Nov 29 11, Warner Losh wrote: > kill it. > > Warner > On Nov 29, 2011, at 2:07 PM, John Baldwin wrote: > > > Any objections to this? It removes a weird line during 'make -s > > buildworld' > > output and I think it was debugging accidentally left in in 213077 by > > Warner: > > > > In

Re: Remove debug echo

2011-11-29 Thread Warner Losh
kill it. Warner On Nov 29, 2011, at 2:07 PM, John Baldwin wrote: > Any objections to this? It removes a weird line during 'make -s buildworld' > output and I think it was debugging accidentally left in in 213077 by Warner: > > Index: newvers.sh > ===

Re: Remove debug echo

2011-11-29 Thread Doug Barton
On 11/29/2011 13:07, John Baldwin wrote: > Any objections to this? Nope. I wondered why it was there myself, but didn't care enough to ask. :) > It removes a weird line during 'make -s buildworld' > output and I think it was debugging accidentally left in in 213077 by Warner: > > Index: newvers

Remove debug echo

2011-11-29 Thread John Baldwin
Any objections to this? It removes a weird line during 'make -s buildworld' output and I think it was debugging accidentally left in in 213077 by Warner: Index: newvers.sh === --- newvers.sh (revision 228074) +++ newvers.sh (worki