svn commit: r216662 - head/share/mk

2010-12-22 Thread Nathan Whitehorn
Author: nwhitehorn
Date: Wed Dec 22 14:59:22 2010
New Revision: 216662
URL: http://svn.freebsd.org/changeset/base/216662

Log:
  Enable build of FDT components by default. dtc is a required build tool
  for all FDT-enabled kernels, and having it off by default means that
  building these kernels fails by default. This fixes FDT-related build
  failures in make universe on ARM and PowerPC.
  
  Reviewed by:  imp

Modified:
  head/share/mk/bsd.own.mk

Modified: head/share/mk/bsd.own.mk
==
--- head/share/mk/bsd.own.mkWed Dec 22 14:49:18 2010(r216661)
+++ head/share/mk/bsd.own.mkWed Dec 22 14:59:22 2010(r216662)
@@ -326,6 +326,7 @@ _clang_no=CLANG
 DICT \
 DYNAMICROOT \
 EXAMPLES \
+FDT \
 FLOPPY \
 FORTH \
 FP_LIBC \
@@ -421,7 +422,6 @@ MK_${var}:= yes
 BIND_XML \
 BSD_GREP \
 ${_clang_no} \
-FDT \
 GPIO \
 HESIOD \
 IDEA
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to svn-src-all-unsubscr...@freebsd.org


Re: svn commit: r216662 - head/share/mk

2010-12-22 Thread Rafal Jaworowski

On 2010-12-22, at 15:59, Nathan Whitehorn wrote:

 Author: nwhitehorn
 Date: Wed Dec 22 14:59:22 2010
 New Revision: 216662
 URL: http://svn.freebsd.org/changeset/base/216662
 
 Log:
  Enable build of FDT components by default. dtc is a required build tool
  for all FDT-enabled kernels, and having it off by default means that
  building these kernels fails by default. This fixes FDT-related build
  failures in make universe on ARM and PowerPC.
 
  Reviewed by: imp

Thanks a lot for taking care of this!

Rafal

___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to svn-src-all-unsubscr...@freebsd.org


Re: svn commit: r216662 - head/share/mk

2010-12-22 Thread Bjoern A. Zeeb

On Wed, 22 Dec 2010, Nathan Whitehorn wrote:


Author: nwhitehorn
Date: Wed Dec 22 14:59:22 2010
New Revision: 216662
URL: http://svn.freebsd.org/changeset/base/216662

Log:
 Enable build of FDT components by default. dtc is a required build tool
 for all FDT-enabled kernels, and having it off by default means that
 building these kernels fails by default. This fixes FDT-related build
 failures in make universe on ARM and PowerPC.


Thanks a lot!  I wonder if it is arch specific conditional?  Is there
any value in having it on non-FDT kernels/architectures?

I am happily looking forward to a usable universe output again:-))



 Reviewed by:   imp

Modified:
 head/share/mk/bsd.own.mk

Modified: head/share/mk/bsd.own.mk
==
--- head/share/mk/bsd.own.mkWed Dec 22 14:49:18 2010(r216661)
+++ head/share/mk/bsd.own.mkWed Dec 22 14:59:22 2010(r216662)
@@ -326,6 +326,7 @@ _clang_no=CLANG
DICT \
DYNAMICROOT \
EXAMPLES \
+FDT \
FLOPPY \
FORTH \
FP_LIBC \
@@ -421,7 +422,6 @@ MK_${var}:= yes
BIND_XML \
BSD_GREP \
${_clang_no} \
-FDT \
GPIO \
HESIOD \
IDEA



--
Bjoern A. Zeeb  Welcome a new stage of life.
ks Going to jail sucks -- bz All my daemons like it!
  http://www.freebsd.org/doc/en_US.ISO8859-1/books/handbook/jails.html
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to svn-src-all-unsubscr...@freebsd.org


Re: svn commit: r216662 - head/share/mk

2010-12-22 Thread Nathan Whitehorn

On 12/22/10 10:18, Bjoern A. Zeeb wrote:

On Wed, 22 Dec 2010, Nathan Whitehorn wrote:


Author: nwhitehorn
Date: Wed Dec 22 14:59:22 2010
New Revision: 216662
URL: http://svn.freebsd.org/changeset/base/216662

Log:
 Enable build of FDT components by default. dtc is a required build tool
 for all FDT-enabled kernels, and having it off by default means that
 building these kernels fails by default. This fixes FDT-related build
 failures in make universe on ARM and PowerPC.


Thanks a lot!  I wonder if it is arch specific conditional?  Is there
any value in having it on non-FDT kernels/architectures?


There's no good way to make it arch-specific, according to Warner. If we 
grow such support, it could be made that way, but I don't think the 300 
K is enough to worry about. Also (hopefully), the list of architectures 
requiring it will continue to grow in the future.



I am happily looking forward to a usable universe output again:-))

Aside from some MIPS kernels requiring 64-bit toolchains not yet 
connected to the build, all make universe failures are now 100% genuine 
problems. Have fun, all :)

-Nathan

___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to svn-src-all-unsubscr...@freebsd.org


Re: svn commit: r216662 - head/share/mk

2010-12-22 Thread Warner Losh

On 12/22/2010 09:18, Bjoern A. Zeeb wrote:

On Wed, 22 Dec 2010, Nathan Whitehorn wrote:


Author: nwhitehorn
Date: Wed Dec 22 14:59:22 2010
New Revision: 216662
URL: http://svn.freebsd.org/changeset/base/216662

Log:
 Enable build of FDT components by default. dtc is a required build tool
 for all FDT-enabled kernels, and having it off by default means that
 building these kernels fails by default. This fixes FDT-related build
 failures in make universe on ARM and PowerPC.


Thanks a lot!  I wonder if it is arch specific conditional?  Is there
any value in having it on non-FDT kernels/architectures?

I am happily looking forward to a usable universe output again:-))

FDT enables building a tool that is sometimes used when building the kernel.

The problem is that it is a cross tool, and building things 
conditionally based on the architecture for cross tools is not well 
supported today.  The gross hack that attempted to do this was too hard 
to fix, so it is better to be always on.  The resulting bloat is tiny, 
it can be turned off if you want and it can be made conditional again 
once we have a better way to specify things like this conditionally.


Warner




 Reviewed by:imp

Modified:
 head/share/mk/bsd.own.mk

Modified: head/share/mk/bsd.own.mk
== 


--- head/share/mk/bsd.own.mkWed Dec 22 14:49:18 2010(r216661)
+++ head/share/mk/bsd.own.mkWed Dec 22 14:59:22 2010(r216662)
@@ -326,6 +326,7 @@ _clang_no=CLANG
DICT \
DYNAMICROOT \
EXAMPLES \
+FDT \
FLOPPY \
FORTH \
FP_LIBC \
@@ -421,7 +422,6 @@ MK_${var}:=yes
BIND_XML \
BSD_GREP \
${_clang_no} \
-FDT \
GPIO \
HESIOD \
IDEA





___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to svn-src-all-unsubscr...@freebsd.org


Re: svn commit: r216662 - head/share/mk

2010-12-22 Thread Warner Losh

On 12/22/2010 09:25, Nathan Whitehorn wrote:

On 12/22/10 10:18, Bjoern A. Zeeb wrote:

On Wed, 22 Dec 2010, Nathan Whitehorn wrote:


Author: nwhitehorn
Date: Wed Dec 22 14:59:22 2010
New Revision: 216662
URL: http://svn.freebsd.org/changeset/base/216662

Log:
 Enable build of FDT components by default. dtc is a required build 
tool

 for all FDT-enabled kernels, and having it off by default means that
 building these kernels fails by default. This fixes FDT-related build
 failures in make universe on ARM and PowerPC.


Thanks a lot!  I wonder if it is arch specific conditional?  Is there
any value in having it on non-FDT kernels/architectures?


There's no good way to make it arch-specific, according to Warner. If 
we grow such support, it could be made that way, but I don't think the 
300 K is enough to worry about. Also (hopefully), the list of 
architectures requiring it will continue to grow in the future.



I am happily looking forward to a usable universe output again:-))

Aside from some MIPS kernels requiring 64-bit toolchains not yet 
connected to the build, all make universe failures are now 100% 
genuine problems. Have fun, all :)


I was going to do a universe run after all the latest changes and try to 
fix that part of things, since those are rather important mips platforms 
to have connected and working :)


Warner

-Nathan





___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to svn-src-all-unsubscr...@freebsd.org