From: Tom Rini <trini@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v4 1/2] tools, config.mk: add binutils-version
Date: Thu, 19 Jul 2012 08:38:39 -0700 [thread overview]
Message-ID: <500829FF.7010109@ti.com> (raw)
In-Reply-To: <201207191121.52936.vapier@gentoo.org>
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 07/19/2012 08:21 AM, Mike Frysinger wrote:
> On Thursday 19 July 2012 11:08:10 Tom Rini wrote:
>> On 07/18/2012 08:11 PM, Mike Frysinger wrote:
>>> On Wednesday 18 July 2012 19:45:52 Allen Martin wrote:
>>>> +MAJOR=$($gas --version | head -1 | awk '{print $NF}' | cut
>>>> -d . -f 1) +MINOR=$($gas --version | head -1 | awk '{print
>>>> $NF}' | cut -d . -f 2) + +printf "%02d%02d\\n" $MAJOR $MINOR
>>>
>>> can be replaced with a single awk script:
>>>
>>> $gas --version | awk '{ gsub(/[.]/, " ", $NF) $0 = $NF printf
>>> "%02d%02d\n", $1, $2 exit }'
>>
>> That looks much longer and we call this once so a few execs is
>> noise.
>
> here's a shorter version: $gas --version | awk '{ gsub(/[.]/, " ",
> $NF); $0 = $NF; printf "%02d%02d\n", $1, $2; exit }'
And still over 80 chars before we assign it to a variable. I could
get it to 77 chars with all whitespace removed.
- --
Tom
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
iQIcBAEBAgAGBQJQCCn/AAoJENk4IS6UOR1WFJwP/2pv0lUXUJVJZk1Wb1w2ln2G
MtjUg1ntwrgQmKCb5D67VNCHN2sjtiWi0HGNZDCvHqRz9AGN7BX/Dz8Jx0hKoqf3
S92J7VpKJ+MGpBVpWxlZMNY5FlBnkiVpUCkIQefhz5sUqq1fA2PjCs82MRadr9WP
KmpfDXZ6OBRl7hKqeHVaYaqfCjxaaXqmPLrXh1VXUjA6oYKv0kZcvW9H2kMPyaTd
+shYx0z/TI6UXHmw0CmcoDvPsy9cIBAtqtuqOeP/YP1sIHq1UQBIehfD4ji4JZAl
wWKymSJwtcIl6hCV+IV3/wGczIdvRfyraa8mN5/MrVWLbgiytW/OOnix7mYlw4ov
ysXz/5pg7dPkaHxczvANpVO7PkzRJZKMQhjyuZmVDSDdEzPks9QHWt3miRErXjMc
kkD2LaKkV9hBBSqD6+/vfd45zLI125UDfkkRLLwr67bHcrGkdBbojOXE357vQx7N
ELp2FPOwfXkBOw//P0KhlVtc9T1Li+LqqfjqzbYQkPYbHzcAal7SPERzxION+pFO
00M+uYwtxPDm/wvmk033VTZNaLdvLHU6Zg1wkKwj484MZ/x+ptFwDvTwppRw/Kme
4JuhjxWGwQFu9tkV0gWFd2D2uYZzbDt5VnZkntp1BxyXEHJ3FvvUvMryr8ZqEAIS
Ss/IDqUlIsctLDiDziSD
=9xj8
-----END PGP SIGNATURE-----
next prev parent reply other threads:[~2012-07-19 15:38 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-07-18 23:45 [U-Boot] [PATCH v4 1/2] tools, config.mk: add binutils-version Allen Martin
2012-07-18 23:45 ` [U-Boot] [PATCH v4 2/2] arm: work around assembler bug Allen Martin
2012-07-19 6:06 ` Albert ARIBAUD
2012-07-19 15:08 ` Tom Rini
2012-08-01 17:34 ` Allen Martin
2012-10-04 8:47 ` Albert ARIBAUD
2012-07-19 3:11 ` [U-Boot] [PATCH v4 1/2] tools, config.mk: add binutils-version Mike Frysinger
2012-07-19 15:08 ` Tom Rini
2012-07-19 15:21 ` Mike Frysinger
2012-07-19 15:38 ` Tom Rini [this message]
2012-07-19 16:43 ` Mike Frysinger
2012-07-19 16:54 ` Tom Rini
2012-08-01 22:31 ` Mike Frysinger
2012-08-01 22:46 ` Allen Martin
2012-08-01 22:55 ` Graeme Russ
2012-08-01 23:01 ` Mike Frysinger
2012-08-02 17:12 ` Allen Martin
2012-08-02 19:30 ` Wolfgang Denk
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=500829FF.7010109@ti.com \
--to=trini@ti.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox