On Thu, Nov 04, 2010 at 02:09:38PM +0100, Landry Breuil wrote: > On Thu, Nov 04, 2010 at 02:04:24PM +0100, Landry Breuil wrote: > > On Thu, Nov 04, 2010 at 01:42:01PM +0100, Landry Breuil wrote: > > > On Wed, Nov 03, 2010 at 08:17:05AM +0100, Jasper Lievisse Adriaanse wrote: > > > > On Sun, Oct 31, 2010 at 12:33:46PM +0100, Marc Espie wrote: > > > > > On Mon, Oct 25, 2010 at 11:27:25PM +0200, Tobias Ulmer wrote: > > > > > > Hi Marc, > > > > > > > > > > > > metaauto should IMHO return an error code instead of failing > > > > > > silently. > > > > > > > > > > > We've now tested this in a bulk buil. It doesn't quite work, so it > > > > > won't > > > > > go in until errors in the corresponding ports are fixed... ;-( > > > > which ports are those? > > > > > > with the diff, automake 1.10 breaks with : > > > checking whether autoconf is installed... no > > > configure: error: Autoconf 2.60 or better is required. > > > Please make sure it is installed and in your PATH. > > > > > > automake 1.9 breaks with : > > > checking whether autoconf is installed... no > > > configure: error: Autoconf 2.58 or better is required. > > > Please make sure it is installed and in your PATH. > > > > > > and iirc other automake ports fails too. > > > > > > so i think build_depends for those ports should be adjusted, but i have > > > no idea how it is related to the metaauto diff itself. > > > > Actually it's simpler than that, and automake 1.8 breaks too : > > > > configure does: > > > > configure:1825: eval autoconf --version > > Provide an AUTOCONF_VERSION environment variable, please > > configure:1828: $? = 127 > > > > So, automake ports should build_depends on the needed version of > > autoconf, and CONFIGURE_ENV+= AUTOCONF_VERSION=${AUTOCONF_VERSION} > > should be added too. > > print/cups breaks too: > > ===> Configuring for cups-1.4.4p6 > cd /usr/obj/ports//cups-1.4.4/cups-1.4.4 && AUTOMAKE_VERSION=1.9 aclocal > -I config-scripts > Provide an AUTOCONF_VERSION environment variable, please > aclocal-1.9: autom4te failed with exit status: 127 > > - cd ${WRKSRC} && AUTOMAKE_VERSION=${AUTOMAKE_VERSION} aclocal -I > config-scripts > - cd ${WRKSRC} && AUTOCONF_VERSION=${AUTOCONF_VERSION} autoconf -I > config-scripts > + cd ${WRKSRC} && AUTOMAKE_VERSION=${AUTOMAKE_VERSION} > AUTOCONF_VERSION=${AUTOCONF_VERSION} aclocal -I config-scripts > + cd ${WRKSRC} && AUTOMAKE_VERSION=${AUTOMAKE_VERSION} > AUTOCONF_VERSION=${AUTOCONF_VERSION} autoconf -I config-scripts > > Fixes it.
databases/freetds breaks too.. for an unknown reason even as it doesn't depend on metaauto/autoconf/automake, authoeader and aclocal are triggered during build, and thus fails due to missing AUTO*_VERSION. ===> Building for freetds-0.82.1pre20100728p0 Making all in include cd .. && make am--refresh cd .. && /bin/sh /usr/obj/ports/freetds-0.82.1pre20100728/freetds-0.82/missing --run autoheader Provide an AUTOCONF_VERSION environment variable, please WARNING: `autoheader' is missing on your system. You should only need it if you modified `acconfig.h' or `configure.ac'. You might want to install the `Autoconf' and `GNU m4' packages. Grab them from any GNU archive site. ... ... ... cd . && /bin/sh /usr/obj/ports/freetds-0.82.1pre20100728/freetds-0.82/missing --run aclocal-1.9 -I m4 Provide an AUTOCONF_VERSION environment variable, please aclocal-1.9: autom4te failed with exit status: 127 *** Error code 1 While here tds.h could use some love to avoid this warning : In file included from ../../include/tds.h:30, from server.c:30: /usr//include/arpa/inet.h:74: warning: 'struct in_addr' declared inside parameter list /usr//include/arpa/inet.h:74: warning: its scope is only this definition or declaration, which is probably not what you want /usr//include/arpa/inet.h:75: warning: 'struct in_addr' declared inside parameter list Landry