[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#43064] [PATCH v2] gexp: computed-file: Prevent mistakenly overridin
From: |
Ludovic Courtès |
Subject: |
[bug#43064] [PATCH v2] gexp: computed-file: Prevent mistakenly overriding default option values. |
Date: |
Mon, 31 Aug 2020 15:34:46 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) |
Hi,
Maxim Cournoyer <maxim.cournoyer@gmail.com> skribis:
> In order to do so, default to an empty options list, and expose options whose
> default values are sensitive directly as keyword arguments.
>
> * guix/gexp.scm (computed-file): Extract the LOCAL-BUILD? parameter from the
> OPTIONS parameter to make it a stand-alone keyword argument. Introduce an
> OPTIONS* binding which is obtained by combining the LOCAL-BUILD? keyword and
> its value with OPTIONS.
>
> Suggested-by: Ludovic Courtès <ludo@gnu.org>
[...]
> (define* (computed-file name gexp
> - #:key guile (options '(#:local-build? #t)))
> + #:key guile (local-build? #t) (options '()))
> "Return an object representing the store item NAME, a file or directory
> -computed by GEXP. OPTIONS is a list of additional arguments to pass
> -to 'gexp->derivation'.
> +computed by GEXP. When LOCAL-BUILD? is #t (the default), it ensures the
> +corresponding derivation is built locally. OPTIONS may be used to pass
> +additional arguments to 'gexp->derivation'.
>
> This is the declarative counterpart of 'gexp->derivation'."
Please update doc/guix.texi as well.
Otherwise LGTM, thanks!
Ludo’.