Re: Remove debug echo

From: Max Khon <fjoe_at_samodelkin.net>
Date: Thu, 15 Dec 2011 10:27:36 +0700
Garrett,

On Thu, Dec 1, 2011 at 2:15 PM, Garrett Cooper <yanegomi_at_gmail.com> 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 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.
>

Can you explain why did you remove MESSAGE() invocations in your patch?
Other than that the patch looks good to me.

Max
Received on Thu Dec 15 2011 - 02:27:38 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:40:21 UTC