[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] gnu: Fix load-extension path in packaging of guile-ncurses.
From: |
Mark H Weaver |
Subject: |
Re: [PATCH] gnu: Fix load-extension path in packaging of guile-ncurses. |
Date: |
Tue, 20 Dec 2016 02:17:42 -0500 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Hi John,
John Darrington <address@hidden> writes:
> * gnu/packages/guile.scm (guile-ncurses) [arguments]: Install shared object
> before
> attempting to build the package. Patch load-extension path before building
> instead
> of after.
The first sentence above is mistaken or misleading:
"Install shared object before attempting to build the package."
Of course this is not possible, and it's not what happens after applying
this commit.
> + (add-before 'build 'fix-libguile-ncurses-file-name
> (lambda* (#:key outputs #:allow-other-keys)
> - [...]
> + (and (zero? (system* "make" "install"
> + "-C" "src/ncurses"
> + "-j" (number->string
> + (parallel-job-count))))
> + (let* ((out (assoc-ref outputs "out"))
> + (dir "src/ncurses")
> + (files (find-files dir ".scm")))
> + (substitute* files
> + (("\"libguile-ncurses\"")
> + (format #f "\"~a/lib/guile/2.0/libguile-ncurses\""
> + out)))
> + #t)))))))
By running "make install -C src/ncurses [...]", you are in fact causing
most if not all of the package to be built and installed right here in
the 'fix-libguile-ncurses-file-name' phase. This is no good.
Can you help me understand what you're trying to do here, and why?
Thanks,
Mark
- [no subject], John Darrington, 2016/12/19
- [PATCH] gnu: Fix load-extension path in packaging of guile-ncurses., John Darrington, 2016/12/19
- Re: [PATCH] gnu: Fix load-extension path in packaging of guile-ncurses., Ludovic Courtès, 2016/12/19
- Re: [PATCH] gnu: Fix load-extension path in packaging of guile-ncurses.,
Mark H Weaver <=
- Re: [PATCH] gnu: Fix load-extension path in packaging of guile-ncurses., John Darrington, 2016/12/20
- Re: [PATCH] gnu: Fix load-extension path in packaging of guile-ncurses., John Darrington, 2016/12/20
- Re: [PATCH] gnu: Fix load-extension path in packaging of guile-ncurses., Danny Milosavljevic, 2016/12/21
- Re: [PATCH] gnu: Fix load-extension path in packaging of guile-ncurses., John Darrington, 2016/12/21
- Re: [PATCH] gnu: Fix load-extension path in packaging of guile-ncurses., Tobias Geerinckx-Rice, 2016/12/22
- Re: [PATCH] gnu: Fix load-extension path in packaging of guile-ncurses., John Darrington, 2016/12/22
- Re: [PATCH] gnu: Fix load-extension path in packaging of guile-ncurses., Mark H Weaver, 2016/12/24
- Re: [PATCH] gnu: Fix load-extension path in packaging of guile-ncurses., John Darrington, 2016/12/24