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

Reply via email to