[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] gnu: Add guile-gnome
From: |
David Pirotte |
Subject: |
Re: [PATCH] gnu: Add guile-gnome |
Date: |
Sat, 11 Jun 2016 18:54:13 -0300 |
Hi Alex,
> >> > > ...
> >> > > + (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>
Thanks for the pointer. FWIW, I agree with Mark
(a) (%global-site-dir) as always been part of gule's search paths for
modules, and that should not be 'altered' for guix, imo;
(b) it seems to me there has been a confuson, in this thread, in between
those of you talking about packages that also need to install things in
$prefix/bin and 'natively' do not distingish themselve from other
version(s)
[1] and those of you talking packages that are compatible with several
version of guile [the one listed in their configure.ac file] and these
should be installed in (%global-site-dir), imo.
But then maybe I'm missing something.
Cheers,
David
[1] the thread sites qt 4 and 5, but guile does that too by the way, even if
you specify --program-suffix, it installs itself and its scripts, in
$prefix/bin, without the suffix
pgpBqT6O3Vs1q.pgp
Description: OpenPGP digital signature