GNU bug report logs -
#41890
28.0.50; [PATCH]: Add bindings for project.el
Previous Next
Reported by: Theodor Thornhill <theo <at> thornhill.no>
Date: Tue, 16 Jun 2020 09:51:02 UTC
Severity: normal
Tags: patch
Found in version 28.0.50
Done: Dmitry Gutov <dgutov <at> yandex.ru>
Bug is archived. No further changes may be made.
Full log
Message #97 received at 41890 <at> debbugs.gnu.org (full text, mbox):
> Cc: 41890 <at> debbugs.gnu.org, theo <at> thornhill.no
> From: Dmitry Gutov <dgutov <at> yandex.ru>
> Date: Thu, 18 Jun 2020 01:23:25 +0300
>
> > I don't see how this is related to the issue at hand. All I'm saying
> > is that a package, including its key bindings, shouldn't be loaded
> > until some of its feature is invoked.
>
> But if we autoload the bindings definition forms, wouldn't that have
> essentially the same effect?
How is this different from bookmark.el?
And if we don't want these key bindings to be available always, we
could have a separate autoloads file for project.el. Some packages do
that already.
> > We could make the keybindings autoloaded without having them defined
> > them when the package loads, couldn't we? By having the define-key on
> > the same line as the autoload cookie, like bookmark.el does.
>
> That would generally be considered problematic because the keymap would
> take effect right after the user updates to the newest version of
> project.el. Because package.el also compiles and evaluates autoloads.
Why is that a problem? A user who updates project.el is most
probably going to use it, right?
And if we do care about this, we could use a separate autoloads file.
This bug report was last modified 4 years and 357 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.