[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Added package xcalib
From: |
Leo Famulari |
Subject: |
Re: [PATCH] Added package xcalib |
Date: |
Fri, 18 Nov 2016 15:10:15 -0500 |
User-agent: |
Mutt/1.7.1 (2016-10-04) |
On Wed, Nov 16, 2016 at 01:37:04AM +0100, Petter wrote:
> I have worked through the issues you brought up, and it should at least be
> better now. Maybe even good.
It was already good, now it's very good :)
> There is no indication of GPL v2 or later in the project though.
Okay.
Pushed as 01278f16a9b with the following two changes:
> + (replace 'install
> + (lambda* (#:key outputs #:allow-other-keys)
> + (let* ((out (assoc-ref outputs "out"))
> + (bin (string-append out "/bin")))
> + (mkdir-p bin)
> + (install-file "xcalib" bin))))
(install-file) does (mkdir-p) itself, so I removed the redundant
(mkdir-p). It's defined in (guix build utils) if you are interested to
see.
> + (add-after 'install 'install-doc
> + (lambda* (#:key outputs #:allow-other-keys)
> + (let ((doc (string-append(assoc-ref outputs "out")
> + "/share/doc/")))
^
I added "xcalib" to the end of the path here -----------------|
So, when a user installs xcalib into their profile, the README will be
at ~/.guix-profile/share/doc/xcalib/README, instead of
~/.guix-profile/share/doc/README.