mesos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jie Yu" <yujie....@gmail.com>
Subject Re: Review Request 24679: Added os::execvpe to stout.
Date Thu, 14 Aug 2014 02:39:25 GMT

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

(Updated Aug. 14, 2014, 2:39 a.m.)


Review request for mesos, Benjamin Hindman and Ben Mahler.


Changes
-------

BenM's comments.


Repository: mesos-git


Description
-------

The idea here is to use the 'p' (search PATH) version consistently in the code base.

Mac does not have os::execvpe defined, so I end up implementing it per this thread:
http://stackoverflow.com/questions/7789750/execve-with-path-search


Diffs (updated)
-----

  3rdparty/libprocess/3rdparty/stout/include/stout/os.hpp 4d67186 

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


Testing
-------

make check


Thanks,

Jie Yu


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message