[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4A0493B0.3090204@linux.intel.com>
Date: Fri, 08 May 2009 13:18:56 -0700
From: "H. Peter Anvin" <hpa@...ux.intel.com>
To: Sam Ravnborg <sam@...nborg.org>
CC: linux-kernel@...r.kernel.org, vgoyal@...hat.com, hbabu@...ibm.com,
kexec@...ts.infradead.org, ying.huang@...el.com, mingo@...e.hu,
tglx@...utronix.de, ebiederm@...ssion.com,
"H. Peter Anvin" <hpa@...or.com>
Subject: Re: [PATCH 05/14] kbuild: allow compressors (gzip, bzip2, lzma) to
take multiple inputs
Sam Ravnborg wrote:
>> diff --git a/scripts/bin_size b/scripts/bin_size
>> index 43e1b36..55f2161 100644
>> --- a/scripts/bin_size
>> +++ b/scripts/bin_size
>> @@ -1,10 +1,14 @@
>> #!/bin/sh
>>
>> if [ $# = 0 ] ; then
>> - echo Usage: $0 file
>> + echo Usage: $0 file...
>> fi
>>
>> -size_dec=`stat -c "%s" $1`
>> +size_dec=0
>> +for file; do
>> + fsize=`stat -c "%s" $file`
>> + size_dec=`expr $size_dec + $fsize`
>> +done
>> size_hex_echo_string=`printf "%08x" $size_dec |
>> sed 's/\(..\)\(..\)\(..\)\(..\)/\\\\x\4\\\\x\3\\\\x\2\\\\x\1/g'`
>> /bin/echo -ne $size_hex_echo_string
>
> But I would rather have had this inside makefile.lib...
>
It's messy enough as a shell script... it seems like baking it into
Makefile syntax would make it even worse.
-hpa
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists