Re: Review Request 40326: libprocess: Marked Boost as a "system" header.

2015-12-08 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40326/#review109385 --- Ship it! Ship It! - Joris Van Remoortere On Nov. 16, 2015,

Re: Review Request 40326: libprocess: Marked Boost as a "system" header.

2015-11-16 Thread Neil Conway
> On Nov. 16, 2015, 8:02 a.m., Benjamin Bannier wrote: > > 3rdparty/libprocess/3rdparty/Makefile.am, line 216 > > > > > > I think it would make sense to also update the mesos `CMakeLists.txt`. We should, but it

Re: Review Request 40326: libprocess: Marked Boost as a "system" header.

2015-11-16 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40326/ --- (Updated Nov. 16, 2015, 8:47 a.m.) Review request for mesos, Benjamin Bannier

Re: Review Request 40326: libprocess: Marked Boost as a "system" header.

2015-11-16 Thread Neil Conway
> On Nov. 16, 2015, 7:55 a.m., Joerg Schad wrote: > > 3rdparty/libprocess/configure.ac, line 211 > > > > > > Should we add this to the CMake files as well? We should, but it requires some other changes -- we want

Re: Review Request 40326: libprocess: Marked Boost as a "system" header.

2015-11-16 Thread Joerg Schad
> On Nov. 16, 2015, 7:55 a.m., Joerg Schad wrote: > > 3rdparty/libprocess/configure.ac, line 211 > > > > > > Should we add this to the CMake files as well? > > Neil Conway wrote: > We should, but it requires

Re: Review Request 40326: libprocess: Marked Boost as a "system" header.

2015-11-16 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40326/#review106614 --- Ship it! Ship It! - Joerg Schad On Nov. 16, 2015, 8:47 a.m.,

Re: Review Request 40326: libprocess: Marked Boost as a "system" header.

2015-11-16 Thread Neil Conway
> On Nov. 16, 2015, 7:55 a.m., Joerg Schad wrote: > > 3rdparty/libprocess/configure.ac, line 211 > > > > > > Should we add this to the CMake files as well? > > Neil Conway wrote: > We should, but it requires

Re: Review Request 40326: libprocess: Marked Boost as a "system" header.

2015-11-16 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40326/#review106604 --- Ship it! 3rdparty/libprocess/3rdparty/Makefile.am (line 212)

Review Request 40326: libprocess: Marked Boost as a "system" header.

2015-11-15 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40326/ --- Review request for mesos, Benjamin Bannier and Joris Van Remoortere. Bugs:

Re: Review Request 40326: libprocess: Marked Boost as a "system" header.

2015-11-15 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40326/#review106602 --- 3rdparty/libprocess/configure.ac (line 211)