Automatic updates for loaddefs-type files

Automatic updates for loaddefs-type files

[Glenn Morris]

I've added automatic updating of ldefs-boot.el and other versioned files
that contain generated autoload information (cl-loaddefs, etc). This is
using the same cron job on fencepost as the autotools stuff. So there's
no need to commit such changes any more (though it's fine if you want to
do so, eg as part of the change that necessitates the update).

Re: Automatic updates for loaddefs-type files

[Glenn Morris]

Though maybe the ldefs-boot part should only be run once a week? Month?
Otherwise it's going to update every time a file with an autoload gets
changed, which is probably going to be annoying.

Re: Automatic updates for loaddefs-type files

[Eli Zaretskii]

> From: Glenn Morris <rgm@gnu.org>
> Date: Wed, 06 Apr 2011 00:17:34 -0400
> 
> 
> Though maybe the ldefs-boot part should only be run once a week? Month?
> Otherwise it's going to update every time a file with an autoload gets
> changed, which is probably going to be annoying.

Can't we have some move-if-changed based wizardry to commit only if it
changed?

Re: Automatic updates for loaddefs-type files

[Glenn Morris]

Eli Zaretskii wrote:

> Can't we have some move-if-changed based wizardry to commit only if it
> changed?

There's no need for such a thing, since you cannot commit unmodified
files (although, it does have such a thing anyway).

It's just that the main loaddefs.el has timestamps for every file it
scans, so every change to such a file (even if it's just a comment and
does not affect autoloads) causes a change in the generated loaddefs
file.

Re: Automatic updates for loaddefs-type files

[Eli Zaretskii]

> From: Glenn Morris <rgm@gnu.org>
> Cc: emacs-devel@gnu.org
> Date: Wed, 06 Apr 2011 13:17:51 -0400
> 
> It's just that the main loaddefs.el has timestamps for every file it
> scans, so every change to such a file (even if it's just a comment and
> does not affect autoloads) causes a change in the generated loaddefs
> file.

Well, that's why I said "move-if-changed based wizardry": I meant to
make the "if-changed" part ignore the time stamps.  E.g., can Diff be
asked (using the -I switch) to ignore the lines with time stamps?

Re: Automatic updates for loaddefs-type files

[Glenn Morris]

Eli Zaretskii wrote:

> Well, that's why I said "move-if-changed based wizardry": I meant to
> make the "if-changed" part ignore the time stamps.  E.g., can Diff be
> asked (using the -I switch) to ignore the lines with time stamps?

Oh, right. Actually looks like it would be really easy:

diff -qI '^;' loaddefs.el ldefs-boot.el > /dev/null || \
    cp loaddefs.el ldefs-boot.el

Re: Automatic updates for loaddefs-type files

[Glenn Morris]

Stefan Monnier wrote:

>> Though maybe the ldefs-boot part should only be run once a week? Month?
>> Otherwise it's going to update every time a file with an autoload gets
>> changed, which is probably going to be annoying.
>
> I was just going to complain about that, yes ;-)
> I think it's OK to update it "manually" when the need appears.
> After all, and old version is only a problem if it breaks bootstrap.

Well, at the moment I made the ldefs-boot bit once a month only.
No harm in keeping it reasonably up-to-date? But it's easily disabled
if you like, or if this turns out to be annoying.