[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/8] fix 'no such file' error from make_device_c
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH 1/8] fix 'no such file' error from make_device_config.sh |
Date: |
Fri, 14 Jan 2011 11:34:10 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.1 (gnu/linux) |
David Ahern <address@hidden> writes:
> On 01/13/11 06:43, Markus Armbruster wrote:
>> David Ahern <address@hidden> writes:
>>
>>> make_device_config currently emits an error:
>>>
>>> make config-all-devices.mak
>>> GEN x86_64-softmmu/config-devices.mak
>>> /home/dsa/kvm/releases/qemu.git/make_device_config.sh: line 21:
>>> /home/dsa/kvm/releases/qemu.git/default-configs/pci.mak : No such file or
>>> directory
>>>
>>> /bin/sh does not like the file redirection for tr so change it to cat.
>>>
>>> Signed-off-by: David Ahern <address@hidden>
>>> ---
>>> make_device_config.sh | 2 +-
>>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>>
>>> diff --git a/make_device_config.sh b/make_device_config.sh
>>> index 596fc5b..1ce4088 100644
>>> --- a/make_device_config.sh
>>> +++ b/make_device_config.sh
>>> @@ -18,7 +18,7 @@ process_includes () {
>>>
>>> f=$src
>>> while [ -n "$f" ] ; do
>>> - f=`tr -d '\r' < $f | awk '/^include / {ORS=" "; print "'$src_dir'/" $2}'`
>>> + f=`cat $f | tr -d '\r' | awk '/^include / {ORS=" "; print "'$src_dir'/"
>>> $2}'`
>>> [ $? = 0 ] || exit 1
>>> all_includes="$all_includes $f"
>>> done
>>
>> This sweeps the real error under the carpet: $f has a trailing space.
>>
>> What about:
>>
>> diff --git a/make_device_config.sh b/make_device_config.sh
>> index 596fc5b..711829c 100644
>> --- a/make_device_config.sh
>> +++ b/make_device_config.sh
>> @@ -18,7 +18,7 @@ process_includes () {
>>
>> f=$src
>> while [ -n "$f" ] ; do
>> - f=`tr -d '\r' < $f | awk '/^include / {ORS=" "; print "'$src_dir'/" $2}'`
>> + f=`tr -d '\r' <"$f" | awk '/^include / {print "'$src_dir'/" $2}'`
>> [ $? = 0 ] || exit 1
>> all_includes="$all_includes $f"
>> done
>
> That works too.
Care to respin your patch?
> Any idea why the '%/config-devices.mak' rule does not work for a build
> directory != src directory?
>
> e.g., src=/my/qemu.git, build=/tmp/build-qemu
>
> x86_64-softmmu/config-devices.mak is not created, so
> config-all-devices.mak is empty and the build blows up.
Should be debuggable :)
"mkdir bld && cd bld && ./configure --disable-werror && make" works for
me. config-all-devices.mak has some 80 entries.
- [Qemu-devel] Re: [PATCH 6/8] x86_64 config: fix compile for CONFIG_VGA_ISA=n, (continued)
[Qemu-devel] [PATCH 8/8] x86_64 config: fix compile for CONFIG_FDC=n, David Ahern, 2011/01/13
[Qemu-devel] [PATCH 7/8] x86_64 config: fix compile for CONFIG_IDE_ISA=n, David Ahern, 2011/01/13
[Qemu-devel] [PATCH 4/8] remove unused parallel_* variables from pc.c, David Ahern, 2011/01/13
[Qemu-devel] [PATCH 1/8] fix 'no such file' error from make_device_config.sh, David Ahern, 2011/01/13
Re: [Qemu-devel] [PATCH 1/8] fix 'no such file' error from make_device_config.sh, Stefan Weil, 2011/01/14
Re: [Qemu-devel] [PATCH 0/8] misc cleanups for x86_64 disabling config settings, Isaku Yamahata, 2011/01/13