----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63422/#review189954 -----------------------------------------------------------
3rdparty/stout/include/stout/posix/os.hpp Lines 175 (patched) <https://reviews.apache.org/r/63422/#comment267197> I'm curious: is there a reason you use `memset()` instead of `setenv()` here? - Greg Mann On Oct. 30, 2017, 6:14 p.m., James Peach wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/63422/ > ----------------------------------------------------------- > > (Updated Oct. 30, 2017, 6:14 p.m.) > > > Review request for mesos and Greg Mann. > > > Bugs: MESOS-8140 > https://issues.apache.org/jira/browse/MESOS-8140 > > > Repository: mesos > > > Description > ------- > > On Linux, updating the environment with unsetenv(3) does not > change the value that other processes can inspect (e.g. through > `/proc/$pid/environ`). Overwrite the existing value (if any) prior > to calling unsetenv(3) so we can know it is really gone. > > > Diffs > ----- > > 3rdparty/stout/include/stout/posix/os.hpp > 8511dfd419a646df17eb687d732bb975f4c23d53 > > > Diff: https://reviews.apache.org/r/63422/diff/1/ > > > Testing > ------- > > make check (Fedora 26) > > > Thanks, > > James Peach > >
