guix-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

08/09: gnu: docker: Honor TESTS? and please guix lint.


From: guix-commits
Subject: 08/09: gnu: docker: Honor TESTS? and please guix lint.
Date: Fri, 24 Jun 2022 01:00:49 -0400 (EDT)

apteryx pushed a commit to branch master
in repository guix.

commit 487070c4254524e1ad00c0c29acaa7b3d71cdda4
Author: Maxim Cournoyer <maxim.cournoyer@gmail.com>
AuthorDate: Thu Jun 23 23:26:08 2022 -0400

    gnu: docker: Honor TESTS? and please guix lint.
    
    * gnu/packages/docker.scm (docker)[phases]{patch-paths}: Split long lines.
    {check}: Honor TESTS?.
    [synopsis]: Do not start synopsis with package name.
---
 gnu/packages/docker.scm | 35 +++++++++++++++++++----------------
 1 file changed, 19 insertions(+), 16 deletions(-)

diff --git a/gnu/packages/docker.scm b/gnu/packages/docker.scm
index 02fb003c36..ae4ee419af 100644
--- a/gnu/packages/docker.scm
+++ b/gnu/packages/docker.scm
@@ -348,7 +348,8 @@ built-in registry server of Docker.")
                  (string-append "DefaultCommand = \""
                                 (search-input-file inputs "/sbin/runc")
                                 "\"\n")))
-              (substitute* 
"vendor/github.com/containerd/containerd/runtime/v1/linux/runtime.go"
+              (substitute* "vendor/github.com/containerd/containerd/\
+runtime/v1/linux/runtime.go"
                 (("defaultRuntime[ \t]*=.*")
                  (string-append "defaultRuntime = \""
                                 (search-input-file inputs "/sbin/runc")
@@ -385,7 +386,8 @@ built-in registry server of Docker.")
                  (string-append "expectedInitPath: \""
                                 (search-input-file inputs "/bin/tini-static")
                                 "\"")))
-              (substitute* 
"vendor/github.com/moby/buildkit/executor/runcexecutor/executor.go"
+              (substitute* "vendor/github.com/moby/buildkit/executor/\
+runcexecutor/executor.go"
                 (("var defaultCommandCandidates = .*")
                  (string-append "var defaultCommandCandidates = []string{\""
                                 (search-input-file inputs "/sbin/runc") 
"\"}")))
@@ -534,19 +536,20 @@ built-in registry server of Docker.")
               ;; dockerd instead.
               (invoke "hack/make.sh" "dynbinary")))
           (replace 'check
-            (lambda _
-              ;; The build process generated a file because the environment
-              ;; variable "AUTO_GOPATH" was set.  Use it.
-              (setenv "GOPATH" (string-append (getcwd) "/.gopath"))
-              ;; ".gopath/src/github.com/docker/docker" is a link to the 
current
-              ;; directory and chdir would canonicalize to that.
-              ;; But go needs to have the uncanonicalized directory name, so
-              ;; store that.
-              (setenv "PWD" (string-append (getcwd)
-                                           
"/.gopath/src/github.com/docker/docker"))
-              (with-directory-excursion ".gopath/src/github.com/docker/docker"
-                (invoke "hack/test/unit"))
-              (setenv "PWD" #f)))
+            (lambda* (#:key tests? #:allow-other-keys)
+              (when tests?
+                ;; The build process generated a file because the environment
+                ;; variable "AUTO_GOPATH" was set.  Use it.
+                (setenv "GOPATH" (string-append (getcwd) "/.gopath"))
+                ;; ".gopath/src/github.com/docker/docker" is a link to the 
current
+                ;; directory and chdir would canonicalize to that.
+                ;; But go needs to have the uncanonicalized directory name, so
+                ;; store that.
+                (setenv "PWD" (string-append
+                               (getcwd) 
"/.gopath/src/github.com/docker/docker"))
+                (with-directory-excursion 
".gopath/src/github.com/docker/docker"
+                  (invoke "hack/test/unit"))
+                (setenv "PWD" #f))))
           (replace 'install
             (lambda* (#:key outputs #:allow-other-keys)
               (let* ((out (assoc-ref outputs "out"))
@@ -580,7 +583,7 @@ built-in registry server of Docker.")
     (native-inputs
      (list eudev ; TODO: Should be propagated by lvm2 (.pc -> .pc)
            go gotestsum pkg-config))
-    (synopsis "Docker container component library, and daemon")
+    (synopsis "Container component library and daemon")
     (description "This package provides a framework to assemble specialized
 container systems.  It includes components for orchestration, image
 management, secret management, configuration management, networking,



reply via email to

[Prev in Thread] Current Thread [Next in Thread]