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

[Alexander Graf]

On 04.07.2011, at 07:51, Stefan Weil wrote:

> Am 04.07.2011 06:23, schrieb Stefan Hajnoczi:
>> On Mon, Jul 4, 2011 at 12:47 AM, Alexander Graf <address@hidden> wrote:
>>> 
>>> On 04.07.2011, at 02:04, address@hidden wrote:
>>> 
>>>> The Buildbot has detected a new failure on builder 
>>>> disable_kvm_x86_64_debian_5_0 while building qemu.
>>>> Full details are available at:
>>>> http://buildbot.b1-systems.de/qemu/builders/disable_kvm_x86_64_debian_5_0/builds/148
>>>> 
>>>> Buildbot URL: http://buildbot.b1-systems.de/qemu/
>>>> 
>>>> Buildslave for this Build: b1_qemu_1
>>>> 
>>>> Build Reason: The Nightly scheduler named 'nightly_disable_kvm' triggered 
>>>> this build
>>>> Build Source Stamp: [branch master] HEAD
>>>> Blamelist:
>>>> 
>>>> BUILD FAILED: failed compile
>>> 
>>> In file included from /usr/include/png.h:438,
>>>                from ui/vnc-enc-tight.c:40:
>>> /usr/include/pngconf.h:326: error: expected '=', ',', ';', 'asm' or 
>>> '__attribute__' before '.' token
>>> /usr/include/pngconf.h:327: error: expected '=', ',', ';', 'asm' or 
>>> '__attribute__' before 'include'
>>> make: *** [ui/vnc-enc-tight.o] Error 1
>>> 
>>> program finished with exit code 2
>>> 
>>> 
>>> Not sure what exactly is missing, but the last change in that code was from 
>>> Stefan Weil (2fb0c09f4ff036f68474277ed4edc036f6529de8).
>> 
>> Daniel,
>> Would it be possible to post the contents of /usr/include/pngconf.h
>> from b1_qemu_1? I checked my local copy and I don't understand these
>> compiler errors. Perhaps you have a different version of the file.
>> 
>> Thanks,
>> Stefan
> 
> The compiler errors come again from the setjmp check in pngconf.h:
> 
>           __pngconf.h__ in libpng already includes setjmp.h;
>           __dont__ include it again.;
> 
> The buildbot runs Debian Lenny which includes an old version of libpng.
> That version does not use PNG_SKIP_SETJMP_CHECK to skip the setjmp check.
> Defining PNG_SETJMP_NOT_SUPPORTED might help with this
> version, but I still have to test that.
> 
> Updating the buildbot to Debian Squeeze would also work.

So it's a real bug and a good thing the buildbot is running on Lenny. Maybe we 
should add the define and #include setjmp.h to configure, so at least that one 
fails when compilation wouldn't work either?


Alex