----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55065/#review170209 -----------------------------------------------------------
src/slave/flags.cpp Lines 152-158 (original), 152-169 (patched) <https://reviews.apache.org/r/55065/#comment242994> We typically don't create duplicate entries like this when simply customizing the default value. Instead we use a lambda with the #ifdef's inside of it. For example, https://github.com/apache/mesos/blob/master/src/slave/flags.cpp#L212 - Kevin Klues On March 27, 2017, 9:25 a.m., Ayanampudi Varsha wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55065/ > ----------------------------------------------------------- > > (Updated March 27, 2017, 9:25 a.m.) > > > Review request for mesos, Till Toenshoff and Vinod Kone. > > > Bugs: MESOS-6742 > https://issues.apache.org/jira/browse/MESOS-6742 > > > Repository: mesos > > > Description > ------- > > Added support for s390x architecture > There are 2 issues addressed in this patch: > 1. LdcacheTest.Parse test case fails on s390x machines. > 2. From the value of flag docker_registry in slave/flags.cpp, amd64 images > get downloaded due to which test cases fail on s390x with "Exec format Error" > > > Diffs > ----- > > src/linux/ldcache.cpp 6260496672cd0226a24ea1b34003a48af9678a63 > src/slave/flags.cpp 1eccea920338032173be4df6c374ec50dbd2eaf3 > > > Diff: https://reviews.apache.org/r/55065/diff/1/ > > > Testing > ------- > > make check > make check GTEST_FILTER="LdcacheTest.Parse" > > > Thanks, > > Ayanampudi Varsha > >
