On Fri, Sep 06, 2013 at 09:46:45AM +0000, David Chisnall wrote:
> --- head/lib/libiconv_modules/Makefile        Fri Sep  6 09:08:40 2013        
> (r255296)
> +++ head/lib/libiconv_modules/Makefile        Fri Sep  6 09:46:44 2013        
> (r255297)
> @@ -2,6 +2,9 @@
>  
>  .include <bsd.own.mk>
>  
> +CFLAGS+=     -Dbool=_Bool
> +.export CFLAGS
> +
>  SUBDIR=      BIG5 DECHanyu EUC EUCTW GBK2K HZ ISO2022 JOHAB MSKanji UES 
> UTF1632 \
>       UTF7 UTF8 VIQR ZW iconv_none iconv_std mapper_646 mapper_none \
>       mapper_parallel mapper_serial mapper_std mapper_zone

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.

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.

Attachment: pgpsa13juBdOr.pgp
Description: PGP signature

Reply via email to