emenel/doomemacs
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Fix helm macro errors due to load order

Browse source 
Some packages (like evil-collection and dumb-jump) use helm macros, but
Doom cannot ensure helm will be installed by the time these packages are
byte-compiled during installation. In fact, the default load order
prevents this.

Rather than imposing hard load order requirements, we just make sure
helm is higher on the packages list, so it gets installed sooner.
This commit is contained in:
Henrik Lissner 2018-06-05 14:08:29 +02:00
parent 3661462a7a
commit bc4fb8f18e
No known key found for this signature in database
GPG key ID: 5F6C0EA160557395
2 changed files with 14 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

7
modules/feature/evil/packages.el
View file

@ -1,6 +1,13 @@
;; -*- no-byte-compile: t; -*-
;;; feature/evil/packages.el
;; `evil-collection' uses the `with-helm-buffer' macro, but this requires helm
;; be loaded before it is byte-compiled during installation. To ensure this, we
;; declare helm before evil-collection.
(when (featurep! :completion helm)
(depends-on! :completion helm))
;;
(package! evil)
(package! evil-args)
(package! evil-commentary)

7
modules/feature/lookup/packages.el
View file

@ -1,6 +1,13 @@
;; -*- no-byte-compile: t; -*-
;;; feature/lookup/packages.el
;; `dumb-jump' uses the `helm-build-sync-source' macro, but this requires helm be
;; loaded before it is byte-compiled during installation. To ensure this, we
;; declare helm before dumb-jump.
(when (featurep! :completion helm)
(depends-on! :completion helm))
;;
(package! dumb-jump)
(when (featurep! :completion ivy)
(package! ivy-xref))