Re: [Qemu-devel] buildbot failure in qemu on default_s390

[Alexander Graf]

On 15.03.2012, at 01:11, Michael Roth <address@hidden> wrote:

> On Thu, Mar 15, 2012 at 01:01:04AM +0100, Andreas Färber wrote:
>> Am 15.03.2012 01:16, schrieb address@hidden:
>>> The Buildbot has detected a new failure on builder default_s390 while 
>>> building qemu.
>>> Full details are available at:
>>> http://buildbot.b1-systems.de/qemu/builders/default_s390/builds/190
>>> 
>>> Buildbot URL: http://buildbot.b1-systems.de/qemu/
>>> 
>>> Buildslave for this Build: qemu-s390.opensuse.org
>>> 
>>> Build Reason: The Nightly scheduler named 'nightly_default' triggered this 
>>> build
>>> Build Source Stamp: [branch master] HEAD
>>> Blamelist: 
>>> 
>>> BUILD FAILED: failed compile
>> 
>> Sorry...
>> 
>>  CC    lm32-softmmu/arch_init.o
>> In file included from /home/build/qemu/default_s390/build/qemu-common.h:123,
>>                 from /home/build/qemu/default_s390/build/monitor.h:4,
>>                 from /home/build/qemu/default_s390/build/arch_init.c:32:
>> /home/build/qemu/default_s390/build/target-lm32/cpu.h:185: error:
>> redefinition of typedef 'CPULM32State'
>> /home/build/qemu/default_s390/build/target-lm32/cpu.h:31: error:
>> previous declaration of 'CPULM32State' was here
>> make[1]: *** [arch_init.o] Error 1
>> make: *** [subdir-lm32-softmmu] Error 2
>> 
>> My GCC handles the following code fine:
>> 
>> typedef struct CPULM32State CPULM32State;
>> 
>> typedef struct CPULM32State {
>>    // ...
>> } CPULM32State;
>> 
>> Solutions would be to
>> a) drop the second typedef,
>> b) use struct CPULM32State,
>> c) move the function using it to after the second typedef.
> 
> I vote for a)

Same here.

Alex