mdebray/tvl-depot
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Remove opam-install variable and assertion

Browse source 
The `prelude/assert` for the existence of the `opam-install` directory was
failing.

I believe this assertion would have been failing sooner, but a bug in my
initialization was preventing Emacs from evaluating `wpc-ocaml.el`. It seems
that I removed whatever was jamming the initialization and as such, I uncovered
some more bugs.

Let this serve as a reminder that just because it hasn't bitten you yet, doesn't
mean that your software doesn't have a bug.
This commit is contained in:
William Carroll 2020-01-20 21:47:16 +00:00
parent 59be1ee981
commit b843e0f8f6
1 changed files with 1 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
configs/shared/.emacs.d/wpc/packages/wpc-ocaml.el
View file

@ -22,17 +22,10 @@
(prelude/assert
(prelude/executable-exists? "opam"))
(defvar opam-installs "~/.opam/4.08.0/share/emacs/site-lisp"
"Path to the Ocaml PAckage Manager installations.")
(defvar opam-user-setup "~/.emacs.d/opam-user-setup.el"
"File for the OPAM Emacs integration.")
(prelude/assert
(f-file? opam-user-setup))
(prelude/assert
(f-dir? opam-installs))
(prelude/assert (f-file? opam-user-setup))
;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
;; Configuration
@ -45,7 +38,6 @@
;; ocamlformat
(require 'opam-user-setup "~/.emacs.d/opam-user-setup.el")
(require 'ocamlformat)
(add-to-list 'load-path opam-installs)
(provide 'wpc-ocaml)
;;; wpc-ocaml.el ends here