Re: [PATCH] gnu: Add guile-gnome

[Alex Kost]

David Pirotte (2016-06-09 22:45 +0300) wrote:

>> > > ...
>> > > +        (modify-phases %standard-phases
>> > > +          (add-before 'configure 'pre-configure
>> > > +               (lambda* (#:key outputs #:allow-other-keys)
>> > > +                 (let ((out (assoc-ref outputs "out")))
>> > > +                   (substitute* (find-files "." "^Makefile.in$")
>> > > +                     (("guilemoduledir =.*guile/site" all)
>> > > +                     (string-append all "/2.0")))
>> > > +                  #t))))))
>> > > ...  
>
>> > Just curious: why do you (string-append all "/2.0")? Is this a guix 
>> > guideline
>> > maybe?
>
>> >    IMO, G-wrap, Guile-Cairo and Guile-Gnome should all be installed in the
>> >    directory returned by (%global-site-dir)
>
>> `guilemoduledir =.*guile/site' is saved in the variable `all' and then
>> used in the second half of the substitute*.  I have a similar one I
>> wrote for aria2 in bittorrent.scm
>
> ok, but why "/2.0" is appended to it?

I think the version is not needed, but Ludovic had another opinion.  I
don't remember details but there was some discussion about it:
<http://lists.gnu.org/archive/html/guix-devel/2014-10/msg00408.html>

-- 
Alex