> On April 20, 2015, 6:27 a.m., Adam B wrote: > > src/cli/execute.cpp, line 77 > > <https://reviews.apache.org/r/33109/diff/2/?file=924729#file924729line77> > > > > Any reason not to just name this 'env' or 'environment'?
I afraid "environment/env" maybe have other usage in the future. But if you think "environment" is ok, I would change it here. - haosdent ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33109/#review80660 ----------------------------------------------------------- On April 13, 2015, 4:42 p.m., haosdent huang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/33109/ > ----------------------------------------------------------- > > (Updated April 13, 2015, 4:42 p.m.) > > > Review request for mesos and Adam B. > > > Bugs: MESOS-2023 > https://issues.apache.org/jira/browse/MESOS-2023 > > > Repository: mesos > > > Description > ------- > > Allow setting environment variables in mesos-execute > > > Diffs > ----- > > src/cli/execute.cpp 84f70dccbc2c5dd43f68105d967f4488c82f582b > > Diff: https://reviews.apache.org/r/33109/diff/ > > > Testing > ------- > > make check > > > Thanks, > > haosdent huang > >
