> On Jan. 23, 2015, 5:38 p.m., Timothy St. Clair wrote: > > 3rdparty/libprocess/configure.ac, line 178 > > <https://reviews.apache.org/r/30048/diff/1/?file=825235#file825235line178> > > > > maybe I'm missing something, but isn't 'test -d' to check directories? > > Kapil Arya wrote: > Yeah, that's exactly what we want to test. The "--with-boost" expects a > directory name. > > Timothy St. Clair wrote: > We should probably bail then if the input is invalid, just to up the bar > a little. > > Kapil Arya wrote: > Wouldn't that be somewhat redundant with the existing code where we > verify the library/package at the given path? > > Timothy St. Clair wrote: > Not really, because you could have multiple versions in your path. > > Timothy St. Clair wrote: > & not all the checks are done prior to compile, only a subset.
If there are multiple versions, wouldn't there be a default version that gets used for linking, etc., when you try to use a library? I took a quick look at configure.ac and it looks like we test for valid packages/libraries in order to set the corresponding AM contitional. If something is missing, I would suggest that we add the missing tests. - Kapil ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30048/#review69493 ----------------------------------------------------------- On Jan. 19, 2015, 2:32 p.m., Kapil Arya wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30048/ > ----------------------------------------------------------- > > (Updated Jan. 19, 2015, 2:32 p.m.) > > > Review request for mesos, Cody Maloney and Timothy St. Clair. > > > Bugs: MESOS-2235 > https://issues.apache.org/jira/browse/MESOS-2235 > > > Repository: mesos-git > > > Description > ------- > > For dependencies, if installation path is not provided, assume system default > locations. > > > Diffs > ----- > > 3rdparty/libprocess/configure.ac 19bc601247f3b1fc6b28d3474fb39f7369bb382f > > Diff: https://reviews.apache.org/r/30048/diff/ > > > Testing > ------- > > ../configure --with-protobuf && make > > > Thanks, > > Kapil Arya > >