[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 6/7] guix build: Add 'build-package'.
From: |
Ludovic Courtès |
Subject: |
Re: [PATCH 6/7] guix build: Add 'build-package'. |
Date: |
Tue, 22 Sep 2015 17:12:01 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Alex Kost <address@hidden> skribis:
> Ludovic Courtès (2015-09-08 23:10 +0300) wrote:
>
>> Alex Kost <address@hidden> skribis:
[...]
>>> Ludovic Courtès (2015-08-26 14:12 +0300) wrote:
>>>
>>>> Maybe ‘build-package’ could go to (guix packages)?
>>>
>>> ‘build-package’ uses ‘show-…’ procedures from (guix ui) module. I
>>> thought that adding this module to (guix packages) was not desired, or
>>> is it OK?
>>
>> Right, sorry. So maybe leave in (guix build scripts) with a comment
>> explaining that it doesn’t have a better place and is for use by the
>> Emacs UI. (Make sure to use the monadic style as suggested in a
>> previous message.)
>
> So I suppose this comment is not needed anymore (right?) as
Correct.
> 'build-package' will beplaced in (guix scripts), and it looks like a
> rather general procedure. For example, it may be used for something
> like ‘guix build --package-from-file’ suggested by David:
> <http://lists.gnu.org/archive/html/guix-devel/2015-08/msg00107.html>.
‘guix package --install-from-file’ was committed but I don’t think it
actually needs ‘build-package’.
> From ce35de1359ae11a2e73f129db4f7685985f3d041 Mon Sep 17 00:00:00 2001
> From: Alex Kost <address@hidden>
> Date: Thu, 23 Jul 2015 16:16:41 +0300
> Subject: [PATCH] scripts: Add 'build-package'.
> MIME-Version: 1.0
> Content-Type: text/plain; charset=UTF-8
> Content-Transfer-Encoding: 8bit
>
> * guix/scripts/system.scm (maybe-build): Move to ...
> * guix/scripts.scm: ...here.
> (build-package): New procedure.
>
> Co-authored-by: Ludovic Courtès <address@hidden>
OK!
Thanks,
Ludo’.
Re: [PATCH 6/7] guix build: Add 'build-package'., Alex Kost, 2015/09/18