From mboxrd@z Thu Jan 1 00:00:00 1970 From: ludo@gnu.org (Ludovic =?utf-8?Q?Court=C3=A8s?=) Subject: Re: [PATCH 3/4] emacs: Move guix-guile-program to 'guix-config'. Date: Sun, 30 Aug 2015 19:32:01 +0200 Message-ID: <87twrgk7hq.fsf@gnu.org> References: <1438033720-30958-1-git-send-email-mthl@openmailbox.org> <1438033720-30958-4-git-send-email-mthl@openmailbox.org> <87oaivl5ro.fsf@gmail.com> <87a8u4z4hb.fsf@openmailbox.org> <87a8u3sgvw.fsf@gmail.com> <8737z7t6cm.fsf@openmailbox.org> 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]:40775) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZW6Sd-0000yh-Aj for guix-devel@gnu.org; Sun, 30 Aug 2015 13:32:08 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZW6Sa-0003id-3F for guix-devel@gnu.org; Sun, 30 Aug 2015 13:32:07 -0400 In-Reply-To: <8737z7t6cm.fsf@openmailbox.org> (Mathieu Lirzin's message of "Tue, 25 Aug 2015 23:18:17 +0200") 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: Mathieu Lirzin Cc: guix-devel@gnu.org, Alex Kost Mathieu Lirzin skribis: > Alex Kost writes: > >> I would rather add 'guix-guile-program' after both 'guix-=E2=80=A6-direc= tory' >> constants, but it doesn't really matter. > > I'm OK with that. Here is the updated patch. [...] > From 65f22b2dfc5c7e3b799399f5d10a1afba5416f3b Mon Sep 17 00:00:00 2001 > From: Mathieu Lirzin > Date: Mon, 27 Jul 2015 22:55:28 +0200 > Subject: [PATCH 2/3] emacs: Fix guix-guile-program default value. > > * emacs/guix-backend.el: Use guix-config. > (guix-guile-program): Move to ... > * emacs/guix-config.el.in (guix-guile-program): ... here. Use Guile > program file name from compile-time instead of depending on > PATH. (Bug#21127) Please replace =E2=80=9C(Bug#21127)=E2=80=9D with a line like this below th= e subject line (see previous commits for examples): Fixes . Otherwise LGTM. Alex, OK? Ludo=E2=80=99.