Re: Review Request 47287: Fixed compilation error caused by prctl and recent subprocess change.

2016-05-11 Thread Alex Clemmer

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


Ship it!




Ship It!

- Alex Clemmer


On May 12, 2016, 5:12 a.m., Kapil Arya wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/47287/
> ---
> 
> (Updated May 12, 2016, 5:12 a.m.)
> 
> 
> Review request for mesos, Alex Clemmer and Michael Park.
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> - Added missing `#include sys/prctl.h`.
> - Replaced `close` with `::close`.
> - Fixed sign-mismatch comparison.
> 
> 
> Diffs
> -
> 
>   3rdparty/libprocess/include/process/posix/subprocess.hpp 
> 70794926a99ca0eccebfe01106cc0f11c842ca58 
> 
> Diff: https://reviews.apache.org/r/47287/diff/
> 
> 
> Testing
> ---
> 
> Local build succeeds.
> 
> 
> Thanks,
> 
> Kapil Arya
> 
>



Review Request 47287: Fixed compilation error caused by prctl and recent subprocess change.

2016-05-11 Thread Kapil Arya

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

Review request for mesos, Alex Clemmer and Michael Park.


Repository: mesos


Description
---

- Added missing `#include sys/prctl.h`.
- Replaced `close` with `::close`.
- Fixed sign-mismatch comparison.


Diffs
-

  3rdparty/libprocess/include/process/posix/subprocess.hpp 
70794926a99ca0eccebfe01106cc0f11c842ca58 

Diff: https://reviews.apache.org/r/47287/diff/


Testing
---

Local build succeeds.


Thanks,

Kapil Arya