qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PULL 00/18] Migration PULL request


From: Peter Maydell
Subject: Re: [Qemu-devel] [PULL 00/18] Migration PULL request
Date: Fri, 22 Sep 2017 09:44:16 +0100

On 22 September 2017 at 09:25, Juan Quintela <address@hidden> wrote:
> Peter Xu <address@hidden> wrote:
>> On Thu, Sep 21, 2017 at 04:29:04PM -0700, address@hidden wrote:
>>
>> [...]
>>
>>> /var/tmp/patchew-tester-tmp-b0y2ycie/src/util/bitmap.c: In function 
>>> ‘bitmap_to_from_le’:
>>> /var/tmp/patchew-tester-tmp-b0y2ycie/src/util/bitmap.c:383:6: error: 
>>> "__WORD_SIZE" is not defined [-Werror=undef]
>>>  # if __WORD_SIZE == 64
>>>       ^~~~~~~~~~~
>>>   CC      util/acl.o
>>> cc1: all warnings being treated as errors
>>> /var/tmp/patchew-tester-tmp-b0y2ycie/src/rules.mak:66: recipe for target 
>>> 'util/bitmap.o' failed
>>> make: *** [util/bitmap.o] Error 1
>>> make: *** Waiting for unfinished jobs....
>>> === OUTPUT END ===
>>>
>>> Test command exited with code: 2
>>
>> This is the same error with previous one.  We need to replace the
>> single usage of __WORD_SIZE into either __WORDSIZE or HOST_LONG_BITS
>> (as suggested by Dave).
>>
>> Again, please let me know if you want me to repost, and sorry for the
>> trouble!
>
> I think we can let that in in the PULL request and fix it later if
> needed, not?

No, a compile failure will always make me reject a merge. Otherwise
I can't easily test anybody else's subsequent merge, and also
patchew will send complaining emails to every patch that's posted
to the list until the breakage in master is fixed...

thanks
-- PMM



reply via email to

[Prev in Thread] Current Thread [Next in Thread]