summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLudovic Courtès <ludo@gnu.org>2020-03-22 12:26:05 +0100
committerLudovic Courtès <ludo@gnu.org>2020-03-22 12:42:52 +0100
commit3e6f65be7ae6f895ceb38f9a129c95e08761182b (patch)
treef8472ec6f061a744c61448c3bc635d4566fcb878
parent81c0b52bd6301a7ded157b270097a8074c8f2d50 (diff)
downloadpatches-3e6f65be7ae6f895ceb38f9a129c95e08761182b.tar
patches-3e6f65be7ae6f895ceb38f9a129c95e08761182b.tar.gz
copy: Use 'with-build-handler'.
* guix/scripts/copy.scm (send-to-remote-host): Remove explicit 'show-what-to-build' call. Call 'build-derivations' unconditionally. (guix-copy): Wrap 'with-status-verbosity' in 'with-build-handler'.
-rw-r--r--guix/scripts/copy.scm19
1 files changed, 9 insertions, 10 deletions
diff --git a/guix/scripts/copy.scm b/guix/scripts/copy.scm
index fdb684c6b6..2fa31ecf45 100644
--- a/guix/scripts/copy.scm
+++ b/guix/scripts/copy.scm
@@ -68,12 +68,7 @@ package names, build the underlying packages before sending them."
(ssh-spec->user+host+port target))
((drv items)
(options->derivations+files local opts)))
- (show-what-to-build local drv
- #:use-substitutes? (assoc-ref opts 'substitutes?)
- #:dry-run? (assoc-ref opts 'dry-run?))
-
- (and (or (assoc-ref opts 'dry-run?)
- (build-derivations local drv))
+ (and (build-derivations local drv)
(let* ((session (open-ssh-session host #:user user
#:port (or port 22)))
(sent (send-files local items
@@ -178,7 +173,11 @@ Copy ITEMS to or from the specified host over SSH.\n"))
(target (assoc-ref opts 'destination)))
(with-store store
(set-build-options-from-command-line store opts)
- (with-status-verbosity (assoc-ref opts 'verbosity)
- (cond (target (send-to-remote-host store target opts))
- (source (retrieve-from-remote-host store source opts))
- (else (leave (G_ "use '--to' or '--from'~%")))))))))
+ (with-build-handler (build-notifier #:use-substitutes?
+ (assoc-ref opts 'substitutes?)
+ #:dry-run?
+ (assoc-ref opts 'dry-run?))
+ (with-status-verbosity (assoc-ref opts 'verbosity)
+ (cond (target (send-to-remote-host store target opts))
+ (source (retrieve-from-remote-host store source opts))
+ (else (leave (G_ "use '--to' or '--from'~%"))))))))))