Re: [PATCH] automake: do not require ltmain.sh for out-of-tree libtool

2017-10-17 Thread Paolo Bonzini
On 17/10/2017 13:45, Mathieu Lirzin wrote: >>> 1 file changed, 11 insertions(+), 1 deletion(-) > I haven't tested this, and I am not a Libtool expert so I trust your > analysis. > > What do you think of adding a test ensuring that ltmain.sh is not > required when no Libtool macro is found? > >

Re: [PATCH] automake: do not require ltmain.sh for out-of-tree libtool

2017-10-12 Thread Paolo Bonzini
On 31/10/2016 13:30, Paolo Bonzini wrote: > If Automake does not see LT_SUPPORTED_TAG, it assumes an old libtool > that does not know about AC_REQUIRE_AUX_FILE. However, if the program > does not use Libtool's configure.ac macros this check gets a > false positive. Do not requi

[PATCH] automake: do not require ltmain.sh for out-of-tree libtool

2016-10-31 Thread Paolo Bonzini
that are not stone-age are already covered by LT_SUPPORTED_TAG and _LT_AC_TAGCONFIG, but add AC_PROG_LIBTOOL just in case for Libtool up to 1.4. 2016-10-31 Paolo Bonzini <bonz...@gnu.org> * bin/automake.in ($libtool_bundled): New. (handle_libtool): Do not require libtool

Re: [PATCHES] Better error messages if obsolete macros are used

2012-12-31 Thread Paolo Bonzini
Il 31/12/2012 10:05, Stefano Lattarini ha scritto: On 12/30/2012 07:08 PM, Paolo Bonzini wrote: Il 30/12/2012 11:23, Stefano Lattarini ha scritto: +AC_DEFUN([AM_CONFIG_HEADER], +[AC_FATAL(['$0': this macro is obsolete. +You should use the 'AC][_CONFIG_HEADERS' macro instead

Re: [PATCHES] Better error messages if obsolete macros are used

2012-12-31 Thread Paolo Bonzini
Il 31/12/2012 11:32, Stefano Lattarini ha scritto: It is indeed possible that the real reason that pushed me to remove AM_CONFIG_HEADER was the fact that I got caught in a cleanup frenzy when I was removing other (real) cruft. You are starting to partly convince me of that. These patches are

Re: [PATCHES] Better error messages if obsolete macros are used

2012-12-30 Thread Paolo Bonzini
Il 30/12/2012 11:23, Stefano Lattarini ha scritto: +AC_DEFUN([AM_CONFIG_HEADER], +[AC_FATAL(['$0': this macro is obsolete. +You should use the 'AC][_CONFIG_HEADERS' macro instead.])]) + What's the point in doing this instead of m4_defun([AM_CONFIG_HEADER], [AC_CONFIG_HEADERS]) or

Re: [PATCH] aclocal: handle ACLOCAL_PATH environment variable

2011-09-20 Thread Paolo Bonzini
On 09/20/2011 12:51 PM, Stefano Lattarini wrote: Yes (and that's on purpose: ACLOCAL_PATH is typically used by a user which has newer versions of libraries installed, so they need their directories to override everything). Yes, this might be desirable. But then, for consistency sake,

Re: [PATCH] aclocal: handle ACLOCAL_PATH environment variable

2011-09-20 Thread Paolo Bonzini
On 09/20/2011 02:09 PM, Stefano Lattarini wrote: Yeah, I think the problem is that in the normal search path (from `aclocal -I /foo -I /bar'): 1. `/foo' 2. `/bar' 3. ACDIR-APIVERSION 4. ACDIR The directories from dirlist and ACLOCAL_PATH should go after (3), rather than

Re: [PATCH] aclocal: handle ACLOCAL_PATH environment variable

2011-09-19 Thread Paolo Bonzini
On 09/19/2011 06:05 PM, Stefano Lattarini wrote: I'll push the attached patch in 72 hours if there is no review by then. Paolo, since it's you who have written the previous version of this patch, from which I've drawn heavily, are you ok to be listed as the main author in the ChangeLog entry,

Re: [PATCH] aclocal: handle ACLOCAL_PATH environment variable

2011-09-19 Thread Paolo Bonzini
On 09/19/2011 06:05 PM, Stefano Lattarini wrote: Resurrecting the old thread Add ACLOCAL_PATH to aclocal; references: http://lists.gnu.org/archive/html/automake-patches/2010-11/msg00089.html http://thread.gmane.org/gmane.comp.sysutils.automake.patches/4972 I really want the ACLOCAL_PATH

Re: [PATCH 2/3] aclocal: remove @automake_includes

2010-11-14 Thread Paolo Bonzini
On 11/14/2010 05:46 PM, Ralf Wildenhues wrote: Hi Paolo, * Paolo Bonzini wrote on Tue, Nov 09, 2010 at 08:14:39PM CET: This patch simplifies the overly complicated rules for ACLOCAL_PATH vs. @automake_includes and @system_includes, by stating that ACLOCAL_PATH will override even

[PATCH 0/3] Add ACLOCAL_PATH to aclocal

2010-11-09 Thread Paolo Bonzini
and system directories to local and global, as discussed in the v1 thread as well. Paolo Bonzini (3): aclocal: handle ACLOCAL_PATH environment variable aclocal: remove @automake_includes aclocal: rename search path variables ChangeLog | 41 +++ NEWS

[PATCH 1/3] aclocal: handle ACLOCAL_PATH environment variable

2010-11-09 Thread Paolo Bonzini
/aclocal.in| 14 +++--- 9 files changed, 195 insertions(+), 8 deletions(-) create mode 100755 tests/acloca24.test create mode 100755 tests/acloca25.test diff --git a/ChangeLog b/ChangeLog index 5fff04a..fa43c14 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,17 @@ +2010-11-09 Paolo Bonzini

[PATCH 2/3] aclocal: remove @automake_includes

2010-11-09 Thread Paolo Bonzini
/acloca25.test |7 +-- 5 files changed, 41 insertions(+), 27 deletions(-) diff --git a/ChangeLog b/ChangeLog index fa43c14..ede73dc 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,20 @@ 2010-11-09 Paolo Bonzini bonz...@gnu.org + aclocal: remove @automake_includes. + * NEWS

[PATCH 3/3] aclocal: rename search path variables

2010-11-09 Thread Paolo Bonzini
changed, 37 insertions(+), 25 deletions(-) diff --git a/ChangeLog b/ChangeLog index ede73dc..b3ec1fb 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,17 @@ 2010-11-09 Paolo Bonzini bonz...@gnu.org + aclocal: rename search path variables. + * aclocal.in (user_includes): Rename

[PATCH] aclocal: handle ACLOCAL_PATH environment variable

2010-11-03 Thread Paolo Bonzini
Hi all, this patch provides a fourth scheme of adding directories to the search path. It is a simple colon-separated list of directories (without globbing). It is useful when you're using a global installation of Automake but you want to add directories from your account as well to the search

Re: [PATCH] aclocal: handle ACLOCAL_PATH environment variable

2010-11-03 Thread Paolo Bonzini
On 11/03/2010 04:24 PM, Stefano Lattarini wrote: + # Add any directory listed in the `ACLOCAL_PATH' environment + # variable. + if (defined $ENV{ACLOCAL_PATH}) +{ + foreach my $dir (split /:/, $ENV{ACLOCAL_PATH}) Shouldn't we use `...@path_separator@' here instead of `:', for better

Re: [trunk][patch] fix --enable-shared=some_libraries

2008-05-15 Thread Paolo Bonzini
Ok for trunk and 4.3, but please also handle enable_static in the same way. The patch is preapproved with this change, but please repost it so that it can also go in Automake (I'm CCing Ralf and automake-patches@gnu.org for this). Paolo

Re: [PATCH] conditional info_TEXINFOS, updated

2007-03-22 Thread Paolo Bonzini
PING... (or just give me commit access so I can do it myself). http://permalink.gmane.org/gmane.comp.sysutils.automake.patches/2731 Paolo

[PATCH] conditional info_TEXINFOS, updated

2007-03-12 Thread Paolo Bonzini
Ok, here's the patch I sent a month ago together with new testcases. I also found a little opportunity for refactoring. Paolo 2007-03-12 Paolo Bonzini [EMAIL PROTECTED] * automake.in (output_texinfo_build_rules): Add COND parameter. Emit INFO_DEPS and TEXINFOS

Re: support cross multilibs

2006-10-15 Thread Paolo Bonzini
I'd appreciate it if this could go on the 1.9.x branch as well as mainline. I think Alexandre doesn't plan another 1.9.x release. Distro makers might take the patch, though. I support Geoff's request. Paolo