----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50992/#review145910 -----------------------------------------------------------
include/mesos/version.hpp.in (line 25) <https://reviews.apache.org/r/50992/#comment212267> Should remove this line. include/mesos/version.hpp.in (line 27) <https://reviews.apache.org/r/50992/#comment212268> The `INT` specified seems out of place. We use semantic 2.0 versionign, so it will more like 'x.x.x', maybe: s/INT/NUM ? - Avinash sridharan On Aug. 15, 2016, 5:31 p.m., Zhitao Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50992/ > ----------------------------------------------------------- > > (Updated Aug. 15, 2016, 5:31 p.m.) > > > Review request for mesos, Xiaojian Huang, haosdent huang, James Peach, Joris > Van Remoortere, Vinod Kone, and Jiang Yan Xu. > > > Bugs: MESOS-4638 > https://issues.apache.org/jira/browse/MESOS-4638 > > > Repository: mesos > > > Description > ------- > > This makes version based conditional compiling much easier for > module writers. > > > Diffs > ----- > > include/mesos/version.hpp.in c8fbc822d91abb61df15715c0c6af98b0bfe2cb9 > > Diff: https://reviews.apache.org/r/50992/diff/ > > > Testing > ------- > > make on Mac OS and Linux. > > > Thanks, > > Zhitao Li > >