svn commit: r213765 - head/sys/dev/aic7xxx/aicasm

[John Nielsen]

On Oct 14, 2010, at 12:17 PM, Jung-uk Kim wrote:

> On Thursday 14 October 2010 11:18 am, John Nielsen wrote:
>> I'm migrating a box from 8-STABLE to -CURRENT this morning and this
>> commit seems to break buildkernel:
>> 
>> cc -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
>> cc1: warnings being treated as errors
>> /usr/src/sys/dev/aic7xxx/aicasm/aicasm_scan.l:840: warning:
>> function declaration isn't a prototype *** Error code 1
>> 
>> I don't have any custom CFLAGS, etc defined. Commenting out the new
>> #defines from this patch allows the build to continue.
>> 
>> I'm guessing this doesn't happen on machines already running
>> -CURRENT or tinderbox (and others) would have noticed. However if
>> this is (going to be) a supported upgrade path from 8.x to 9.0
>> perhaps there's a way to make both clang and gcc from 8.x happy?
> 
> If you want to upgrade from N to N+1, "make buildworld" is required.

Sorry, I did do a "make buildworld" (which succeeded) prior to the "make buildkernel" (which failed as I described).