-
Notifications
You must be signed in to change notification settings - Fork 4.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
bookmark-default-file is defined differently in two different places #2594
Comments
I filed this because my bookmarks file was showing up in my recent file list and I was wondering why. It is partially due to the fact I have the spacemacs repo checked out to |
The configuration in |
Only set `bookmark-default-file` in the package configuration. Put the file in the Spacemacs cache directory instead of the Emacs user directory. Also only set `bookmark-save-flag` in the package configuration. This is a breaking change. Those with existing bookmarks file will have to do something equivalent to this: `mv ~/.emacs.d/bookmarks ~/.emacs.d/.cache` and restart Emacs. Fixes syl20bnr#2594
Only set `bookmark-default-file` in the package configuration. Put the file in the Spacemacs cache directory instead of the Emacs user directory. Also only set `bookmark-save-flag` in the package configuration. This is a breaking change. Those with existing bookmarks file will have to do something equivalent to this: `mv ~/.emacs.d/bookmarks ~/.emacs.d/.cache` and restart Emacs. Fixes #2594
bookmark-default-file
and its associatedbookmark-save-flag
are set in two different places, and in the case ofbookmark-default-file
, to two different values. I'm not seeing any bad behavior as such, but I noticed this when checking for something else.Linking to the most recent commit, they are in:
spacemacs/spacemacs/packages.el
Lines 431 to 433 in 88143a3
and
spacemacs/spacemacs/config.el
Lines 269 to 271 in 88143a3
I would offer a pull-request but I'm unsure which is the correct place to set it. I think putting it in the cache would be right.
The text was updated successfully, but these errors were encountered: