Re: [Gnewsense-dev] Re: [gNewSense-users] possible error in gen-kernel builder script

[Karl Goetz]

On Tue, 18 Aug 2009 23:27:18 +1000
Tony <address@hidden> wrote:

> That proposed regex is syntactically incorrect, I believe.
> 
> It does not close the \( with a corresponding \)
> 

Good catch.
Should have been:
sed -i 's/^\([^ ?]+\)$/\1 ?/' $i

As for leaving it broken - yeah. thats an option too. I'm not going to
block the release on it, for example ;)
kk

> Cheers,
> 
> gurrier


> >>> The gen-kernel builder script includes the following lines:
> >>>
> >>>     # Changes so it'll compile without some modules
> >>>     for i in debian/d-i/modules/*; do
> >>>         sed -i 's/^\([[^ ?][^ ?]*\)$/\1 ?/' $i
> >>>     done

> >>>         sed -i 's/^\([^ ?][^ ?]*\)$/\1 ?/' $i
> >>>

> > his regex is `sed -i 's/^\([^ ?]+$/\1 ?/' $i`
> >
> > (Afaik its the same as that prop


-- 
Karl Goetz, (Kamping_Kaiser / VK5FOSS)
Debian contributor / gNewSense Maintainer
http://www.kgoetz.id.au
No, I won't join your social networking group

signature.asc
Description: PGP signature