search

purcell / emacs.d

An Emacs configuration bundle with batteries included
BSD 2-Clause "Simplified" License
6.87k stars 2.05k forks source link

Relocate init-preload-local #863

Closed AlMaVizca closed 3 months ago

AlMaVizca commented 3 months ago

Thanks for your effort. Your Emacs dots facilitate my journey.

I was using straight to manage some packages, and recently decided to give it a try to elpaca. On their installer code they mention it has to be loaded early into the init.el. I've found init-preload-local.el and the issue #199, but it's being loaded after 'init-elpa' which makes elpaca to throw a warning. I found that moving: https://github.com/purcell/emacs.d/blob/adf0e27c95cae4e19e2fee174170bac2ff7e83d3/init.el#L56-L57 In between of: https://github.com/purcell/emacs.d/blob/adf0e27c95cae4e19e2fee174170bac2ff7e83d3/init.el#L19-L21

Will reduce the warnings and keep the dots working as usual, at least with the default configs. Would this change create any trouble?

purcell commented 3 months ago

I think that would break configurations for people who expect the package system to have been initialised when init-preload-local is loaded. But you could always modify init.el directly in your fork — I don't intend these init*-local.el extension points to cover all potential use cases anyway.