On 06/06/2012 02:24 AM, Rich Felker wrote:
On Mon, Jun 04, 2012 at 02:56:15PM +0200, Denys Vlasenko wrote:
On Sun, Jun 3, 2012 at 3:54 PM, Rich Felker<[email protected]>  wrote:
Ping. Anybody looked at this? I filed it as bug # 5246.
fixed in git:

commit 21f6fbf545e7fa58f0eaa444001a9d25bc37c4eb
Author: Denys Vlasenko<[email protected]>
Date:   Mon Jun 4 14:44:47 2012 +0200

     sed: fix zero chars match/replace
The fix broke it worse:

$ sed 's/\(["`$\\]\)/\\\1/g'<<  "EOF"
{\"\1\"
EOF
{\\\"\\1\\\"
$ busybox sed 's/\(["`$\\]\)/\\\1/g'<<  "EOF"
{\"\1\"
EOF
{\\1\\
i installed bb 1.20.1 and applied this single commit, then tried to build stuff here's a log of the errors i encountered (different output with gnu sed and bb sed)

http://sprunge.us/OMgf

there's at least one more broken case than the above quoting error.

Previous version of busybox sed handled this case fine.

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


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

Reply via email to