Re: [PATCH 1/3]: Replace kernel/timeconst.pl with kernel/timeconst.sh

Valdis.Kletnieks
2009-01-04T05:08:21Z

[-- Attachment #1: Type: text/plain, Size: 769 bytes --]

On Sat, 03 Jan 2009 19:36:04 CST, Rob Landley said:
> On Saturday 03 January 2009 06:28:22 Ingo Oeser wrote:
> > > +for i in "MSEC 1000" "USEC 1000000"
> > > +do
> > > + NAME=$(echo $i | awk '{print $1}')
> >
> > cut -d' ' -f1 does the same
> >
> > > + PERIOD=$(echo $i | awk '{print $2}')
> >
> > cut -d' ' -f2 does the same

Close, but no cee-gar. cut does something counter-intuitive with multiple
blanks:

% echo 'a b' | awk '{print $2}'
b
% echo 'a b' | cut -d' ' -f2

% echo 'a b' | sed -r 's/[ ]+/ /g' | cut -d' ' -f2
b

Unfortunately, 'sed -r' isn't in the opengroup.org list of required options,
and sed 's/ / /g' doesn't DTRT for 3 or more blanks (as it won't recursively
apply the change to a *new* double blank formed by the previous change).

[-- Attachment #2: Type: application/pgp-signature, Size: 226 bytes --]