From mboxrd@z Thu Jan 1 00:00:00 1970 From: ludo@gnu.org (Ludovic =?utf-8?Q?Court=C3=A8s?=) Subject: Re: [PATCH] gnu: guile: Add "site-ccache" to the search path. Date: Thu, 30 Jun 2016 23:14:30 +0200 Message-ID: <877fd6nzux.fsf@gnu.org> References: <87por0fmkl.fsf@gmail.com> <87inws87ct.fsf@gnu.org> <87por0m6wr.fsf@igalia.com> <87inwr55on.fsf@gnu.org> <87twgaahff.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]:51268) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bIjIF-0000cs-I3 for guix-devel@gnu.org; Thu, 30 Jun 2016 17:14:40 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bIjIB-00052T-GW for guix-devel@gnu.org; Thu, 30 Jun 2016 17:14:39 -0400 In-Reply-To: <87twgaahff.fsf@gmail.com> (Alex Kost's message of "Thu, 30 Jun 2016 17:18:44 +0300") 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" To: Alex Kost Cc: guix-devel@gnu.org Alex Kost skribis: > Ludovic Court=C3=A8s (2016-06-30 13:31 +0300) wrote: > >> Andy Wingo skribis: >> >>> On Wed 29 Jun 2016 15:14, ludo@gnu.org (Ludovic Court=C3=A8s) writes: >>> >>>>> --- a/gnu/packages/guile.scm >>>>> +++ b/gnu/packages/guile.scm >>>>> @@ -180,6 +180,7 @@ without requiring the source code to be rewritten= .") >>>>> (search-path-specification >>>>> (variable "GUILE_LOAD_COMPILED_PATH") >>>>> (files '("lib/guile/2.0/ccache" >>>>> + "lib/guile/2.0/site-ccache" >>>>> "share/guile/site/2.0"))))) >>>> >>>> I wonder if it makes sense to keep lib/guile/2.0/ccache since that is >>>> (meant to be) used exclusively by Guile itself, I think. >>>> >>>> Andy, WDYT? >>> >>> Removing Guile's own compiled paths from GUILE_LOAD_COMPILED_PATH sounds >>> like a good idea to me, since these are implicitly included via >>> GUILE_SYSTEM_COMPILED_PATH or whatever that is. >> >> OK. Alex, could you update the patch to not only add 2.x/site-ccache, >> but also remove 2.x/ccache? > > Done, committed to 'core-updates-next' as a08539dd1f. > > ... or did you mean to *send* the updated patch at first? oops :-) That=E2=80=99s fine. :-) Thanks! Ludo=E2=80=99.