https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=233545

Jilles Tjoelker <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]
           Assignee|[email protected]            |[email protected]
             Status|New                         |Open

--- Comment #2 from Jilles Tjoelker <[email protected]> ---
Good catch. It somewhat surprises me that this has been broken for so long.

The environment() part of the patch looks good, but the exportcmd() part is
redundant since setvar() already sets VUNSET if the value to be set is NULL
(something like  sh -uc 'export thisshouldnotbehere; echo
"$thisshouldnotbehere"'  detects the unset variable as it should). Also, a test
should be added, preferably using env(1) or ${SH} itself only and not
printenv(1) since it's not in POSIX.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.
_______________________________________________
[email protected] mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "[email protected]"

Reply via email to