From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alex Kost Subject: Re: [PATCH] gnu: guile: Add "site-ccache" to the search path. Date: Thu, 30 Jun 2016 17:18:44 +0300 Message-ID: <87twgaahff.fsf@gmail.com> References: <87por0fmkl.fsf@gmail.com> <87inws87ct.fsf@gnu.org> <87por0m6wr.fsf@igalia.com> <87inwr55on.fsf@gnu.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]:34294) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bIcnr-0002Jv-3B for guix-devel@gnu.org; Thu, 30 Jun 2016 10:18:52 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bIcnl-0005Ej-VV for guix-devel@gnu.org; Thu, 30 Jun 2016 10:18:50 -0400 In-Reply-To: <87inwr55on.fsf@gnu.org> ("Ludovic \=\?utf-8\?Q\?Court\=C3\=A8s\=22'\?\= \=\?utf-8\?Q\?s\?\= message of "Thu, 30 Jun 2016 12:31:20 +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" To: Ludovic =?utf-8?Q?Court=C3=A8s?= Cc: guix-devel@gnu.org 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 :-) --=20 Alex