----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20750/#review41567 -----------------------------------------------------------
Patch looks great! Reviews applied: [20750] All tests passed. - Mesos ReviewBot On April 27, 2014, 8:24 a.m., Bernardo Gomez Palacio wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20750/ > ----------------------------------------------------------- > > (Updated April 27, 2014, 8:24 a.m.) > > > Review request for mesos and Mesos ReviewBot. > > > Repository: mesos-git > > > Description > ------- > > [MESOS-1252]: Support ENV MAVEN_HOME on the build. > > The build now supports the `MAVEN_HOME` environment variable which is > used to resolve the `mvn` executable. If no `MAVEN_HOME` is specified we > will use the one available in the `PATH`. > > ticket: https://issues.apache.org/jira/browse/MESOS-1252 > > > Diffs > ----- > > configure.ac 01182b3 > src/Makefile.am c2029c7 > > Diff: https://reviews.apache.org/r/20750/diff/ > > > Testing > ------- > > Success > > 1. Testing MAVEN_HOME without a `mvn` executable in your path. > 1.1 Specify MAVEN_HOME=/path/to/your/maven/home `./configure` > 1.2 Check src/Makefile and validate that the variables MAVEN_HOME and MVN > point to the correct path. > 1.3 See that `$(MVN)` is used instead of `mvn`. > > 2. With a `mvn` executable in your path. > 2.1 Do not specify a MAVEN_HOME just make sure you have a `mvn` executable in > your path and call `./configure` > 2.2 Check src/Makefile and validate that the variables MAVEN_HOME is empty > but the MVN point to the correct executable. > 2.3 See that `$(MVN)` is used instead of `mvn`. > > > Failures > 2. With a MAVEN_HOME that has a `bin/mvn` executable missing. > 2.1 Call `MAVEN_HOME=/the/path ./configure` > 2.2 you should get an error that tells that `mvn` is missing from > "/the/path/bin". > > 2. Without a `mvn` executable in your path. > 2.1 Call `./configure` > 2.2 you should get an error that tells that `mvn` is missing. > > > Thanks, > > Bernardo Gomez Palacio > >
