On Thu, Jun 07, 2012 at 04:37:17PM +0200, Denys Vlasenko wrote:
> > The fix broke it worse:
> >
> > $ sed 's/\(["`$\\]\)/\\\1/g' << "EOF"
> > {\"\1\"
> > EOF
> > {\\\"\\1\\\"
> > $ busybox sed 's/\(["`$\\]\)/\\\1/g' << "EOF"
> > {\"\1\"
> > EOF
> > {\\1\\
> >
> > Previous version of busybox sed handled this case fine.
> 
> Indeed, null match code falsely triggers here.
> Fixed in git, please try...

Thanks, seems much better now!
I think this finally makes busybox sed a viable replacement for GNU
sed for general-purpose use (including for a system that can rebuild
itself).

Rich
_______________________________________________
busybox mailing list
[email protected]
http://lists.busybox.net/mailman/listinfo/busybox

Reply via email to