]> xenbits.xensource.com Git - people/julieng/freebsd.git/commitdiff
Fix wrong PATH being set for world 'includes' stage after r289438.
authorbdrewery <bdrewery@FreeBSD.org>
Sat, 17 Oct 2015 18:22:18 +0000 (18:22 +0000)
committerbdrewery <bdrewery@FreeBSD.org>
Sat, 17 Oct 2015 18:22:18 +0000 (18:22 +0000)
The 'includes' target is currently a pseudo target in bsd.subdir.mk that
does 'cd ${.CURDIR} && ${MAKE} buildincludes && ${MAKE} installincludes',
versus all over targets that just recurse.

In Makefile.inc1 the older duplicated bsd.subdir.mk logic for calling
'includes' was being executed in each subdir directly, meaning 'cd lib && make
includes' became 'cd lib && make buildincludes && make installincludes'.  Now
that the bsd.subdir.mk logic is used it is calling 'make buildincludes && make
installincludes' from the top-level which pulls in the PATH=<default path>
from /Makefile.

The sub-make logic for 'includes' in bsd.subdir.mk was attempted to be removed
in r289282 but turned out to be wrong.  I have a working version now but
it is not yet ready for commit.  So for now in Makefile.inc1 split out
'includes' to 'buildincludes' and 'installincludes' which will avoid the
problem.

MFC after: 2 weeks
X-MFC-With: r289438
Sponsored by: EMC / Isilon Storage Division

Makefile.inc1

index c0ef145d2e9780d1b0a49e45ea29edcc40a37a97..44b1171cdfa70d6d530713b006edb9d4a97f1b40 100644 (file)
@@ -624,7 +624,8 @@ _includes:
        @echo "--------------------------------------------------------------"
        @echo ">>> stage 4.1: building includes"
        @echo "--------------------------------------------------------------"
-       ${_+_}cd ${.CURDIR}; ${WMAKE} SHARED=symlinks includes
+       ${_+_}cd ${.CURDIR}; ${WMAKE} SHARED=symlinks buildincludes
+       ${_+_}cd ${.CURDIR}; ${WMAKE} SHARED=symlinks installincludes
 _libraries:
        @echo
        @echo "--------------------------------------------------------------"
@@ -2268,7 +2269,9 @@ _xi-cross-tools:
 .endfor
 
 _xi-includes:
-       ${_+_}cd ${.CURDIR}; ${CD2MAKE} -f Makefile.inc1 includes \
+       ${_+_}cd ${.CURDIR}; ${CD2MAKE} -f Makefile.inc1 buildincludes \
+               DESTDIR=${XDDESTDIR}
+       ${_+_}cd ${.CURDIR}; ${CD2MAKE} -f Makefile.inc1 installincludes \
                DESTDIR=${XDDESTDIR}
 
 _xi-libraries: