Re: [PATCH] gnu: guile: Add "site-ccache" to the search path.

[Ludovic Courtès]

Alex Kost <address@hidden> skribis:

> Ludovic Courtès (2016-06-30 13:31 +0300) wrote:
>
>> Andy Wingo <address@hidden> skribis:
>>
>>> On Wed 29 Jun 2016 15:14, address@hidden (Ludovic Courtès) 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’s fine.  :-)

Thanks!
Ludo’.