Re: CVS commit: src/tools

[enami tsugutomo <tsugutomo.enami%jp.sony.com@localhost>]

matthew green <mrg%NetBSD.org@localhost> writes:

> Module Name:  src
> Committed By: mrg
> Date:         Mon Dec 21 20:57:36 UTC 2009
> 
> Modified Files:
>       src/tools: Makefile
> 
> Log Message:
> move the build of pax before libelf.  fixes my build of tools/libelf,
> though i didn't look to see why libelf needs pax.
> 
> 
> To generate a diff of this commit:
> cvs rdiff -u -r1.130 -r1.131 src/tools/Makefile

    Index: tools/Makefile
    ===================================================================
    RCS file: /cvsroot/src/tools/Makefile,v
    retrieving revision 1.130
    retrieving revision 1.131
    diff -u -r1.130 -r1.131
    --- tools/Makefile        21 Dec 2009 18:21:17 -0000        1.130
    +++ tools/Makefile        21 Dec 2009 20:57:36 -0000        1.131
    @@ -43,12 +43,12 @@
             yacc .WAIT \
             awk .WAIT \
             lex .WAIT \
    +        pax .WAIT \
             libelf .WAIT \
             ${TOOLCHAIN_BITS} \
                     asn1_compile atf-compile cat cksum compile_et config db \
                     file lint1 \
                     makefs menuc mkcsmapper mkesdb mklocale mknod msgc \
    -                pax .WAIT \
                     disklabel .WAIT \
                     paxctl .WAIT \
                     fdisk .WAIT \

I guess this may break parallel build.  Before, building disklabel is
deferred but now it will be built more earlier.

The notation like below

        xxx .WAIT \
        yyy .WAIT \
        zzz ...

is quite misleading and it should be rewritten as follows instead:

        xxx \
        .WAIT yyy \
        .WAIT zzz ...

enami.