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

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Karl Berry <karl <at> freefriends.org>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#67516: closed ([PATCH] aclocal: handle both # and dnl for
 serial number lines)
Date: Tue, 28 Nov 2023 21:14:02 +0000

[Message part 1 (text/plain, inline)]

Your message dated Tue, 28 Nov 2023 14:13:40 -0700
with message-id <202311282113.3ASLDe8S010996 <at> freefriends.org>
and subject line Re: bug#67516: Acknowledgement ([PATCH] aclocal: handle both # and dnl for serial number lines)
has caused the debbugs.gnu.org bug report #67516,
regarding [PATCH] aclocal: handle both # and dnl for serial number lines
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
67516: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=67516
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Karl Berry <karl <at> freefriends.org>
To: ross.burton <at> intel.com
Cc: automake-patches <at> gnu.org
Subject: Re: [PATCH] aclocal: handle both # and dnl for serial number lines
Date: Tue, 28 Nov 2023 14:06:47 -0700

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/msg00001.html

    Instead of only accepting comment lines that start with # for serial
    lines, also accept dnl.
    ...
    -my $serial_line_rx = '^#\s*serial\s+(\S*)';
    +my $serial_line_rx = '^(?:#|dnl)\s*serial\s+(\S*)';

While there's no technical problem with the patch, I'm not sure
supporting dnl is desirable. It seems better for the serial line to
always end up in whatever file is generated, for the sake of debugging
and documentation, instead of being eliminated at m4-time. If there was
a specific case where it was better to eliminate it, please describe ...

Also, because it's been so many years and I don't think it's ever come
up as a request, clearly things go on in any case :).

(Thanks to Bogdan for unearthing this from the archives.) --best, karl.

[Message part 3 (message/rfc822, inline)]

From: Karl Berry <karl <at> freefriends.org>
To: 67516-done <at> debbugs.gnu.org
Subject: Re: bug#67516: Acknowledgement ([PATCH] aclocal: handle both # and
 dnl for serial number lines)
Date: Tue, 28 Nov 2023 14:13:40 -0700

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.