-
Notifications
You must be signed in to change notification settings - Fork 12
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
XDG support? #51
Comments
I hit some bad behavior due to this, reported (incorrectly) to ocaml/opam#6346 BackgroundI am a doom emacs user, setting up a new PC. Doom has moved to using XDG-compatible locations its default configuration source, which means keeping all configuration data in ProblemThe next time I restarted emacs, I was suddenly back into the default emacs configuration. As per the docs, this is because the
From the end users perspective, it looks like opam setup has hijacked the emacs configuration! This is easy to diagnose and fix if you happen to know the minutia, but could easily open up a rabbit hole hiding a red herring. Suggested solutionThe ideal behavior, I think, would be to search for existing emacs configuration data starting from the most marginal location (meaning, the one which is most easily overridden), which would mean first checking whether an existing config is located in the XDG-compatible location, and deferring to that location if found. A complication to consider: a user could have ended up in a situation where they are actually using config data located in When in doubt, it would be better if opam prompted for input from the user rather than making a change that can override existing configurations. |
Maybe we should just use In the meantime, the fix to make use of
|
Recent Emacs will also check
~/.config/emacs/init.el
andopam user-setup
should emulate the main logic used by Emacs. That is, perhaps this function needs to be changed:opam-user-setup/emacs/emacs.ml
Lines 101 to 108 in 8208197
It might also be useful to have an option to specify the location of configuration files that will somehow work with
--editors
.The text was updated successfully, but these errors were encountered: