mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ammar Askar <>
Subject Re: Review Request 50002: Allow all flags load methods to specify a prefix.
Date Fri, 05 Aug 2016 23:58:14 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Aug. 5, 2016, 11:58 p.m.)

Review request for mesos, Greg Mann and Vinod Kone.

Bugs: MESOS-5613

Repository: mesos


Allow all flags load methods to specify a prefix.

This also refactors the prefix logic into one place, so that's nice.
Required for the actual fix for passing work_dir in local.

Diffs (updated)

  3rdparty/stout/include/stout/flags/flags.hpp dd93627 
  3rdparty/stout/tests/flags_tests.cpp 77f3a6a 



Added additional tests to flags_tests.cpp to ensure that prefix works on the other methods.

`make check` passes


Ammar Askar

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