Final sanity pass: xdev

[John Hein]

M. Warner Losh wrote at 08:08 +0900 on Mar 15, 2009:
 > In message: <18875.60334.947446.966085 at gromit.timing.com>
 >             John Hein <jhein at timing.com> writes:
 > : M. Warner Losh wrote at 09:00 -0600 on Mar 13, 2009:
 > :  > In message: <20090313125115.GU31961 at hoeg.nl>
 > :  >             Ed Schouten <ed at 80386.nl> writes:
 > :  > : Hi Warner,
 > :  > : 
 > :  > : The last couple of days/weeks I've been working a lot with LLVM+Clang.
 > :  > : I'm currently writing some BSD makefiles to see how hard it is to add
 > :  > : Clang to our base system, as a proof of concept.
 > :  > : 
 > :  > : A nice thing about Clang is that it seems to be very easy to compile in
 > :  > : multiple backends into the same Clang binary. If we ever switch to Clang
 > :  > : in the base system (who knows), it would be interesting to see whether
 > :  > : it's useful to just compile in all backends by default.
 > :  > : 
 > :  > : Anyway, I took a quick look at the xdev patch and it looks okay. :-)
 > :  > 
 > :  > Woot!  Please let me know if it causes problems.
 > : 
 > : I meant to mention this earlier - OSREL (referenced in the _xi-links:
 > : target) is not defined.  So ports looking for arm-freebsd8.0-cc (for
 > : instance) won't find it.
 > : 
 > : 
 > : An earlier patch of yours had:
 > : 
 > : .if !defined(OSREL)
 > : OSREL!= uname -r | sed -e 's/[-(].*//'
 > : .endif
 > : 
 > : But the latest (and the one committed) does not.
 > 
 > You are right... Dang.  I must have comitted the wrong thing...

One more thing... Now that we've switched to bsd ar, installing
gnu/usr.bin/binutils installs the gnu ar as gnu-ar.  Cross-built
ports can't find arm-freebsd8.0-ar

Adding usr.bin/ar to the cross tools does the job...

Index: Makefile.inc1
===================================================================
RCS file: /base/FreeBSD-CVS/src/Makefile.inc1,v
retrieving revision 1.620
diff -u -p -r1.620 Makefile.inc1
--- Makefile.inc1	13 Mar 2009 10:40:38 -0000	1.620
+++ Makefile.inc1	16 Mar 2009 05:11:12 -0000
@@ -1035,6 +1035,7 @@ cross-tools:
 .for _tool in \
     gnu/usr.bin/binutils \
     gnu/usr.bin/cc \
+    usr.bin/ar \
     usr.bin/sed \
     usr.bin/xlint/lint1 usr.bin/xlint/lint2 usr.bin/xlint/xlint \
     ${_btxld} \
@@ -1370,7 +1371,8 @@ _xb-build-tools:
 _xb-cross-tools:
 .for _tool in \
     gnu/usr.bin/binutils \
-    gnu/usr.bin/cc
+    gnu/usr.bin/cc \
+    usr.bin/ar
 	${_+_}@${ECHODIR} "===> xdev ${_tool} (obj,depend,all)"; \
 	cd ${.CURDIR}/${_tool}; \
 	${CDMAKE} DIRPRFX=${_tool}/ obj; \
@@ -1395,7 +1397,8 @@ _xi-cross-tools:
 	@echo "_xi-cross-tools"
 .for _tool in \
     gnu/usr.bin/binutils \
-    gnu/usr.bin/cc
+    gnu/usr.bin/cc \
+    usr.bin/ar
 	${_+_}@${ECHODIR} "===> xdev ${_tool} (install)"; \
 	cd ${.CURDIR}/${_tool}; \
 	${CDMAKE} DIRPRFX=${_tool}/ install DESTDIR=${XDDESTDIR}