]> git.proxmox.com Git - qemu.git/commitdiff
make_device_config: Fix non-fatal error message with dash and other shells
authorStefan Weil <weil@mail.berlios.de>
Thu, 30 Dec 2010 12:04:57 +0000 (12:04 +0000)
committerBlue Swirl <blauwirbel@gmail.com>
Fri, 14 Jan 2011 22:00:07 +0000 (22:00 +0000)
ORS=" " adds a blank to the name of the include file.
Some shells (e.g. dash) don't accept input redirection
(tr -d '\r' < $f) when $f ends with a blank, so they
print an error message instead of reading pci.mak.
This is a non-fatal error because pci.mak does not
contain an include line. It was introduced by commit
5d6b423c5cd6f9dfac30959ff1d5c088996719c3.

Using printf avoids adding a blank and is also supported
by older awk versions (this solution was suggested by
Paolo Bonzini, thank you).

Cc: Blue Swirl <blauwirbel@gmail.com>
Signed-off-by: Stefan Weil <weil@mail.berlios.de>
Tested-by: Andreas Färber <andreas.faerber@web.de>
make_device_config.sh

index 596fc5bf698ce1fa6e5a68dda8158a3924619dc1..5d14885dfc21f1ac2d285dc36d12a903e9a33ce0 100644 (file)
@@ -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 / {printf "'$src_dir'/%s", $2}'`
   [ $? = 0 ] || exit 1
   all_includes="$all_includes $f"
 done