[bug#67516] [PATCH] aclocal: handle both # and dnl for serial number lines

2023-11-28 Thread Nick Bowler
On 2023-11-28 16:06, Karl Berry wrote: > Hi Ross - you sent a change to automake-patches back in February 2017. > Sorry for the absurdly delayed reply. > > https://lists.gnu.org/archive/html/automake-patches/2017-02/msg1.html > > Instead of only accepting comment lines that start

[bug#65600] [PATCH] Allow parameters AM_PROG_LEX

2023-08-31 Thread Nick Bowler
On 31/08/2023, Karl Berry wrote: > Hi Bogdan, > > In reference to: > https://lists.gnu.org/archive/html/automake/2023-07/msg7.html. > > Thanks much! > > Since it's Autoconf 2.70 that started using the parameter, I've > bumped the required value. > > I don't think we should

[bug#60807] [PATCH v2] tests: reuse am_cv_filesystem_timestamp_resolution

2023-01-14 Thread Nick Bowler
On 14/01/2023, Zack Weinberg wrote: > On Sat, Jan 14, 2023, at 7:18 PM, Mike Frysinger wrote: >> Rather than assume such coarse delays, re-use existing logic for >> probing the current filesystem resolution. This speeds up the >> testsuite significantly.> [...] > No objection to this patch in

[bug#60535] [PATCH] depend2: switch echo|sed to automatic vars

2023-01-05 Thread Nick Bowler
On 2023-01-04, Mike Frysinger wrote: [...] > dmake is one implementation that fails, and your suggestion doesn't work > :/. > $ dmake foo/bar.o > dmake: Error: -- Incomplete macro expression [)' b='$(at_f:.o=)'; test > x"$$a.o" = x"$(@F)" || a=$$b;\ > echo $$a] It might also be

[bug#60535] [PATCH] depend2: switch echo|sed to automatic vars

2023-01-04 Thread Nick Bowler
On 2023-01-04, Mike Frysinger wrote: > On 04 Jan 2023 21:10, Nick Bowler wrote: [...] >> maybe something like: >> >> % cat >Makefile <<'EOF' >> at_f = $(@F) >> foo/bar.o: >> a='$(@F:.o=)' b='$(at_f:.o=)'; test x"$$a.o" = x"$(@F

[bug#60535] [PATCH] depend2: switch echo|sed to automatic vars

2023-01-04 Thread Nick Bowler
On 2023-01-03, Mike Frysinger wrote: > The echo|sed is used to split the dirname & filename so it can insert > $(DEPDIR) in the middle, and then chop the trailing object suffix. In > the generic case, %OBJ% is $@, so we can leverage the POSIX vars $(@D) > and $(@F) to do the pathname splitting

[RFC PATCH] Improve diagnostics wrt. mutliple AM_INIT_AUTOMAKE calls.

2021-08-16 Thread Nick Bowler
If you expand AM_INIT_AUTOMAKE more than once with options, automake currently barfs on the resulting m4 traces with a less-than-helpful assertion failure ("global options already processed"). This also prompts users to report bugs against Automake, when the actual problem is with their autoconf

Re: [PATCH] Add --jobserver/jobserver option for GCC -flto=jobserver

2020-10-28 Thread Nick Bowler
On 2020-10-28, Zack Weinberg wrote: > On Wed, Oct 28, 2020 at 2:16 PM Nick Bowler wrote: >> On 2020-10-28, H.J. Lu wrote: >> > GCC introduced some time ago option -flto=jobserver in order to use the >> > GNU Make jobserver when parallelising LTO builds. It is actuall

Re: [PATCH] Add --jobserver/jobserver option for GCC -flto=jobserver

2020-10-28 Thread Nick Bowler
On 2020-10-28, H.J. Lu wrote: > GCC introduced some time ago option -flto=jobserver in order to use the > GNU Make jobserver when parallelising LTO builds. It is actually a > similar "recursive make". When doing a recursive make, you need to > place a '+' character at the beginning of the

Re: [PATCH] up_to_date_p: treat equal mtime as outdated.

2020-04-14 Thread Nick Bowler
On 2020-04-13, Paul Eggert wrote: > I just checked, and GNU Make uses high-resolution file timestamps when > available, and considers a file to be up-to-date if it has exactly the same > timestamp as its dependency. I suspect that this is because Makefile rules > like > this: > > a: b > cp

Re: PATCH: silence Shellcheck warning

2018-07-22 Thread Nick Bowler
On 7/21/18, Ben Elliston wrote: > This patch silences a warning from Shellcheck about using old-style > `...` command substitutions. [...] > commit 4d35c7aae97234bf055519075ef03cd4090a1dfc > Author: Ben Elliston > Date: Sun Jul 22 08:22:44 2018 +1000 > > * missing: Use $(..) command

Re: bug#13435: Please don't kill DJGPP support...

2013-03-05 Thread Nick Bowler
, -- Nick Bowler, Elliptic Technologies (http://www.elliptictech.com/)

[PATCH 2/2] Automatically call AM_PROG_CC_C_O as required.

2013-01-13 Thread Nick Bowler
If subdir-objects is made the default Automake behaviour, packages will need to add AM_PROG_CC_C_O to their configure.ac. Instead of that, let's just make the functionality provided by AM_PROG_CC_C_O mandatory for C projects, and have Automake automatically call it as required. This change

Re: bug#13378: [PATCH] compile: use 'compile' script when -c -o is used with losing compilers

2013-01-13 Thread Nick Bowler
are required. These patches no longer apply to master, but here they are anyway. Nick Bowler (2): Use AC_DEFUN_ONCE to define AM_PROG_CC_C_O. Automatically call AM_PROG_CC_C_O as required. m4/init.m4 | 5 + m4/minuso.m4 | 2 +- 2 files changed, 6 insertions(+), 1 deletion(-) -- 1.8.1

[PATCH 1/2] Use AC_DEFUN_ONCE to define AM_PROG_CC_C_O.

2013-01-13 Thread Nick Bowler
If AM_PROG_CC_C_O is expanded multiple times, and the compiler does not support -c and -o together, each expansion of the macro will prepend the compile script to CC. This can result in the compile script invoking the compile script, which at best pointless and silly. Fortunately, there does not

Re: [PATCH 2/2] Automatically call AM_PROG_CC_C_O as required.

2013-01-13 Thread Nick Bowler
On 2013-01-13, Stefano Lattarini stefano.lattar...@gmail.com wrote: On 01/13/2013 09:01 PM, Nick Bowler wrote: +dnl Automatically invoke AM_PROG_CC_C_O as necessary. Since AC_PROG_CC is +dnl usually called after AM_INIT_AUTOMAKE, we arrange for the test to be +dnl done later

Re: [PATCH 6/6] aclocal: avoid spurious warnings from autom4te with AC_CONFIG_MACRO_DIRS

2012-11-07 Thread Nick Bowler
-m4defun /dev/null Cheers, -- Nick Bowler, Elliptic Technologies (http://www.elliptictech.com/)

Re: [PATCH 0/2] AC_CONFIG_MACRO_DIRS: implementation and documentation

2012-11-02 Thread Nick Bowler
On 2012-11-02 10:46 +0100, Stefano Lattarini wrote: On 10/23/2012 12:59 AM, Nick Bowler wrote: 4) I get a bunch of new warnings from aclocal, of the form: configure.ac:42: warning: DX_EXPORTED_SH is m4_require'd but not m4_defun'd The invocation comes from an AC_REQUIRE

Re: [PATCH 1/2] AC_CONFIG_MACRO_DIR: accept more than one argument

2012-09-20 Thread Nick Bowler
did... Cheers, -- Nick Bowler, Elliptic Technologies (http://www.elliptictech.com/)

Re: bug#10878: make dist with read-only srcdir generates read-only tarball

2012-02-25 Thread Nick Bowler
to modify the content of an existing file +in the distribution directory, it should explicitly ensure to make +it readable first: writable I also would drop the words ensure to. Cheers, -- Nick Bowler, Elliptic Technologies (http://www.elliptictech.com/)

Re: bug#10878: make dist with read-only srcdir generates read-only tarball

2012-02-24 Thread Nick Bowler
On 2012-02-24 18:37 +0100, Stefano Lattarini wrote: [...] On 02/24/2012 08:09 AM, Nick Bowler wrote: Automake should at least add user write permissions to all files in distdir prior to running dist-hook (and hence prior to generating the distribution tarball). I disagree; in case

Re: bug#10878: make dist with read-only srcdir generates read-only tarball

2012-02-24 Thread Nick Bowler
On 2012-02-24 19:19 +0100, Stefano Lattarini wrote: On 02/24/2012 06:53 PM, Nick Bowler wrote: On 2012-02-24 18:37 +0100, Stefano Lattarini wrote: [...] On 02/24/2012 08:09 AM, Nick Bowler wrote: Automake should at least add user write permissions to all files in distdir prior

Re: bug#10878: make dist with read-only srcdir generates read-only tarball

2012-02-24 Thread Nick Bowler
On 2012-02-24 20:25 +0100, Stefano Lattarini wrote: On 02/24/2012 07:34 PM, Nick Bowler wrote: On 2012-02-24 19:19 +0100, Stefano Lattarini wrote: But it's the package that expects its distributed files to be writable that is assuming too much; if such package wants its expectation

Re: bug#10878: make dist with read-only srcdir generates read-only tarball

2012-02-24 Thread Nick Bowler
this and discovered that there was a source file missing from the distribution that distcheck didn't notice because only maintainer-clean builds failed. Oops!) Cheers, -- Nick Bowler, Elliptic Technologies (http://www.elliptictech.com/)

Re: bug#9579: distcheck does not detect incomplete uninstall as advertised

2011-09-26 Thread Nick Bowler
project, and now distcheck fails when I revert my uninstall-local rule. It's perhaps a bit unfortunate that make infodir='${prefix}/somewhere_else' distcheck no longer works, but I agree that it seems non-trivial to make that work properly. Thanks, -- Nick Bowler, Elliptic Technologies (http

Re: [RFC] Docs: document silent make rules in a new chapter (was: Re: AM_V_GEN - better docs)

2010-11-18 Thread Nick Bowler
not that interesting to see GEN foo.bar five times in a row. -- Nick Bowler, Elliptic Technologies (http://www.elliptictech.com/)

Re: [PATCH] {maint} Document in detail some limitations of aclocal. (was: Re: Strangeness with m4_include and aclocal.)

2010-11-04 Thread Nick Bowler
On 2010-11-04 20:47 +0100, Stefano Lattarini wrote: On Thursday 28 October 2010, Stefano Lattarini wrote: On Wednesday 27 October 2010, Nick Bowler wrote: On 2010-10-23 15:23 +0200, Stefano Lattarini wrote: So I think your first problem is just an aclocal limitation we should resign

Re: [PATCH] {maint} Document in detail some limitations of aclocal.

2010-11-04 Thread Nick Bowler
On 2010-11-04 22:28 +0100, Stefano Lattarini wrote: On Thursday 04 November 2010, Nick Bowler wrote: On 2010-11-04 20:47 +0100, Stefano Lattarini wrote: +AC_DEFUN([MY_DEFUN], [m4_apply([AC_DEFUN], [$1], [$2])]) This is insufficiently quoted, it should