Re: Remove debug echo

Garrett,

On Thu, Dec 1, 2011 at 2:15 PM, Garrett Cooper <yanegomi_at_gmail.com> 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
> > *** Error code 1
> >
> > Stop in /tmp.
> >
> > Otherwise diagnosing issues becomes a PITA with -j > 1 (with pmake I
> > have to start using some serious grep'ing, and if I'm lucky I can find
> > the source of error). If I get a few spare cycles I might just
> > implement it and post a patch somewhere (the entering and leaving
> > directory feature of gmake is really nice too, but it's less
> > important.. unless you have the same target in multiple directories)..
>
> 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 traceback when things fail. Pastebin available here, just
> in case the mailing list eats my patch: http://pastebin.com/dFqcDRfv
>
> $ cat ~/Makefile
> all:
>        cd $$HOME/foo; ${MAKE} $_at_
> $ cat ~/foo/Makefile
> all: foo bar barf yadda
>
> foo bar yadda:
>        _at_true
>
> baz:
>        _at_false
>
> barf: baz
> $ $PWD/make -j4 -f ~/Makefile all
> cd $HOME/foo; /usr/src/usr.bin/make/make all
> *** [baz] Error code 1
> 1 error
> *** [all] Error code 2
> 1 error
> $
>
> 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
> frankly tired of having to playing guessing games fishing through logs
> trying to determine build errors on FreeBSD if and when they do occur
> with pmake, and I'm sure that a number of developers and build/release
> engineers out there are in the same boat as I am.
>

I hit the same problem regularly. The patch looks good to me as well.

Max