> On May 23, 2018, 8:39 p.m., Benjamin Bannier wrote: > > Could you follow this up with a patch which compiles the example at least > > as part of `check`? > > > > For cmake we could e.g., add a `CMakeLists.txt` to the `example/` directory > > with just > > > > add_executable(example EXCLUDE_FROM_ALL example.cpp) > > target_link_libraries(example process) > > add_dependencies(libprocess-tests example) > > > > and add the subdirectory to `libprocess/CMakeLists.txt`. > > > > Similar for autotools. > > > > Maybe we also should format the file according to our style guide in > > another patch. > > Benno Evers wrote: > Sorry, I don't really have the time right now to work on that. But I'll > volunteer to review the patch if you want to write it.
https://reviews.apache.org/r/67281/ for formatting of the current code, https://reviews.apache.org/r/67280/ for adding the fixed files to the build setups. - Benjamin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67270/#review203676 ----------------------------------------------------------- On May 23, 2018, 7:22 p.m., Benno Evers wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67270/ > ----------------------------------------------------------- > > (Updated May 23, 2018, 7:22 p.m.) > > > Review request for mesos and Benjamin Mahler. > > > Repository: mesos > > > Description > ------- > > The 'example.cpp' file in libprocess was missing the mandatory > help string argument. > > > Diffs > ----- > > 3rdparty/libprocess/examples/example.cpp > a97665ee6b75c5b31397b5a10002fa23a7fe370a > > > Diff: https://reviews.apache.org/r/67270/diff/1/ > > > Testing > ------- > > Compiled the `example.cpp` file. > > > Thanks, > > Benno Evers > >