Alex Kost <alez...@gmail.com> writes:

> As I wrote in another message, I would leave 'guix-load-path' in
> "guix-backend.el" with 'guix-emacs-ui-dir' as the value, and just remove
> it from "guix-init.el".  Do you have other ideas?

You are the elisp expert :). I trust your judgement.

Thank you very much for the review!

--
Mathieu Lirzin

>From d73122381955ea25d05eff9b54f5f72ed5b37d87 Mon Sep 17 00:00:00 2001
From: Mathieu Lirzin <m...@openmailbox.org>
Date: Mon, 27 Jul 2015 23:20:09 +0200
Subject: [PATCH 3/3] emacs: Use only one guix-load-path variable.

emacs/guix-init.el (guix-load-path): Move to ...
emacs/guix-backend.el (guix-load-path): ... Here. Redefine it.
---
 emacs/guix-backend.el | 4 +---
 emacs/guix-init.el    | 4 ----
 2 files changed, 1 insertion(+), 7 deletions(-)

diff --git a/emacs/guix-backend.el b/emacs/guix-backend.el
index 3fafae4..7db1daa 100644
--- a/emacs/guix-backend.el
+++ b/emacs/guix-backend.el
@@ -55,9 +55,7 @@
 (require 'guix-config)
 (require 'guix-emacs)
 
-(defvar guix-load-path
-  (file-name-directory (or load-file-name
-                           (locate-library "guix")))
+(defvar guix-load-path guix-emacs-interface-directory
   "Directory with scheme files for \"guix.el\" package.")
 
 (defvar guix-helper-file
diff --git a/emacs/guix-init.el b/emacs/guix-init.el
index 6ea6fcd..3a727c7 100644
--- a/emacs/guix-init.el
+++ b/emacs/guix-init.el
@@ -1,9 +1,5 @@
-(require 'guix-config)
 (require 'guix-autoloads)
 
-(defvar guix-load-path guix-emacs-interface-directory
-  "Directory with scheme files for \"guix.el\" package.")
-
 (defcustom guix-package-enable-at-startup t
   "If non-nil, activate Emacs packages installed in a user profile.
 Set this variable to nil before requiring `guix-init' file to
-- 
2.1.4

Reply via email to