Fix custom theme cannot be auto installed

[goofansu]

According to README, I put (setq prelude-theme 'solarized-dark) in personal/preload/override.el. But it reports error as the package has not been installed.

I want to make the package auto installed while not modifying the Prelude code, so I suppose to move the (load-theme prelude-theme t) to init.el and after the personal configs. Then I can use (prelude-require-package 'solarized-dark) to auto install the theme package.

[CNBorn]

Second this one. This actually makes theme changing possible.

[CNBorn]

@goofansu do you mind to rebase with the upstream?

[toctan]

@goofansu Nope. That's not the way to do it. The personal directory is loaded after prelude but the personal/preload directory gets loaded before prelude. So you can put this line (prelude-require-package 'solarized-dark) anywhere in personal/preload, then it would work perfectly.

See https://github.com/bbatsov/prelude#preloading-personal-config

[CNBorn]

@toctan https://github.com/bbatsov/prelude/blob/master/init.el#L101 prelude-require-package are in prelude-package.el, which is loaded after preload scripts. There will be an error when you do prelude-require-package in preload script

[goofansu]

@CNBorn Thanks 👍

[toctan]

@CNBorn @goofansu Oh my bad, but I still do not think it's a very good idea to move the theme loading code out of prelude-ui.el.

[goofansu]

@toctan This method is tricky. I cannot find other way to achieve the goal.

[toctan]

@goofansu You can circumvent this by (require 'prelude-packages) before (prelude-require-package 'solarized-dark) in your preloaded configuration file.

[CNBorn]

@goofansu You can circumvent this by (require 'prelude-packages) before (prelude-require-package 'solarized-dark) in your preloaded configuration file.

Tested, that's seemingly a good way to do this.

[goofansu]

@toctan Thanks, closed.