> On Nov. 27, 2013, 11:20 p.m., Ben Mahler wrote: > > bootstrap, line 45 > > <https://reviews.apache.org/r/15862/diff/2/?file=391862#file391862line45> > > > > I was applying this patch and noticed this denies only AUTOMAKE but it > > does not deny only ACLOCAL: > > > > $ ACLOCAL=foo ./bootstrap > > autoreconf: Entering directory `.' > > autoreconf: configure.ac: not using Gettext > > autoreconf: running: foo -I m4 --output=aclocal.m4t > > Can't exec "foo": No such file or directory at > > /usr/local/Cellar/autoconf/2.69/share/autoconf/Autom4te/FileUtils.pm line > > 326. > > autoreconf: failed to run foo: No such file or directory > > > > Is this intended or should we fix this?
Thanks Ben! Will fix. - Bernardo ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15862/#review29524 ----------------------------------------------------------- On Nov. 27, 2013, 7:55 p.m., Bernardo Gomez Palacio wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/15862/ > ----------------------------------------------------------- > > (Updated Nov. 27, 2013, 7:55 p.m.) > > > Review request for mesos and Ben Mahler. > > > Repository: mesos-git > > > Description > ------- > > [MESOS-849] > > Enable AUTOMAKE and ACLOCAL env variables for bootstrap. > > The bootstrap script now supports an environment variable that points to > the AUTOMAKE, as well as ACLOCAL, that you want to use for building Mesos. > It also validates that if you specify an env entry for AUTOMAKE you do > the same for ACLOCAL. > > [ticket: MESOS-849]: As a developer I should be able to set the AUTOMAKE > and ACLOCAL environment variables for autoconf to pickup when using the > bootstrap script. > > > Diffs > ----- > > bootstrap 9e6c959 > > Diff: https://reviews.apache.org/r/15862/diff/ > > > Testing > ------- > > Tested using OSX > > #Base Scenario: > ./bootstrap > > #Defining AUTOMAKE without ACLOCAL: > # Expectation: ERROR message should be displayed. > AUTOMAKE="$(which automake-1.12)" ./bootstrap > > #Defining AUTOMAKE and ACLOCAL: > AUTOMAKE="$(which automake-1.12)" ACLOCAL="$(which aclocal-1.12)" ./bootstrap > > > Thanks, > > Bernardo Gomez Palacio > >
