kbuild: fix bzImage build for x86
authorMichael Tokarev <mjt@tls.msk.ru>
Tue, 13 Oct 2009 20:22:46 +0000 (22:22 +0200)
committerMichal Marek <mmarek@suse.cz>
Sat, 12 Dec 2009 12:08:13 +0000 (13:08 +0100)
As has been discussed previously (and Sam has been CC'ed), the fix
is still incorrect.  It replaces "echo -ne" with "/bin/echo -ne",
but neither of the two are guaranteed to support the necessary
arguments and necessary (hexadecimal) escape sequences.  What should
be used here is printf(1).  The trivial patch below (on top of these
kbuild changes) fixes this issue.

Signed-Off-By: Michael Tokarev <mjt@tls.msk.ru>
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Signed-off-by: Michal Marek <mmarek@suse.cz>
scripts/Makefile.lib

index ffdafb26f5390b414d74ec07a1dde3078fd9a540..d9f0cb837400985be39589eb8e0b572b7e5fbd54 100644 (file)
@@ -208,7 +208,7 @@ cmd_gzip = (cat $(filter-out FORCE,$^) | gzip -f -9 > $@) || \
 
 # Bzip2 and LZMA do not include size in file... so we have to fake that;
 # append the size as a 32-bit littleendian number as gzip does.
-size_append = /bin/echo -ne $(shell                                    \
+size_append = printf $(shell                                           \
 dec_size=0;                                                            \
 for F in $1; do                                                                \
        fsize=$$(stat -c "%s" $$F);                                     \