On 6 Sep 2013, at 11:36, Konstantin Belousov <kostik...@gmail.com> wrote:

> This is extremely rude. Such things are traditionally (and properly)
> done with Makefile.inc in our tree.  Look at the very beginning of
> mk/bsd.init.mk.

I was not aware of Makefile.inc, thank you.  

> But the whole commit is hack.  If you want to use C99 _Bool, use it directly
> instead of obfuscating the code through the build system.

My intent was to minimise diffs from the citrus code.  Our iconv.h leaks 
stdbool.h into c89 files that use it, which is currently breaking around a 
thousand ports (as would have been detected if there had been an exp run before 
it was enabled by default).  Removing this from the header fixes the ports, but 
breaks the modules that expect bool to be defined.  This was the minimal change 
that would allow it to continue to build.

The more elegant fix would be to modify all of the modules to explicitly 
include stdbool.h if they wanted to use it, but this would then impose lots 
more work on whoever does the next import from upstream.

David
(Who is currently trying to get the ports tree into a useable state for 10.0)
_______________________________________________
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to