<div dir="auto">Thanks, it's there now. I'll see how that goes...</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr"><strong>From:</strong>Rob Landley <span dir="ltr"><<a href="mailto:rob@landley.net">rob@landley.net</a>></span><br><strong>Date:</strong>Fri, May 3, 2019, 13:29<br><strong>To:</strong>enh, toybox<br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Forgot to push, sorry.<br>
<br>
Rob<br>
<br>
On 5/3/19 2:47 PM, enh via Toybox wrote:<br>
> ping?<br>
> <br>
> On Thu, May 2, 2019 at 6:47 PM enh <<a href="mailto:enh@google.com" target="_blank" rel="noreferrer">enh@google.com</a>> wrote:<br>
>><br>
>> Found when trying to update the toybox prebuilt used for the Android<br>
>> build.<br>
>><br>
>> Also add the corresponding test.<br>
>> ---<br>
>>  lib/env.c      | 4 +---<br>
>>  tests/env.test | 2 ++<br>
>>  2 files changed, 3 insertions(+), 3 deletions(-)<br>
>><br>
>> diff --git a/lib/env.c b/lib/env.c<br>
>> index 35ef688c..bc23b753 100644<br>
>> --- a/lib/env.c<br>
>> +++ b/lib/env.c<br>
>> @@ -58,11 +58,9 @@ void xsetenv(char *name, char *val)<br>
>>      if (!memcmp(name, environ[i], len) && environ[i][len]=='=') {<br>
>>        if (i>=envc) free(environ[i]);<br>
>>        else {<br>
>> -        char **delete = environ+i;<br>
>> -<br>
>>          // move old entries down, add at end of old data<br>
>>          toys.envc = envc--;<br>
>> -        for (i=0; new ? i<envc : !!delete[i]; i++) delete[i] = delete[i+1];<br>
>> +        for (; new ? i<envc : !!environ[i]; i++) environ[i] = environ[i+1];<br>
>>          i = envc;<br>
>>        }<br>
>>        break;<br>
>> diff --git a/tests/env.test b/tests/env.test<br>
>> index 3098731d..4df118dd 100755<br>
>> --- a/tests/env.test<br>
>> +++ b/tests/env.test<br>
>> @@ -19,3 +19,5 @@ unset WALRUS BANANA LETTERS FILTER<br>
>><br>
>>  testcmd "early fail" '--oops 2> /dev/null ; echo $?' "125\n" "" ""<br>
>>  testcmd "why is this allowed" "=BLAH env | grep '^=BLAH\$'" "=BLAH\n" "" ""<br>
>> +<br>
>> +testcmd "replace" "A=foo PATH= `which printenv` A" "foo\n" "" ""<br>
>> --<br>
>> 2.21.0.1020.gf2820cf01a-goog<br>
> _______________________________________________<br>
> Toybox mailing list<br>
> <a href="mailto:Toybox@lists.landley.net" target="_blank" rel="noreferrer">Toybox@lists.landley.net</a><br>
> <a href="http://lists.landley.net/listinfo.cgi/toybox-landley.net" rel="noreferrer noreferrer" target="_blank">http://lists.landley.net/listinfo.cgi/toybox-landley.net</a><br>
> .<br>
> <br>
</blockquote></div>