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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.