-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/65145/#review195476
-----------------------------------------------------------


Ship it!




Ship It!

- Jie Yu


On Jan. 14, 2018, 2 a.m., Andrew Schwartzmeyer wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/65145/
> -----------------------------------------------------------
> 
> (Updated Jan. 14, 2018, 2 a.m.)
> 
> 
> Review request for mesos, Akash Gupta, Gilbert Song, Jie Yu, and Joseph Wu.
> 
> 
> Bugs: MESOS-8225
>     https://issues.apache.org/jira/browse/MESOS-8225
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Because `os::which` was added to its own header, all uses of it now need
> to include said header.
> 
> In `tests/environment.cpp`, instead of using `os::system("which foo")`,
> we now use `os::which("foo")` to be compatible with Windows.
> 
> 
> Diffs
> -----
> 
>   src/slave/containerizer/mesos/isolators/docker/volume/isolator.cpp 
> ba9e20c16841bfaa2a5c72d449a2da1a637b71df 
>   src/slave/containerizer/mesos/isolators/network/cni/cni.cpp 
> 61de16ba7f1b0ba80663a8544baa98fe36302df0 
>   
> src/slave/containerizer/mesos/isolators/network/cni/plugins/port_mapper/port_mapper.cpp
>  de64d6572c59da22dff528b308b4a4b0e9be9d2f 
>   src/slave/containerizer/mesos/launch.cpp 
> c45a038f191d7ddc536bb1ffc58532df90aff153 
>   src/tests/environment.cpp 72bd621f02f97ea5fd553f3dc0bd52adb8ddee8f 
> 
> 
> Diff: https://reviews.apache.org/r/65145/diff/1/
> 
> 
> Testing
> -------
> 
> This looks bigger than it is because `#include <stout/os/which.hpp>` was 
> added to each file that used `os::which`. Logic change is constrained to a 
> test file.
> 
> 
> Thanks,
> 
> Andrew Schwartzmeyer
> 
>

Reply via email to