mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Bannier <benjamin.bann...@mesosphere.io>
Subject Re: Review Request 68319: Put binaries into root folder in libprocess.
Date Mon, 20 Aug 2018 18:30:20 GMT


> On Aug. 20, 2018, 8:18 nachm., Andrew Schwartzmeyer wrote:
> > I have to ask: why? I've personally thought it would be nicer to remove this rather
unexpected logic (from a CMake standpoint) from Mesos. Why add it to stout and libprocess?

Thanks for engaging. The main idea would be to make the various binaries easier to run, e.g.,
when manually executing tests. IMO the alternative to the patches proposed here would be to
all put them into a shared, _global_ folder -- many projects use bin/ for that.


- Benjamin


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


On Aug. 20, 2018, 11:53 vorm., Benjamin Bannier wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/68319/
> -----------------------------------------------------------
> 
> (Updated Aug. 20, 2018, 11:53 vorm.)
> 
> 
> Review request for mesos and Andrew Schwartzmeyer.
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Put binaries into root folder in libprocess.
> 
> 
> Diffs
> -----
> 
>   3rdparty/libprocess/CMakeLists.txt 5bba8e2c59ea3d7c37d186a2273bcaad6ffb5c46 
>   3rdparty/libprocess/src/tests/CMakeLists.txt a03a77eb5e289b4daac0bbd414dc17c8acc848dc

> 
> 
> Diff: https://reviews.apache.org/r/68319/diff/1/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Benjamin Bannier
> 
>


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