GNU bug report logs - #74799
Support setting `save-place-file' after running `save-place-mode'

Previous Next

To reply to this bug, email your comments to 74799 AT debbugs.gnu.org.

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

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

From: Hong Xu <hong <at> topbug.net>
To: bug-gnu-emacs <at> gnu.org
Subject: [PATCH] Clarify that save-place-file must be set before
 save-place-mode starts
Date: Wed, 11 Dec 2024 12:00:57 -0800

---
 lisp/saveplace.el | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/lisp/saveplace.el b/lisp/saveplace.el
index 012e305f7f45..ea67c4567b63 100644
--- a/lisp/saveplace.el
+++ b/lisp/saveplace.el
@@ -63,7 +63,8 @@ save-place-alist
 This alist is saved between Emacs sessions.")
 
 (defcustom save-place-file (locate-user-emacs-file "places" ".emacs-places")
-  "Name of the file that records `save-place-alist' value."
+  "Name of the file that records `save-place-alist' value.
+Must be set before starting save-place-mode."
   :version "24.4"                       ; added locate-user-emacs-file
   :type 'file)
 
-- 
2.45.2

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: Hong Xu <hong <at> topbug.net>
Cc: 74799 <at> debbugs.gnu.org
Subject: Re: bug#74799: [PATCH] Clarify that save-place-file must be set before
 save-place-mode starts
Date: Wed, 11 Dec 2024 22:29:16 +0200

> From: Hong Xu <hong <at> topbug.net>
> Date: Wed, 11 Dec 2024 12:00:57 -0800
> 
> ---
>  lisp/saveplace.el | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/lisp/saveplace.el b/lisp/saveplace.el
> index 012e305f7f45..ea67c4567b63 100644
> --- a/lisp/saveplace.el
> +++ b/lisp/saveplace.el
> @@ -63,7 +63,8 @@ save-place-alist
>  This alist is saved between Emacs sessions.")
>  
>  (defcustom save-place-file (locate-user-emacs-file "places" ".emacs-places")
> -  "Name of the file that records `save-place-alist' value."
> +  "Name of the file that records `save-place-alist' value.
> +Must be set before starting save-place-mode."
>    :version "24.4"                       ; added locate-user-emacs-file
>    :type 'file)

Thanks, but how about removing the limitation instead?

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

From: Hong Xu <hong <at> topbug.net>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 74799 <at> debbugs.gnu.org
Subject: Re: bug#74799: [PATCH] Clarify that save-place-file must be set
 before save-place-mode starts
Date: Wed, 11 Dec 2024 12:41:32 -0800

On 2024-12-11 Wed 12:29 GMT-08, Eli Zaretskii <eliz <at> gnu.org> wrote:

>> From: Hong Xu <hong <at> topbug.net>
>> Date: Wed, 11 Dec 2024 12:00:57 -0800
>> 
>> ---
>>  lisp/saveplace.el | 3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>> 
>> diff --git a/lisp/saveplace.el b/lisp/saveplace.el
>> index 012e305f7f45..ea67c4567b63 100644
>> --- a/lisp/saveplace.el
>> +++ b/lisp/saveplace.el
>> @@ -63,7 +63,8 @@ save-place-alist
>>  This alist is saved between Emacs sessions.")
>>  
>>  (defcustom save-place-file (locate-user-emacs-file "places" ".emacs-places")
>> -  "Name of the file that records `save-place-alist' value."
>> +  "Name of the file that records `save-place-alist' value.
>> +Must be set before starting save-place-mode."
>>    :version "24.4"                       ; added locate-user-emacs-file
>>    :type 'file)
>
> Thanks, but how about removing the limitation instead?

I'm not sure how to do this. It seems to me the limitation didn't exist
in Emacs 29 but suddenly I have to set save-place-file before calling
save-place-mode. I thought this was intended but maybe this is a
regression?

(It's difficult for me to try out Emacs 29 now to 100% confirm, sorry.)

-- 
Hong

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

From: Ship Mints <shipmints <at> gmail.com>
To: Hong Xu <hong <at> topbug.net>
Cc: 74799 <at> debbugs.gnu.org, Eli Zaretskii <eliz <at> gnu.org>
Subject: Re: bug#74799: [PATCH] Clarify that save-place-file must be set
 before save-place-mode starts
Date: Wed, 11 Dec 2024 15:54:53 -0500

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

Using saveplace myself, and having added functionality such as a save timer
for which I'll contribute a patch for at some point, I've read the code.

If one changes the value of save-place-file from its default to another
value after the file has first been read, you'll have to manually reset
save-place-loaded to nil to force a reload. The defcustom
for save-place-file could be patched to provide a setter which would do
that, if that helps. I don't see this issue as I defer save-place-mode
until after init when my overrides have already taken effect.

On Wed, Dec 11, 2024 at 3:42 PM Hong Xu <hong <at> topbug.net> wrote:

> On 2024-12-11 Wed 12:29 GMT-08, Eli Zaretskii <eliz <at> gnu.org> wrote:
>
> >> From: Hong Xu <hong <at> topbug.net>
> >> Date: Wed, 11 Dec 2024 12:00:57 -0800
> >>
> >> ---
> >>  lisp/saveplace.el | 3 ++-
> >>  1 file changed, 2 insertions(+), 1 deletion(-)
> >>
> >> diff --git a/lisp/saveplace.el b/lisp/saveplace.el
> >> index 012e305f7f45..ea67c4567b63 100644
> >> --- a/lisp/saveplace.el
> >> +++ b/lisp/saveplace.el
> >> @@ -63,7 +63,8 @@ save-place-alist
> >>  This alist is saved between Emacs sessions.")
> >>
> >>  (defcustom save-place-file (locate-user-emacs-file "places"
> ".emacs-places")
> >> -  "Name of the file that records `save-place-alist' value."
> >> +  "Name of the file that records `save-place-alist' value.
> >> +Must be set before starting save-place-mode."
> >>    :version "24.4"                       ; added locate-user-emacs-file
> >>    :type 'file)
> >
> > Thanks, but how about removing the limitation instead?
>
> I'm not sure how to do this. It seems to me the limitation didn't exist
> in Emacs 29 but suddenly I have to set save-place-file before calling
> save-place-mode. I thought this was intended but maybe this is a
> regression?
>
> (It's difficult for me to try out Emacs 29 now to 100% confirm, sorry.)
>
> --
> Hong
>
>
>
>

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

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: Hong Xu <hong <at> topbug.net>
Cc: 74799 <at> debbugs.gnu.org
Subject: Re: bug#74799: [PATCH] Clarify that save-place-file must be set
 before save-place-mode starts
Date: Wed, 11 Dec 2024 22:58:38 +0200

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: Hong Xu <hong <at> topbug.net>
Cc: 74799 <at> debbugs.gnu.org
Subject: Re: bug#74799: [PATCH] Clarify that save-place-file must be set
 before save-place-mode starts
Date: Wed, 11 Dec 2024 23:01:08 +0200

> From: Hong Xu <hong <at> topbug.net>
> Cc: 74799 <at> debbugs.gnu.org
> Date: Wed, 11 Dec 2024 12:41:32 -0800
> 
> On 2024-12-11 Wed 12:29 GMT-08, Eli Zaretskii <eliz <at> gnu.org> wrote:
> 
> >> -  "Name of the file that records `save-place-alist' value."
> >> +  "Name of the file that records `save-place-alist' value.
> >> +Must be set before starting save-place-mode."
> >>    :version "24.4"                       ; added locate-user-emacs-file
> >>    :type 'file)
> >
> > Thanks, but how about removing the limitation instead?
> 
> I'm not sure how to do this. It seems to me the limitation didn't exist
> in Emacs 29 but suddenly I have to set save-place-file before calling
> save-place-mode. I thought this was intended but maybe this is a
> regression?
> 
> (It's difficult for me to try out Emacs 29 now to 100% confirm, sorry.)

I'd start by comparing saveplace.el in Emacs 28 and Emacs 29.

You could also try loading saveplace.el from Emacs 28 into an Emacs 29
session, to see if the problems you have go away.

Btw, what actually happens if you modify the file name after you turn
on save-place-mode? what doesn't work?

Previous Next

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