[Toybox] toybox sed broken in all AOSP Marshmallow builds

[enh]

if you search the archives (assuming they're working today) for
"toybox sed broken in Android Marshmallow (6.0) for LP32", you can see
the bionic patch that fixed this.

this has no impact on security, so it won't be backported.

On Wed, Jan 20, 2016 at 10:32 AM, Chris Renshaw <chris.renshaw at live.com> wrote:
> Hi guys, I just wanted to give you the heads up on sed being broken on my
> Nexus 5 and Nexus 7 2013 running the current Marshmallow 6.0.1 (MMB29S and
> MMB29O, respectively).
>
> Here is some example output:
>
> root at flo:/ # echo "blah" | sed "s/blah/test/";
> testblah
> root at flo:/ # echo "blah" | busybox sed "s/blah/test/";
> test
>
>
> Since Google's release tags show (up to the current 6.0.1_r10) Marshmallow's
> toybox frozen from back in July that means sed is broken on all official
> Marshmallow (6.0-6.0.1) releases thus far.
>
> Cyanogen has recently followed AOSP's lead and started including toybox in
> their recovery, and I notice the sed output works correctly there,
> suggesting the issue has already been fixed upstream; I also notice Elliott
> has been keeping up with the official GitHub master on GoogleSource thus
> it's likely fixed in master there too, so I was just hoping that a newer
> toybox could be sync'd and merged into one of the upcoming monthly security
> update branches if either Elliott or Rob have the ability to nudge for such
> a thing, since it's no fun having broken Linux utilities in Android. Thanks!
>
> Chris Renshaw



-- 
Elliott Hughes - http://who/enh - http://jessies.org/~enh/
Android native code/tools questions? Mail me/drop by/add me as a reviewer.