[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] Fix build for less common build directories nam
From: |
Michael Tokarev |
Subject: |
Re: [Qemu-devel] [PATCH] Fix build for less common build directories names |
Date: |
Fri, 28 Oct 2016 17:57:50 +0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Icedove/45.4.0 |
17.10.2016 07:43, Stefan Weil wrote:
> Am 16.10.2016 um 21:04 schrieb Peter Maydell:
>> On 16 October 2016 at 15:31, Michael Tokarev <address@hidden> wrote:
>>> 13.10.2016 21:29, Stefan Weil wrote:
>>>> scripts/tracetool generates a C preprocessor macro from the name of the
>>>> build directory. Any characters which are possible in a directory name
>>>> but not allowed in a macro name must be substituted, otherwise builds
>>>> will fail.
>>>
>>> Applied to -trivial, thank you!
>>
>> Consensus on this and the other thread seemed to be
>> that we should just fix the bug where tracetool is
>> putting the build directory name into symbols
>> rather than working around that...
>>
>> thanks
>> -- PMM
>
> IMHO the patch can be applied nevertheless, as it uses a clearer
> regular expression for characters allowed in macro names
> (not one that is specific for the current QEMU code).
So, what's the thing here?
Once again I hit this bug today, when my usual build environment doesn't
work:
./trace/generated-tracers.h:3:15: error: extra tokens at end of
#ifndef directive [-Werror]
#ifndef TRACE_.B64_GENERATED_TRACERS_H
^
and this patch fixes the issue for me.
Thanks,
/mjt