* [PATCH] scripts: build: Add --file option.
@ 2015-10-26 22:23 David Thompson
2015-10-27 13:29 ` Alex Kost
2015-10-28 9:26 ` Ludovic Courtès
0 siblings, 2 replies; 4+ messages in thread
From: David Thompson @ 2015-10-26 22:23 UTC (permalink / raw)
To: guix-devel
[-- Attachment #1: Type: text/plain, Size: 203 bytes --]
This is to accompany the --install-from-file option I added to 'guix
package' awhile back. Makes it easy for folks to build one-off package
files like the 'guix.scm' files I keep in my personal repos.
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-scripts-build-Add-file-option.patch --]
[-- Type: text/x-patch, Size: 5489 bytes --]
From d3dd1b2d05f17702f7fa6095132db00e2146e702 Mon Sep 17 00:00:00 2001
From: David Thompson <dthompson2@worcester.edu>
Date: Mon, 26 Oct 2015 18:09:28 -0400
Subject: [PATCH] scripts: build: Add --file option.
* guix/scripts/build.scm (show-help): Add help text for --file option.
(%options): Add --file option.
(options/resolve-packages): Handle 'file' options.
* tests/guix-build.sh: Add tests.
* doc/guix.texi ("invoking guix build"): Add doc.
---
doc/guix.texi | 13 +++++++++++++
guix/scripts/build.scm | 45 ++++++++++++++++++++++++++++-----------------
tests/guix-build.sh | 27 +++++++++++++++++++++++++++
3 files changed, 68 insertions(+), 17 deletions(-)
diff --git a/doc/guix.texi b/doc/guix.texi
index 7715b72..abd8de3 100644
--- a/doc/guix.texi
+++ b/doc/guix.texi
@@ -3615,6 +3615,19 @@ The @var{options} may be zero or more of the following:
@table @code
+@item --file=@var{file}
+@itemx -e @var{file}
+
+Build the package or derivation that the code within @var{file}
+evaluates to.
+
+As an example, @var{file} might contain a package definition like this
+(@pxref{Defining Packages}):
+
+@example
+@verbatiminclude package-hello.scm
+@end example
+
@item --expression=@var{expr}
@itemx -e @var{expr}
Build the package or derivation @var{expr} evaluates to.
diff --git a/guix/scripts/build.scm b/guix/scripts/build.scm
index a357cf8..ee7e5b9 100644
--- a/guix/scripts/build.scm
+++ b/guix/scripts/build.scm
@@ -290,6 +290,9 @@ Build the given PACKAGE-OR-DERIVATION and return their output paths.\n"))
(display (_ "
-e, --expression=EXPR build the package or derivation EXPR evaluates to"))
(display (_ "
+ -f, --file=FILE build the package or derivation that the code within
+ FILE evaluates to"))
+ (display (_ "
-S, --source build the packages' source derivations"))
(display (_ "
--sources[=TYPE] build source derivations; TYPE may optionally be one
@@ -359,6 +362,9 @@ must be one of 'package', 'all', or 'transitive'~%")
(option '(#\e "expression") #t #f
(lambda (opt name arg result)
(alist-cons 'expression arg result)))
+ (option '(#\f "file") #t #f
+ (lambda (opt name arg result)
+ (alist-cons 'file arg result)))
(option '(#\n "dry-run") #f #f
(lambda (opt name arg result)
(alist-cons 'dry-run? #t result)))
@@ -422,29 +428,34 @@ packages."
(define system
(or (assoc-ref opts 'system) (%current-system)))
+ (define (object->argument obj)
+ (match obj
+ ((? package? p)
+ `(argument . ,p))
+ ((? procedure? proc)
+ (let ((drv (run-with-store store
+ (mbegin %store-monad
+ (set-guile-for-build (default-guile))
+ (proc))
+ #:system system)))
+ `(argument . ,drv)))
+ ((? gexp? gexp)
+ (let ((drv (run-with-store store
+ (mbegin %store-monad
+ (set-guile-for-build (default-guile))
+ (gexp->derivation "gexp" gexp
+ #:system system)))))
+ `(argument . ,drv)))))
+
(map (match-lambda
(('argument . (? string? spec))
(if (store-path? spec)
`(argument . ,spec)
`(argument . ,(specification->package spec))))
+ (('file . file)
+ (object->argument (load* file (make-user-module '()))))
(('expression . str)
- (match (read/eval str)
- ((? package? p)
- `(argument . ,p))
- ((? procedure? proc)
- (let ((drv (run-with-store store
- (mbegin %store-monad
- (set-guile-for-build (default-guile))
- (proc))
- #:system system)))
- `(argument . ,drv)))
- ((? gexp? gexp)
- (let ((drv (run-with-store store
- (mbegin %store-monad
- (set-guile-for-build (default-guile))
- (gexp->derivation "gexp" gexp
- #:system system)))))
- `(argument . ,drv)))))
+ (object->argument (read/eval str)))
(opt opt))
opts))
diff --git a/tests/guix-build.sh b/tests/guix-build.sh
index a72ce09..f7fb3c5 100644
--- a/tests/guix-build.sh
+++ b/tests/guix-build.sh
@@ -167,6 +167,33 @@ guix build -e "(begin
guix build -e '#~(mkdir #$output)' -d
guix build -e '#~(mkdir #$output)' -d | grep 'gexp\.drv'
+# Building from a package file.
+cat > "$module_dir/package.scm"<<EOF
+(use-modules (gnu))
+(use-package-modules bootstrap)
+
+%bootstrap-guile
+EOF
+guix build --file="$module_dir/package.scm"
+
+# Building from a monadic procedure file.
+cat > "$module_dir/proc.scm"<<EOF
+(use-modules (guix gexp))
+(lambda ()
+ (gexp->derivation "test"
+ (gexp (mkdir (ungexp output)))))
+EOF
+guix build --file="$module_dir/proc.scm" --dry-run
+
+# Building from a gexp file.
+cat > "$module_dir/gexp.scm"<<EOF
+(use-modules (guix gexp))
+
+(gexp (mkdir (ungexp output)))
+EOF
+guix build --file="$module_dir/gexp.scm" -d
+guix build --file="$module_dir/gexp.scm" -d | grep 'gexp\.drv'
+
# Using 'GUIX_BUILD_OPTIONS'.
GUIX_BUILD_OPTIONS="--dry-run"
export GUIX_BUILD_OPTIONS
--
2.5.0
[-- Attachment #3: Type: text/plain, Size: 38 bytes --]
--
David Thompson
GPG Key: 0FF1D807
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] scripts: build: Add --file option.
2015-10-26 22:23 [PATCH] scripts: build: Add --file option David Thompson
@ 2015-10-27 13:29 ` Alex Kost
2015-10-28 9:26 ` Ludovic Courtès
1 sibling, 0 replies; 4+ messages in thread
From: Alex Kost @ 2015-10-27 13:29 UTC (permalink / raw)
To: David Thompson; +Cc: guix-devel
David Thompson (2015-10-27 01:23 +0300) wrote:
> diff --git a/doc/guix.texi b/doc/guix.texi
> index 7715b72..abd8de3 100644
> --- a/doc/guix.texi
> +++ b/doc/guix.texi
> @@ -3615,6 +3615,19 @@ The @var{options} may be zero or more of the following:
>
> @table @code
>
> +@item --file=@var{file}
> +@itemx -e @var{file}
Typo: -f
--
Alex
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] scripts: build: Add --file option.
2015-10-26 22:23 [PATCH] scripts: build: Add --file option David Thompson
2015-10-27 13:29 ` Alex Kost
@ 2015-10-28 9:26 ` Ludovic Courtès
2015-10-28 12:40 ` Thompson, David
1 sibling, 1 reply; 4+ messages in thread
From: Ludovic Courtès @ 2015-10-28 9:26 UTC (permalink / raw)
To: David Thompson; +Cc: guix-devel
David Thompson <dthompson2@worcester.edu> skribis:
> From d3dd1b2d05f17702f7fa6095132db00e2146e702 Mon Sep 17 00:00:00 2001
> From: David Thompson <dthompson2@worcester.edu>
> Date: Mon, 26 Oct 2015 18:09:28 -0400
> Subject: [PATCH] scripts: build: Add --file option.
>
> * guix/scripts/build.scm (show-help): Add help text for --file option.
> (%options): Add --file option.
> (options/resolve-packages): Handle 'file' options.
> * tests/guix-build.sh: Add tests.
> * doc/guix.texi ("invoking guix build"): Add doc.
LGTM, thanks!
Ludo’.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] scripts: build: Add --file option.
2015-10-28 9:26 ` Ludovic Courtès
@ 2015-10-28 12:40 ` Thompson, David
0 siblings, 0 replies; 4+ messages in thread
From: Thompson, David @ 2015-10-28 12:40 UTC (permalink / raw)
To: Ludovic Courtès; +Cc: guix-devel
On Wed, Oct 28, 2015 at 5:26 AM, Ludovic Courtès <ludo@gnu.org> wrote:
> David Thompson <dthompson2@worcester.edu> skribis:
>
>> From d3dd1b2d05f17702f7fa6095132db00e2146e702 Mon Sep 17 00:00:00 2001
>> From: David Thompson <dthompson2@worcester.edu>
>> Date: Mon, 26 Oct 2015 18:09:28 -0400
>> Subject: [PATCH] scripts: build: Add --file option.
>>
>> * guix/scripts/build.scm (show-help): Add help text for --file option.
>> (%options): Add --file option.
>> (options/resolve-packages): Handle 'file' options.
>> * tests/guix-build.sh: Add tests.
>> * doc/guix.texi ("invoking guix build"): Add doc.
Fixed typo and pushed. Thanks Alex and Ludo!
- Dave
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-10-28 12:40 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-10-26 22:23 [PATCH] scripts: build: Add --file option David Thompson
2015-10-27 13:29 ` Alex Kost
2015-10-28 9:26 ` Ludovic Courtès
2015-10-28 12:40 ` Thompson, David
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).