> On Aug. 16, 2016, 6:15 p.m., Jiang Yan Xu wrote: > > include/mesos/version.hpp.in, line 24 > > <https://reviews.apache.org/r/50992/diff/2/?file=1473426#file1473426line24> > > > > Ideally the TODO is to change the variables to integers after the > > deprecation cycles.
Yes, something like the following works: ``` #ifdef MESOS_MAJOR_VERSION_INT #if MESOS_MAJOR_VERSION_INT >= 1 ...... #endif #endif ``` - Zhitao ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50992/#review145892 ----------------------------------------------------------- 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 > >
