You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@mesos.apache.org by Alex Clemmer <cl...@gmail.com> on 2015/09/10 22:53:12 UTC

Re: Review Request 37369: Add firewall.cpp to the process library CMakeLists.

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

(Updated Sept. 10, 2015, 8:53 p.m.)


Review request for mesos, Benjamin Hindman, Artem Harutyunyan, Joris Van Remoortere, and Joseph Wu.


Repository: mesos


Description
-------

Right now, if you try to build on a non-Windows machine with CMakeLists,
you'll get a linker error claiming that we can't find a symbol from the
process library's firewall.cpp.

This commit simply adds that source file, which fixes the problem.


Diffs
-----

  3rdparty/libprocess/src/CMakeLists.txt 9d1b1f54d63b3c4740550c2b7934200c54b48021 

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


Testing
-------


Thanks,

Alex Clemmer