mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Vinod Kone <>
Subject Re: Review Request 46669: Added deprecation support to Flag name.
Date Fri, 29 Apr 2016 03:28:35 GMT

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

(Updated April 29, 2016, 3:28 a.m.)

Review request for mesos, Ben Mahler, Greg Mann, and Michael Park.


Ben's comments.

Bugs: MESOS-5271

Repository: mesos


Added deprecation support to Flag name.

Diffs (updated)

  3rdparty/libprocess/3rdparty/stout/include/stout/flags/flag.hpp 2502d9e8a515b9adc1b3aa2f719e5710a0e7ef29

  3rdparty/libprocess/3rdparty/stout/include/stout/flags/flags.hpp c3cbcdb781e1c282d381de1ad2bf4f386ee1db21

  3rdparty/libprocess/3rdparty/stout/include/stout/subcommand.hpp 4d37c4afdcf4c3d6d511dd8a27916332b086afa7

  3rdparty/libprocess/3rdparty/stout/tests/flags_tests.cpp 66e16abe914e2a1ee7599bab857ff478b7ec20dc



Tested manually by running ./bin/ --authenticate

The one caveat is that the deprecation warning might be printed before logging the library
is initialized (e.g., master/main.cpp calls flags.load() before logging::initialize(flags)).


Vinod Kone

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