Re: Review Request 69311: Enabled parallel test runner to cmake build.

2019-07-31 Thread Benjamin Bannier
> On Nov. 13, 2018, 11:48 a.m., Benno Evers wrote: > > src/tests/CMakeLists.txt > > Lines 365 (patched) > > > > > > Does this preserve user-specified `MESOS_GTEST_RUNNER_FLAGS`? Or, the > > other way round, will a

Re: Review Request 69311: Enabled parallel test runner to cmake build.

2019-07-31 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69311/ --- (Updated July 31, 2019, 12:10 p.m.) Review request for mesos, Benjamin Mahler

Re: Review Request 69311: Enabled parallel test runner to cmake build.

2018-11-20 Thread Till Toenshoff via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69311/#review210707 --- Ship it! Ship It! - Till Toenshoff On Nov. 14, 2018, 1:11

Re: Review Request 69311: Enabled parallel test runner to cmake build.

2018-11-15 Thread Benno Evers
> On Nov. 13, 2018, 10:48 a.m., Benno Evers wrote: > > src/tests/CMakeLists.txt > > Lines 365 (patched) > > > > > > Does this preserve user-specified `MESOS_GTEST_RUNNER_FLAGS`? Or, the > > other way round, will a

Re: Review Request 69311: Enabled parallel test runner to cmake build.

2018-11-14 Thread Joseph Wu
> On Nov. 13, 2018, 2:48 a.m., Benno Evers wrote: > > src/tests/CMakeLists.txt > > Lines 365 (patched) > > > > > > Does this preserve user-specified `MESOS_GTEST_RUNNER_FLAGS`? Or, the > > other way round, will a

Re: Review Request 69311: Enabled parallel test runner to cmake build.

2018-11-14 Thread Benjamin Bannier
> On Nov. 13, 2018, 11:48 a.m., Benno Evers wrote: > > cmake/MesosConfigure.cmake > > Lines 56 (patched) > > > > > > If I configure with `ENABLE_PARALLEL_TEST_EXECUTION=ON` and then > > reconfigure with

Re: Review Request 69311: Enabled parallel test runner to cmake build.

2018-11-14 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69311/ --- (Updated Nov. 14, 2018, 2:11 p.m.) Review request for mesos, Benjamin Mahler

Re: Review Request 69311: Enabled parallel test runner to cmake build.

2018-11-13 Thread Joseph Wu
> On Nov. 13, 2018, 2:48 a.m., Benno Evers wrote: > > cmake/MesosConfigure.cmake > > Lines 56 (patched) > > > > > > If I configure with `ENABLE_PARALLEL_TEST_EXECUTION=ON` and then > > reconfigure with

Re: Review Request 69311: Enabled parallel test runner to cmake build.

2018-11-13 Thread Benno Evers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69311/#review210500 --- Fix it, then Ship it! cmake/MesosConfigure.cmake Lines 56

Review Request 69311: Enabled parallel test runner to cmake build.

2018-11-10 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69311/ --- Review request for mesos, Benjamin Mahler and Joseph Wu. Repository: mesos