Re: [Qemu-devel] [A cry for sanity]: make_device_config.sh: Fix target path in generated dependency file

[Anthony Liguori]

Andreas Färber <address@hidden> writes:

> Am 24.01.2013 11:18, schrieb Michael Tokarev:
>> Guys, it isn't sane anymore.  It if FAR from being funny.
>> 
>> Ping^7.. if i counted correctly...
>
> Anthony asked me to resend this one but I haven't found it among all my
> branches and repositories yet o:-)

I don't even have a copy of this patch in my mail history so it really
needs to be resubmitted.  I'd be amazed after all of this time if it
didn't need reworking.

Regards,

Anthony Liguori

> and had higher priorities. Further, I
> have not encountered it recently - it was about additions to pci.mak
> *sometimes* breaking the build due to bad ordering (back then: megasas).
>
> However, I'm not aware of recent build system changes that would
> obsolete this patch - Paolo is still CC.
>
> Andreas
>
>> 
>> 03.12.2012 23:31, mdroth wrote:
>>> On Sat, Oct 27, 2012 at 12:32:28PM +0400, Michael Tokarev wrote:
>>>> Ping?
>>>
>>> Ping.
>>>
>>> Fix still applicable for 1.3, also looking to pull it in for 1.2.2.
>>>
>>>>
>>>> /mjt
>>>>
>>>> On 18.09.2012 18:32, Andreas Färber wrote:
>>>>> Am 18.09.2012 14:29, schrieb Michael Tokarev:
>>>>>> Has it been applied to anything?  I don't think so.
>>>>>> Is it still needed?
>>>>>
>>>>> Not in qemu.git yet, still applicable AFAICT. CC'ing Paolo.
>>>>>
>>>>> /-F
>>>>>
>>>>>>
>>>>>> Thanks,
>>>>>>
>>>>>> /mjt
>>>>>>
>>>>>> On 07.06.2012 20:23, Andreas Färber wrote:
>>>>>>> config-devices.mak.d is included from Makefile.target, i.e. from
>>>>>>> inside
>>>>>>> the *-softmmu/ directory. It included the directory path, so never
>>>>>>> applied to the actual config-devices.mak. Symptoms were spurious
>>>>>>> dependency issues with default-configs/pci.mak.
>>>>>>>
>>>>>>> Fix by using `basename` to strip the directory path.
>>>>>>>
>>>>>>> Reported-by: Gerhard Wiesinger <address@hidden>
>>>>>>> Signed-off-by: Andreas Färber <address@hidden>
>>>>>>> ---
>>>>>>>   Seems I forgot to send this out before 1.1...
>>>>>>>
>>>>>>>   scripts/make_device_config.sh |    2 +-
>>>>>>>   1 files changed, 1 insertions(+), 1 deletions(-)
>>>>>>>
>>>>>>> diff --git a/scripts/make_device_config.sh
>>>>>>> b/scripts/make_device_config.sh
>>>>>>> index 5d14885..0778fe2 100644
>>>>>>> --- a/scripts/make_device_config.sh
>>>>>>> +++ b/scripts/make_device_config.sh
>>>>>>> @@ -25,4 +25,4 @@ done
>>>>>>>   process_includes $src > $dest
>>>>>>>
>>>>>>>   cat $src $all_includes | grep -v '^include' > $dest
>>>>>>> -echo "$1: $all_includes" > $dep
>>>>>>> +echo "`basename $1`: $all_includes" > $dep
>>>>>>
>>>>>
>>>>>
>>>>
>>>>
>>>
>> 
>
>
> -- 
> SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
> GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg