GNU bug report logs - #36780
26.1; request: savehist should also save log-edit-comment-ring by default

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 36780 in the body.
You can then email your comments to 36780 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

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

From: ndame <emacsuser <at> freemail.hu>
To: "bug-gnu-emacs <at> gnu.org" <bug-gnu-emacs <at> gnu.org>
Subject: 26.1; request:  savehist should also save log-edit-comment-ring by
 default
Date: Wed, 24 Jul 2019 10:59:05 +0200 (CEST)

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

When committing things with vc I often reuse previous commit messages, usually
by editing them a bit. So it's very annoying when I restart emacs and lose the commit
message history.

I now added log-edit-comment-ring to savehist-additional-variables, but it should be 
the default, because minibuffer histories are saved by default, so I'd expect vc commit
messages are also saved, though it does not use the minibuffer, but it is frequently used
input field.

I recommend adding 

  (add-to-list 'savehist-additional-variables 'log-edit-comment-ring)

to log-edit.el, so if savehist is enabled then it is saved as well.

[Message part 2 (text/html, inline)]

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: ndame <emacsuser <at> freemail.hu>
Cc: 36780 <at> debbugs.gnu.org
Subject: Re: bug#36780: 26.1; request:  savehist should also save
 log-edit-comment-ring by default
Date: Fri, 23 Aug 2019 07:12:57 +0200

ndame <emacsuser <at> freemail.hu> writes:

> When committing things with vc I often reuse previous commit messages,
> usually by editing them a bit. So it's very annoying when I restart
> emacs and lose the commit message history.
>
> I now added log-edit-comment-ring to savehist-additional-variables,
> but it should be the default, because minibuffer histories are saved
> by default, so I'd expect vc commit messages are also saved, though it
> does not use the minibuffer, but it is frequently used input field.
>
> I recommend adding 
>
>   (add-to-list 'savehist-additional-variables 'log-edit-comment-ring)
>
> to log-edit.el, so if savehist is enabled then it is saved as well.

`savehist-additional-variables' is explicitly reserved for users to add
stuff to, so I don't think it'd be appropriate for log-edit.el to add
anything to it.

But we could add a new variable, say `savehist-variables', that as a
default value could be

(log-edit-comment-ring)

and anything else the Emacs maintainers think could be saved by default.

Opinions?

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: emacsuser <at> freemail.hu, 36780 <at> debbugs.gnu.org
Subject: Re: bug#36780: 26.1;
 request:  savehist should also save log-edit-comment-ring by default
Date: Fri, 23 Aug 2019 11:57:53 +0300

> From: Lars Ingebrigtsen <larsi <at> gnus.org>
> Date: Fri, 23 Aug 2019 07:12:57 +0200
> Cc: 36780 <at> debbugs.gnu.org
> 
> > I recommend adding 
> >
> >   (add-to-list 'savehist-additional-variables 'log-edit-comment-ring)
> >
> > to log-edit.el, so if savehist is enabled then it is saved as well.
> 
> `savehist-additional-variables' is explicitly reserved for users to add
> stuff to, so I don't think it'd be appropriate for log-edit.el to add
> anything to it.
> 
> But we could add a new variable, say `savehist-variables', that as a
> default value could be
> 
> (log-edit-comment-ring)
> 
> and anything else the Emacs maintainers think could be saved by default.
> 
> Opinions?

I don't see why the solution of customizing
savehist-additional-variables by the user is not the right one.  After
all, we are not talking about minibuffer history here.

The savehist package defines itself as being for "saving the
minibuffer history".  What you suggest makes it a much more general
facility, and we then would need to review many other variables and
decide whether they are candidates for saving (some of them are
already saved by separate features).  Maybe we should do that, but
that's a much broader issue than this particular bug report.

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

From: Drew Adams <drew.adams <at> oracle.com>
To: Eli Zaretskii <eliz <at> gnu.org>, Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: emacsuser <at> freemail.hu, 36780 <at> debbugs.gnu.org
Subject: RE: bug#36780: 26.1; request:  savehist should also save
 log-edit-comment-ring by default
Date: Fri, 23 Aug 2019 11:04:01 -0700 (PDT)

> > `savehist-additional-variables' is explicitly reserved for users to
> > add stuff to, so I don't think it'd be appropriate for log-edit.el
> > to add anything to it.

+1

> I don't see why the solution of customizing
> savehist-additional-variables by the user is not the right one.
> After all, we are not talking about minibuffer history here.

+1

> The savehist package defines itself as being for "saving the
> minibuffer history".  What you suggest makes it a much more general
> facility, and we then would need to review many other variables and
> decide whether they are candidates for saving (some of them are
> already saved by separate features).  Maybe we should do that, but
> that's a much broader issue than this particular bug report.

YAGNI.  Emacs doesn't need to pre-populate a list
of variables to save.  `savehist-additional-variables'
is the right way (for users) to save variables, IMO.

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: emacsuser <at> freemail.hu, 36780 <at> debbugs.gnu.org
Subject: Re: bug#36780: 26.1; request:  savehist should also save
 log-edit-comment-ring by default
Date: Fri, 23 Aug 2019 20:36:51 +0200

Eli Zaretskii <eliz <at> gnu.org> writes:

> I don't see why the solution of customizing
> savehist-additional-variables by the user is not the right one.  After
> all, we are not talking about minibuffer history here.
>
> The savehist package defines itself as being for "saving the
> minibuffer history".

Oh, I thought it was a more general facility.  In that case, I'm closing
this bug report.

> What you suggest makes it a much more general facility, and we then
> would need to review many other variables and decide whether they are
> candidates for saving (some of them are already saved by separate
> features).  Maybe we should do that, but that's a much broader issue
> than this particular bug report.

Yup.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

Previous Next

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