reassign 301455 initrd-tools
severity 303281 important
merge 301455 303281
thanks

At Wed, 6 Apr 2005 00:13:54 +0300,
George Cristian Birzan wrote:
> On Tue, Apr 05, 2005 at 10:44:12PM +0200, Harald Dunkel wrote:
> > > sed -n 's/.*\(=>\)\?[[:blank:]]\+\(\/[^[:blank:]]*\).*/\2/p'
> > >
> > 
> > Something like '[:blank:]' is too much gawk. A few weeks ago
> > I had sent the attached patch.
> 
> > --- /u/harri/initrd-tools/initrd-tools-0.1.77/mkinitrd      2005-01-23 
> > 19:37:41.000000000 +0100
> > +++ mkinitrd        2005-03-10 16:57:41.933283835 +0100
> > @@ -1181,7 +1181,16 @@
> >                     add_command $i
> >             done
> >             exec 3>&-
> > -           < tmp2 sed 's/.*=>[[:blank:]]*\([^[:blank:]]*\).*/\1/'
> > +           < tmp2 awk '
> > +                   /[      ]*.*=>[         ]*\/[^  ]*.*/ {
> > +                           print $3;
> > +                           next;
> > +                   };
> > +                   /[      ]*\/[^  ]*.*/ {
> > +                           print $1;
> > +                           next;
> > +                   };
> > +                   '
> > 
> >             echo /dev/console
> >             echo /dev/null
> 
> I don't really care either way, but I think [[:blank:]] works with all
> locales. Plus, it's easier to read. :-)

As I wrote in #301455, I plan to put the change to initrd-tools:

        sed 's/\(.*=>\)\?[[:blank:]]*\(.*\)[[:blank:]]\+\((.*)\)/\2/;/^$/d' 

It's just straightforward and small change.

I merged these two reports.

This change also fixes #279382, #292080, #295412, #295422, #297724,
and actual fix covered over the previous workaround e2fsprogs change.

Regards,
-- gotom


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to