Re: toolchain/42242 (tools build broken)

[christos%zoulas.com@localhost (Christos Zoulas)]

The following reply was made to PR toolchain/42242; it has been noted by GNATS.

From: christos%zoulas.com@localhost (Christos Zoulas)
To: Ignatios Souvatzis <ignatios%cs.uni-bonn.de@localhost>
Cc: gnats-bugs%NetBSD.org@localhost, toolchain-manager%NetBSD.org@localhost, 
        netbsd-bugs%NetBSD.org@localhost, gnats-admin%NetBSD.org@localhost, 
is%NetBSD.org@localhost
Subject: Re: toolchain/42242 (tools build broken)
Date: Thu, 29 Oct 2009 10:57:40 -0400

 On Oct 29,  2:52pm, ignatios%cs.uni-bonn.de@localhost (Ignatios Souvatzis) 
wrote:
 -- Subject: Re: toolchain/42242 (tools build broken)
 
 | On Thu, Oct 29, 2009 at 09:04:09AM -0400, Christos Zoulas wrote:
 | 
 | > | 2. Fixing m4 didn't make toolchain lex work, as toolchain m4 wasn't
 | > | installed in time. It seems to me that the .WAIT magic used in the
 | > | tools Makefiles doesn't work as expected, but only forces early
 | > | installation of the tool immediately before the .WAIT in the list.
 | > | (This is from observation, not from fully understanding the
 | > |  higher-order Makefile magic used.)
 | > | 
 | > | Giving m4 its own .WAIT fixed this for me; maybe a full rewrite
 | > | of the Makefile magic would be better in the long run.
 | > 
 | > Are you sure this is needed. I will add it anyway, but it looks fishy
 | > to me.
 | 
 | which "this"?
 | 
 | 2., first paragraph, is needed. apb pointed out that it's even documented
 | in some comment of the tools/Makefile:
 | 
 | # For each .WAIT point, make sure the immediately preceding target is
 | # installed before building anything after that point.
 | # (dsl: which means that with: 'a b .WAIT c' the build of 'c' waits for the
 | # install of 'b', but not the install of 'a'.)
 | 
 | Given this, my 2nd paragraph is bogus.
 | 
 | I've committed the above patch, because I didn't see your message in
 | time; feel free to do corrections if needed.
 
 I will leave it as is :-)
 
 christos