Re: [Qemu-devel] [PATCH v4 12/23] acpi: add rules to compile ASL source

[Laszlo Ersek]

On 09/23/13 15:39, Michael S. Tsirkin wrote:
> On Mon, Sep 23, 2013 at 02:36:41PM +0200, Paolo Bonzini wrote:
>> Il 22/09/2013 15:37, Michael S. Tsirkin ha scritto:
>>> Detect presence of IASL compiler and use it
>>> to process ASL source. If not there, use pre-compiled
>>> files in-tree. Add script to update the in-tree files.
>>>
>>> Note: distros are known to silently update iasl
>>> so detect correct iasl flags for the installed version on each run as
>>> opposed to at configure time.
>>
>> This is not any different from a C compiler, which is likely updated
>> much much more often than iasl.
> 
> Yes but it's not a theoretical issue:
> we did catch iasl changing flags semantics on the fly, once.
> 
> I think compilers don't do this as a norm :)

(Fully tangentially... gcc and clang emit warnings for a new set of code
constructs every other week. When combined with -Werror, it's super
annoying; valid and intentional code can stop to build unexpectedly.
Good thing we have --disable-werror for this.)

Laszlo