HEAD after 9-BETA1 broken

[René Ladan]

2011/8/10 Alexander Best <arundel at freebsd.org>:
> On Wed Aug 10 11, René Ladan wrote:
>> Hi,
>>
>> it seems that HEAD is broken somewhere after 9.0-BETA1 amd64:
>>
>> acer % pwd
>> /usr/src
>> acer % svn info
>> Path: .
>> URL: svn+ssh://svn.freebsd.org/base/head
>> Repository Root: svn+ssh://svn.freebsd.org/base
>> Repository UUID: ccf9f872-aa2e-dd11-9fc8-001c23d0bc1f
>> Revision: 224747
>> Node Kind: directory
>> Schedule: normal
>> Last Changed Author: kevlo
>> Last Changed Rev: 224747
>> Last Changed Date: 2011-08-10 08:17:06 +0200 (wo, 10 aug 2011)
>>
>> acer % svn diff
>> Index: sys/modules/Makefile
>> ===================================================================
>> --- sys/modules/Makefile      (revision 224747)
>> +++ sys/modules/Makefile      (working copy)
>> @@ -335,6 +335,7 @@
>>       ${_xe} \
>>       xfs \
>>       xl \
>> +     xtaf \
>>       ${_zfs} \
>>       zlib \
>>
>> Index: sys/modules/geom/Makefile
>> ===================================================================
>> --- sys/modules/geom/Makefile (revision 224747)
>> +++ sys/modules/geom/Makefile (working copy)
>> @@ -28,6 +28,7 @@
>>       geom_vinum \
>>       geom_virstor \
>>       geom_vol_ffs \
>> +     geom_xbox360 \
>>       geom_zero
>>
>>  .include <bsd.subdir.mk>
>>
>> acer % cat /usr/src/sys/amd64/conf/ACER
>> include GENERIC
>> options       CAPABILITIES
>>
>> acer % make buildkernel
>>
>> --------------------------------------------------------------
>> >>> Kernel build for ACER started on Wed Aug 10 08:21:53 CEST 2011
>> --------------------------------------------------------------
>> ===> ACER
>> mkdir -p /usr/obj/usr/src/sys
>>
>> [...]
>>
>> --------------------------------------------------------------
>> >>> stage 2.3: build tools
>> --------------------------------------------------------------
>> cd /usr/obj/usr/src/sys/ACER;
>> MAKESRCPATH=/usr/src/sys/dev/aic7xxx/aicasm  make SSP_CFLAGS=
>> -DNO_CPU_CFLAGS -DNO_CTF  -f /usr/src/sys/dev/aic7xxx/aicasm/Makefile
>> Warning: Object directory not changed from original /usr/obj/usr/src/sys/ACER
>> yacc -b aicasm_gram  -d -o aicasm_gram.c
>> /usr/src/sys/dev/aic7xxx/aicasm/aicasm_gram.y
>> yacc -b aicasm_macro_gram -p mm -d -o aicasm_macro_gram.c
>> /usr/src/sys/dev/aic7xxx/aicasm/aicasm_macro_gram.y
>> clang -O2 -pipe -nostdinc -I/usr/include -I.
>> -I/usr/src/sys/dev/aic7xxx/aicasm -std=gnu99  -Wsystem-headers -Werror
>> -Wall -Wno-format-y2k -W -Wno-unused-parameter -Wstrict-prototypes
>> -Wmissing-prototypes -Wpointer-arith -Wreturn-type -Wcast-qual
>> -Wwrite-strings -Wswitch -Wshadow -Wunused-parameter -Wcast-align
>> -Wno-pointer-sign -c /usr/src/sys/dev/aic7xxx/aicasm/aicasm.c
>> clang -O2 -pipe -nostdinc -I/usr/include -I.
>> -I/usr/src/sys/dev/aic7xxx/aicasm -std=gnu99  -Wsystem-headers -Werror
>> -Wall -Wno-format-y2k -W -Wno-unused-parameter -Wstrict-prototypes
>> -Wmissing-prototypes -Wpointer-arith -Wreturn-type -Wcast-qual
>> -Wwrite-strings -Wswitch -Wshadow -Wunused-parameter -Wcast-align
>> -Wno-pointer-sign -c /usr/src/sys/dev/aic7xxx/aicasm/aicasm_symbol.c
>> clang -O2 -pipe -nostdinc -I/usr/include -I.
>> -I/usr/src/sys/dev/aic7xxx/aicasm -std=gnu99  -Wsystem-headers -Werror
>> -Wall -Wno-format-y2k -W -Wno-unused-parameter -Wstrict-prototypes
>> -Wmissing-prototypes -Wpointer-arith -Wreturn-type -Wcast-qual
>> -Wwrite-strings -Wswitch -Wshadow -Wunused-parameter -Wcast-align
>> -Wno-pointer-sign -c aicasm_gram.c
>> clang -O2 -pipe -nostdinc -I/usr/include -I.
>> -I/usr/src/sys/dev/aic7xxx/aicasm -std=gnu99  -Wsystem-headers -Werror
>> -Wall -Wno-format-y2k -W -Wno-unused-parameter -Wstrict-prototypes
>> -Wmissing-prototypes -Wpointer-arith -Wreturn-type -Wcast-qual
>> -Wwrite-strings -Wswitch -Wshadow -Wunused-parameter -Wcast-align
>> -Wno-pointer-sign -c aicasm_macro_gram.c
>> lex -t   /usr/src/sys/dev/aic7xxx/aicasm/aicasm_scan.l > aicasm_scan.c
>> clang -O2 -pipe -nostdinc -I/usr/include -I.
>> -I/usr/src/sys/dev/aic7xxx/aicasm -std=gnu99  -Wsystem-headers -Werror
>> -Wall -Wno-format-y2k -W -Wno-unused-parameter -Wstrict-prototypes
>> -Wmissing-prototypes -Wpointer-arith -Wreturn-type -Wcast-qual
>> -Wwrite-strings -Wswitch -Wshadow -Wunused-parameter -Wcast-align
>> -Wno-pointer-sign -c aicasm_scan.c
>> /usr/src/sys/dev/aic7xxx/aicasm/aicasm_scan.l:843:12: error: function 'input' is
>>       not needed and will not be emitted
>>       [-Werror,-Wunneeded-internal-declaration]
>> static int input(void)
>>            ^
>
> problem here is that -Werror got set. could you try removing the following
> part from your make.conf:
>
> .if !empty(.CURDIR:M/usr/src*)
> # Don't die on warnings
> NO_WERROR=
> WERROR=
> .endif
>
> and add
>
> # Don't die on warnings
> NO_WERROR=
> WERROR=
>
> to your src.conf?
>
This works, but why did the conditional version work for buildworld
and for earlier buildkernel invocations?

René
--
http://www.rene-ladan.nl/

GPG fingerprint = E738 5471 D185 7013 0EE0  4FC8 3C1D 6F83 12E1 84F6
(subkeys.pgp.net)