code review 149300046: syscall: fix Setenv for plan 9

syscall: fix Setenv for plan 9

envi needs to be updated during Setenv so the key can be correctly deleted later with Unsetenv.

Update issue 8849.

[mischief, 2014-10-02 07:07:35]

Hello bradfitz@golang.org, 0intro@gmail.com (cc:
golang-codereviews@googlegroups.com),

I'd like you to review this change to
https://code.google.com/p/go/

[0intro, 2014-10-02 07:12:09]

LGTM
Waiting for Brad.

I think s/Fixes/Update/ would be more accurate in the description.

[0intro, 2014-10-02 08:25:37]

I believe Brad went to sleep. Submitting it anyway to fix the Plan 9 build.

[0intro, 2014-10-02 08:26:34]

*** Submitted as https://code.google.com/p/go/source/detail?r=7af2b6fa04ee ***

syscall: fix Setenv for plan 9

envi needs to be updated during Setenv so the key can be correctly deleted later
with Unsetenv.

Update issue 8849.

LGTM=0intro
R=bradfitz, 0intro
CC=golang-codereviews
https://codereview.appspot.com/149300046

Committer: David du Colombier <0intro@gmail.com>

[bradfitz, 2014-10-02 16:12:00]

LGTM as short-term fix.
 On Oct 2, 2014 1:26 AM, <0intro@gmail.com> wrote:

> *** Submitted as
> https://code.google.com/p/go/source/detail?r=7af2b6fa04ee ***
>
> syscall: fix Setenv for plan 9
>
> envi needs to be updated during Setenv so the key can be correctly
> deleted later with Unsetenv.
>
> Update issue 8849.
>
> LGTM=0intro
> R=bradfitz, 0intro
> CC=golang-codereviews
> https://codereview.appspot.com/149300046
>
> Committer: David du Colombier <0intro@gmail.com>
>
>
> https://codereview.appspot.com/149300046/
>