> On Oct. 23, 2014, 9:37 p.m., Ben Mahler wrote: > > I'm curious if we can make the kernel + libnl version checking explicit at > > configure time. > > > > It would be nice to tell them when running ./configure that their kernel or > > libnl version are not valid for enabling network isolation, rather than > > failing make check, or failing at runtime. > > Jie Yu wrote: > We delibrately choose to do dynamic check because the libnl that mesos > links at runtime maybe different from the libnl during compilation. Same > argument for kernel version. > > Ben Mahler wrote: > Agreed, I'm not saying get rid of the dynamic check, that is definitely > required. > > I'm just saying, it would be nice to tell the user at configure time that > what they're trying to configure is not valid. Up to you guys.
Make sense. I'll create a ticket for tracking it. - Jie ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27105/#review58114 ----------------------------------------------------------- On Oct. 23, 2014, 7:09 p.m., Chi Zhang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/27105/ > ----------------------------------------------------------- > > (Updated Oct. 23, 2014, 7:09 p.m.) > > > Review request for mesos, Jie Yu and Cong Wang. > > > Bugs: mesos-1967 > https://issues.apache.org/jira/browse/mesos-1967 > > > Repository: mesos-git > > > Description > ------- > > see summary. > > > Diffs > ----- > > src/linux/routing/utils.cpp 80b62d0 > src/tests/routing_tests.cpp bd4d071 > > Diff: https://reviews.apache.org/r/27105/diff/ > > > Testing > ------- > > Tested on a 3.4 and 3.10, and made sure on the check works on 3.4. > > > Thanks, > > Chi Zhang > >