GNU bug report logs - #62734
Always fully rebuild autoloads in package-generate-autoloads

Previous Next

Package: emacs;

Reported by: Leo Georg Gaskin <leo.gaskin <at> le0.gs>

Date: Sun, 9 Apr 2023 04:12:02 UTC

Severity: normal

Tags: patch

Done: Philip Kaludercic <philipk <at> posteo.net>

Bug is archived. No further changes may be made.

Full log


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

From: Eli Zaretskii <eliz <at> gnu.org>
To: Philip Kaludercic <philipk <at> posteo.net>
Cc: 62734 <at> debbugs.gnu.org, leo.gaskin <at> le0.gs
Subject: Re: bug#62734: Always fully rebuild autoloads in
 package-generate-autoloads
Date: Sat, 29 Apr 2023 15:21:47 +0300
> From: Philip Kaludercic <philipk <at> posteo.net>
> Cc: leo.gaskin <at> le0.gs,  62734 <at> debbugs.gnu.org
> Date: Sat, 29 Apr 2023 11:18:25 +0000
> 
> > "Building" is a strange term when we are talking about a Lisp package.
> 
> How come?  

There's nothing to "build".  Everything is already built.

> >> I think the central issue here is the
> >> 
> >>   (and (not defs) extra-data)
> >> 
> >> check.  Just because we did not find any new definitions to autoload
> >> /and/ EXTRA-DATA is non-nil, does not mean the old contents should be
> >> discarded.  The else-case already does the right thing, so I really do
> >> think that getting rid of the `if' could resolve the issue:
> >
> > What happens if a package has no autoloads?  The doc string says in
> > that case passing EXTRA-DATA will produce OUTPUT-FILE regardless.
> > Does your patch handle that?  (It's hard to tell, given all the
> > whitespace changes in the patch.)
> 
> It would, as the else-case of the if branch I am proposing to eliminate
> would still insert the EXTRA-DATA.

And if EXTRA-DATA is nil, then will we generate an empty OUTPUT file?




This bug report was last modified 2 years and 25 days ago.

Previous Next


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