Re: [PATCH management.git] Fix autotools to include the necessary M4 files
On Mon, Oct 25, 2010 at 08:05:56PM +0200, Sasha Khapyorsky wrote: Hi Jason, On 16:33 Thu 07 Oct , Jason Gunthorpe wrote: Otherwise running autogen.sh with a new version of autotools and then building on a system with an older version tends to explode. Unfortunately this is sometimes necessary since the new version is required by the package. This is how GNU envisions this mess works at least.. Signed-off-by: Jason Gunthorpe jguntho...@obsidianresearch.com infiniband-diags/Makefile.am |1 + infiniband-diags/configure.in |1 + libibmad/Makefile.am |2 +- libibmad/configure.in |1 + libibumad/Makefile.am |2 +- libibumad/configure.in|1 + opensm/configure.in |1 + 7 files changed, 7 insertions(+), 2 deletions(-) diff --git a/infiniband-diags/Makefile.am b/infiniband-diags/Makefile.am index af90b05..e4d3d90 100644 +++ b/infiniband-diags/Makefile.am @@ -1,3 +1,4 @@ +ACLOCAL_AMFLAGS = -I config SUBDIRS = libibnetdisc INCLUDES = -I$(top_builddir)/include/ -I$(srcdir)/include -I$(includedir) \ diff --git a/infiniband-diags/configure.in b/infiniband-diags/configure.in index b9326c0..3c16aec 100644 +++ b/infiniband-diags/configure.in @@ -3,6 +3,7 @@ dnl Process this file with autoconf to produce a configure script. AC_PREREQ(2.57) AC_INIT(infiniband-diags, 1.5.7, linux-rdma@vger.kernel.org) AC_CONFIG_AUX_DIR(config) +AC_CONFIG_MACRO_DIR(config) AM_CONFIG_HEADER(config.h) AM_INIT_AUTOMAKE infiniband-diags directory does have any custom m4 macros (libibmad and libibumad too). Could you explain why should it fail without the change? Is this for potential future use? No, this is needed right now, libtool will install its custom m4 macros if this directive is present, otherwise it doesn't and that can muck things up later. If you don't have this directive then you get warnings from libtool about this possible risk: jggl{jgg}/tmp/ofa/management/libibumad#./autogen.sh + aclocal -I config + libtoolize --force --copy libtoolize: putting auxiliary files in AC_CONFIG_AUX_DIR, `config'. libtoolize: copying file `config/ltmain.sh' libtoolize: Consider adding `AC_CONFIG_MACRO_DIR([m4])' to configure.in and libtoolize: rerunning libtoolize, to keep the correct libtool macros in-tree. libtoolize: Consider adding `-I m4' to ACLOCAL_AMFLAGS in Makefile.am. + autoheader + automake --foreign --add-missing --copy configure.in:45: installing `config/config.guess' configure.in:45: installing `config/config.sub' configure.in:8: installing `config/install-sh' configure.in:8: installing `config/missing' Makefile.am: installing `config/depcomp' + autoconf vs after patching: jggl{jgg}/tmp/ofa/management/libibumad#./autogen.sh + aclocal -I config + libtoolize --force --copy libtoolize: putting auxiliary files in AC_CONFIG_AUX_DIR, `config'. libtoolize: copying file `config/ltmain.sh' libtoolize: putting macros in AC_CONFIG_MACRO_DIR, `config'. libtoolize: copying file `config/libtool.m4' libtoolize: copying file `config/ltoptions.m4' libtoolize: copying file `config/ltsugar.m4' libtoolize: copying file `config/ltversion.m4' libtoolize: copying file `config/lt~obsolete.m4' + autoheader + automake --foreign --add-missing --copy + autoconf Much happier. Jason -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH management.git] Fix autotools to include the necessary M4 files
On 12:14 Mon 25 Oct , Jason Gunthorpe wrote: vs after patching: jggl{jgg}/tmp/ofa/management/libibumad#./autogen.sh + aclocal -I config + libtoolize --force --copy libtoolize: putting auxiliary files in AC_CONFIG_AUX_DIR, `config'. libtoolize: copying file `config/ltmain.sh' libtoolize: putting macros in AC_CONFIG_MACRO_DIR, `config'. libtoolize: copying file `config/libtool.m4' libtoolize: copying file `config/ltoptions.m4' libtoolize: copying file `config/ltsugar.m4' libtoolize: copying file `config/ltversion.m4' libtoolize: copying file `config/lt~obsolete.m4' + autoheader + automake --foreign --add-missing --copy + autoconf Much happier. Ok. I see now. Sasha -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH management.git] Fix autotools to include the necessary M4 files
On 16:33 Thu 07 Oct , Jason Gunthorpe wrote: Otherwise running autogen.sh with a new version of autotools and then building on a system with an older version tends to explode. Unfortunately this is sometimes necessary since the new version is required by the package. This is how GNU envisions this mess works at least.. Signed-off-by: Jason Gunthorpe jguntho...@obsidianresearch.com Applied. Thanks. Sasha -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH management.git] Fix autotools to include the necessary M4 files
Otherwise running autogen.sh with a new version of autotools and then building on a system with an older version tends to explode. Unfortunately this is sometimes necessary since the new version is required by the package. This is how GNU envisions this mess works at least.. Signed-off-by: Jason Gunthorpe jguntho...@obsidianresearch.com --- infiniband-diags/Makefile.am |1 + infiniband-diags/configure.in |1 + libibmad/Makefile.am |2 +- libibmad/configure.in |1 + libibumad/Makefile.am |2 +- libibumad/configure.in|1 + opensm/configure.in |1 + 7 files changed, 7 insertions(+), 2 deletions(-) diff --git a/infiniband-diags/Makefile.am b/infiniband-diags/Makefile.am index af90b05..e4d3d90 100644 --- a/infiniband-diags/Makefile.am +++ b/infiniband-diags/Makefile.am @@ -1,3 +1,4 @@ +ACLOCAL_AMFLAGS = -I config SUBDIRS = libibnetdisc INCLUDES = -I$(top_builddir)/include/ -I$(srcdir)/include -I$(includedir) \ diff --git a/infiniband-diags/configure.in b/infiniband-diags/configure.in index b9326c0..3c16aec 100644 --- a/infiniband-diags/configure.in +++ b/infiniband-diags/configure.in @@ -3,6 +3,7 @@ dnl Process this file with autoconf to produce a configure script. AC_PREREQ(2.57) AC_INIT(infiniband-diags, 1.5.7, linux-rdma@vger.kernel.org) AC_CONFIG_AUX_DIR(config) +AC_CONFIG_MACRO_DIR(config) AM_CONFIG_HEADER(config.h) AM_INIT_AUTOMAKE diff --git a/libibmad/Makefile.am b/libibmad/Makefile.am index 6881453..f732c6c 100644 --- a/libibmad/Makefile.am +++ b/libibmad/Makefile.am @@ -1,4 +1,4 @@ - +ACLOCAL_AMFLAGS = -I config SUBDIRS = . INCLUDES = -I$(srcdir)/include -I$(includedir) diff --git a/libibmad/configure.in b/libibmad/configure.in index 23ec914..97e8cb3 100644 --- a/libibmad/configure.in +++ b/libibmad/configure.in @@ -4,6 +4,7 @@ AC_PREREQ(2.57) AC_INIT(libibmad, 1.3.6, linux-rdma@vger.kernel.org) AC_CONFIG_SRCDIR([src/sa.c]) AC_CONFIG_AUX_DIR(config) +AC_CONFIG_MACRO_DIR(config) AM_CONFIG_HEADER(config.h) AM_INIT_AUTOMAKE diff --git a/libibumad/Makefile.am b/libibumad/Makefile.am index 6e3f518..28bf69c 100644 --- a/libibumad/Makefile.am +++ b/libibumad/Makefile.am @@ -1,4 +1,4 @@ - +ACLOCAL_AMFLAGS = -I config SUBDIRS = . INCLUDES = -I$(srcdir)/include/infiniband -I$(includedir) diff --git a/libibumad/configure.in b/libibumad/configure.in index 9fcd609..04ecb0e 100644 --- a/libibumad/configure.in +++ b/libibumad/configure.in @@ -4,6 +4,7 @@ AC_PREREQ(2.57) AC_INIT(libibumad, 1.3.6, linux-rdma@vger.kernel.org) AC_CONFIG_SRCDIR([src/umad.c]) AC_CONFIG_AUX_DIR(config) +AC_CONFIG_MACRO_DIR(config) AM_CONFIG_HEADER(config.h) AM_INIT_AUTOMAKE diff --git a/opensm/configure.in b/opensm/configure.in index 8695965..057724a 100644 --- a/opensm/configure.in +++ b/opensm/configure.in @@ -4,6 +4,7 @@ AC_PREREQ(2.57) AC_INIT(opensm, 3.3.7, linux-rdma@vger.kernel.org) AC_CONFIG_SRCDIR([opensm/osm_opensm.c]) AC_CONFIG_AUX_DIR(config) +AC_CONFIG_MACRO_DIR(config) AC_CONFIG_HEADERS(include/config.h include/opensm/osm_config.h) AM_INIT_AUTOMAKE -- 1.6.0.4 -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html