GNU bug report logs - #48592
[PATCH 0/2] Support plural forms of Author and Maintainer library headers

Previous Next

Package: emacs;

Reported by: Jonas Bernoulli <jonas <at> bernoul.li>

Date: Sat, 22 May 2021 20:26:02 UTC

Severity: normal

Tags: patch

Fixed in version 28.1

Done: Lars Ingebrigtsen <larsi <at> gnus.org>

Bug is archived. No further changes may be made.

Full log


Message #56 received at 48592 <at> debbugs.gnu.org (full text, mbox):

From: Michael Albinus <michael.albinus <at> gmx.de>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 48592 <at> debbugs.gnu.org, Lars Ingebrigtsen <larsi <at> gnus.org>,
 Jonas Bernoulli <jonas <at> bernoul.li>
Subject: Re: bug#48592: [PATCH 0/2] Support plural forms of Author and
 Maintainer library headers
Date: Mon, 24 May 2021 14:10:55 +0200
Eli Zaretskii <eliz <at> gnu.org> writes:

>> Sure adding a note to lm-maintainer technically accomplishes the same,
>> but once one has started using lm-maintainer, then one doesn't
>> periodically go back to see whether a new notes have been added to its
>> doc-string.  But something like this would do the trick of guiding the
>> attention towards the extended functionality and its updated
>> documentation:
>> 
>>   In package-build--desc-from-library:
>> lib/package-build/package-build.el:516:26: Warning: ‘lm-maintainer’ is
>> an
>>       obsolete function (as of 28.1); use ‘lm-maintainers’ instead.
>> 
>> Yes, there is nothing wrong with ignoring all but the first maintainer
>> (except of course, not properly attributing the contributions of the
>> others as they choose to present it), but it seems to me that having
>> to:
>> 
>>   - (lm-maintainer)
>>   + (car (lm-maintainers))
>> 
>> is perfectly acceptable in cases where only "the" maintainer can be
>> mentioned because there is not enough room to display the names of all
>> maintainers.  (So it is still a good idea to list the primus inter
>> pares
>> maintainer first.)
>
>
> I think this warning will be a gratuitous annoyance in enough
> legitimate use cases to make the complaints serious.  If it's okay to
> take the 'car' of a list, then it should also be okay to call a
> function which does just that.  It's not like lm-maintainers returns
> an opaque object.
>
> Again, if the others are fine with the deprecation, I will yield.

Still fine by me, with this obsoletion warning. FWIW.

Best regards, Michael.




This bug report was last modified 1 year and 92 days ago.

Previous Next


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