[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL v2 00/52] Misc patches for QEMU 5.1 soft freeze
From: |
Claudio Fontana |
Subject: |
Re: [PULL v2 00/52] Misc patches for QEMU 5.1 soft freeze |
Date: |
Fri, 10 Jul 2020 16:44:16 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.4.1 |
On 7/10/20 3:11 PM, Philippe Mathieu-Daudé wrote:
> On 7/10/20 2:55 PM, Peter Maydell wrote:
>> On Fri, 10 Jul 2020 at 13:52, Claudio Fontana <cfontana@suse.de> wrote:
>>> If you can access it,
>>>
>>> https://cirrus-ci.com/task/5537514263937024?command=main#L2039
>>>
>>> the thing is treated here as a warning, which is in the middle of a large
>>> amount of other warnings.
>>>
>>> private/var/folders/3y/l0z1x3693dl_8n0qybp4dqwh0000gn/T/cirrus-ci-build/ui/vnc-auth-sasl.c:648:29:
>>> warning: 'sasl_errdetail' is deprecated: first deprecated in macOS 10.11
>>> [-Wdeprecated-declarations]
>>> sasl_errdetail(vs->sasl.conn));
>>>
>>> /* ... lots of similar warnings */
>>>
>>> private/var/folders/3y/l0z1x3693dl_8n0qybp4dqwh0000gn/T/cirrus-ci-build/ui/cocoa.m:1478:9:
>>> warning: implicit declaration of function 'cpu_throttle_set' is invalid in
>>> C99 [-Wimplicit-function-declaration]
>>> cpu_throttle_set(throttle_pct);
>>> ^
>>>
>>>
>>> Apparently the cirrus-ci I am using is not treating this as an error, while
>>> what you are using is..
>>
>> Ah, I see. Yeah, configure by default doesn't enable -Werror for OSX.
>> My build tree has
>> '--extra-cflags=-fdiagnostics-color=never -Werror
>> -Wno-error=deprecated-declarations'
>
> TIL this is different on OSX...
>
>>
>> (of which the last two are relevant here). We should probably
>> make configure use -Werror on OSX now with the same logic as Linux,
>> I've been using it that way for ages without issues.
>
> Yes please!
>
Speaking of MacOS and CI, commit 57ee95ed4ee7b4c039ec5f0705c45734c56706bc
Author: Max Reitz <mreitz@redhat.com>
Date: Thu Jun 25 14:55:30 2020 +0200
iotests: Make _filter_img_create more active
broke cirrus-ci completely for me due to missing "readarray" builtin.
Maybe it is bash vs zsh?
Ciao,
Claudio
- [PULL v2 52/52] apic: Report current_count via 'info lapic', (continued)
- [PULL v2 52/52] apic: Report current_count via 'info lapic', Paolo Bonzini, 2020/07/08
- [PULL v2 44/52] accel/kvm: Simplify kvm_check_extension(), Paolo Bonzini, 2020/07/08
- Re: [PULL v2 00/52] Misc patches for QEMU 5.1 soft freeze, Peter Maydell, 2020/07/10
- Re: [PULL v2 00/52] Misc patches for QEMU 5.1 soft freeze, Peter Maydell, 2020/07/10
- Re: [PULL v2 00/52] Misc patches for QEMU 5.1 soft freeze, Claudio Fontana, 2020/07/10
- Re: [PULL v2 00/52] Misc patches for QEMU 5.1 soft freeze, Peter Maydell, 2020/07/10
- Re: [PULL v2 00/52] Misc patches for QEMU 5.1 soft freeze, Claudio Fontana, 2020/07/10
- Re: [PULL v2 00/52] Misc patches for QEMU 5.1 soft freeze, Peter Maydell, 2020/07/10
- Re: [PULL v2 00/52] Misc patches for QEMU 5.1 soft freeze, Philippe Mathieu-Daudé, 2020/07/10
- Re: [PULL v2 00/52] Misc patches for QEMU 5.1 soft freeze,
Claudio Fontana <=
- Re: [PULL v2 00/52] Misc patches for QEMU 5.1 soft freeze, Max Reitz, 2020/07/10
- Re: [PULL v2 00/52] Misc patches for QEMU 5.1 soft freeze, Peter Maydell, 2020/07/10
- Re: [PULL v2 00/52] Misc patches for QEMU 5.1 soft freeze, Max Reitz, 2020/07/10
- Re: [PULL v2 00/52] Misc patches for QEMU 5.1 soft freeze, Peter Maydell, 2020/07/10
- Re: [PULL v2 00/52] Misc patches for QEMU 5.1 soft freeze, Max Reitz, 2020/07/10
- Re: [PULL v2 00/52] Misc patches for QEMU 5.1 soft freeze, Peter Maydell, 2020/07/10
- Re: [PULL v2 00/52] Misc patches for QEMU 5.1 soft freeze, Daniel P . Berrangé, 2020/07/10
- Re: [PULL v2 00/52] Misc patches for QEMU 5.1 soft freeze, Thomas Huth, 2020/07/10