From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alex Kost Subject: Re: [PATCH] gnu: geiser: Symlink Guile modules to the right 'guilesitedir'. Date: Sun, 21 Jun 2015 17:05:15 +0300 Message-ID: <877fqxyx5w.fsf@gmail.com> References: <1434889965-10697-1-git-send-email-iyzsong@gmail.com> <87ioahtcx6.fsf@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:35630) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Z6fsD-0006Os-5i for guix-devel@gnu.org; Sun, 21 Jun 2015 10:05:26 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Z6fs1-0004uc-UQ for guix-devel@gnu.org; Sun, 21 Jun 2015 10:05:25 -0400 Received: from mail-la0-x22d.google.com ([2a00:1450:4010:c03::22d]:33184) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Z6fs1-0004to-NG for guix-devel@gnu.org; Sun, 21 Jun 2015 10:05:13 -0400 Received: by laka10 with SMTP id a10so96544786lak.0 for ; Sun, 21 Jun 2015 07:05:13 -0700 (PDT) In-Reply-To: <87ioahtcx6.fsf@gmail.com> (=?utf-8?B?IuWui+aWh+atpiIncw==?= message of "Sun, 21 Jun 2015 21:21:25 +0800") List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org To: =?utf-8?B?5a6L5paH5q2m?= Cc: guix-devel@gnu.org =E5=AE=8B=E6=96=87=E6=AD=A6 (2015-06-21 16:21 +0300) wrote: > The previous patch was intend to fix a issue that manually spawned Guile > doesn't know the geiser module. > > When discuss this issue in IRC with alezost, I suddenly notice that add > a simple '(require 'geiser-install)' to my init.el make it work. > > Here is the new patch for '/etc/emacs/site-start.el' (not tested yet!): > > > From d26e6299c9d468ff4a745a7efe9ea4f1a5d7c63a Mon Sep 17 00:00:00 2001 > From: =3D?UTF-8?q?=3DE5=3DAE=3D8B=3DE6=3D96=3D87=3DE6=3DAD=3DA6?=3D > Date: Sun, 21 Jun 2015 21:09:42 +0800 > Subject: [PATCH] system: emacs-site-file: Use 'geiser-install to load gei= ser. > > Suggested by Alex Kost . > > * gnu/system.scm (emacs-site-file): Use 'geiser-install to load geiser > instead of setting 'geiser-guile-load-path' manually. > --- > gnu/system.scm | 8 ++------ > 1 file changed, 2 insertions(+), 6 deletions(-) > > diff --git a/gnu/system.scm b/gnu/system.scm > index 565d6c1..82b7fbc 100644 > --- a/gnu/system.scm > +++ b/gnu/system.scm > @@ -391,12 +391,8 @@ settings for 'guix.el' to work out-of-the-box." > ;; Attempt to load guix.el. > (require 'guix-init nil t) >=20=20 > - (when (require 'geiser-guile nil t) > - ;; Make sure Geiser's Scheme modules are in Guile's s= earch > - ;; path. > - (add-to-list > - 'geiser-guile-load-path > - "/run/current-system/profile/share/geiser/guile"))))) > + ;; Attempt to load geiser. > + (require 'geiser-install nil t)))) >=20=20 > (define (emacs-site-directory) > "Return the Emacs site directory, aka. /etc/emacs." I didn't test it but I think it should work, as according to the geiser manual (info "(geiser) From the source's mouth"), requiring 'geiser-install' is the proper way to set up Geiser (that's why we symlink "geiser-install.el" by "geiser-autoloads.el" in a geiser package recipe). --=20 Alex