[Top][All Lists]

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

12/15: packages: 'specification->package+output' distinguishes "no outpu

From: guix-commits
Subject: 12/15: packages: 'specification->package+output' distinguishes "no output specified".
Date: Thu, 27 Jun 2019 05:15:46 -0400 (EDT)

civodul pushed a commit to branch master
in repository guix.

commit 066eeae1a197b1926c7ffc42919d684686f56fdb
Author: Ludovic Court├Ęs <address@hidden>
Date:   Sun Jun 23 18:28:45 2019 +0200

    packages: 'specification->package+output' distinguishes "no output 
    Until now the caller couldn't tell the different between a spec like
    "foo:out" and one like "foo".  This change allows users to distinguish
    between these two cases.
    * gnu/packages.scm (specification->package+output): Disable output
    membership test when OUTPUT = #f and SUB-DRV = #f.
    * tests/packages.scm ("specification->package+output")
    ("specification->package+output invalid output")
    ("specification->package+output no default output")
    ("specification->package+output invalid output, no default"): New tests.
 gnu/packages.scm   |  8 ++++++--
 tests/packages.scm | 32 ++++++++++++++++++++++++++++++++
 2 files changed, 38 insertions(+), 2 deletions(-)

diff --git a/gnu/packages.scm b/gnu/packages.scm
index 4839057..acb247e 100644
--- a/gnu/packages.scm
+++ b/gnu/packages.scm
@@ -534,14 +534,18 @@ optionally contain a version number and an output name, 
as in these examples:
 If SPEC does not specify a version number, return the preferred newest
-version; if SPEC does not specify an output, return OUTPUT."
+version; if SPEC does not specify an output, return OUTPUT.
+When OUTPUT is false and SPEC does not specify any output, return #f as the
   (let-values (((name version sub-drv)
                 (package-specification->name+version+output spec output)))
     (match (%find-package spec name version)
        (values #f #f))
-       (if (member sub-drv (package-outputs package))
+       (if (or (and (not output) (not sub-drv))
+               (member sub-drv (package-outputs package)))
            (values package sub-drv)
            (leave (G_ "package `~a' lacks output `~a'~%")
                   (package-full-name package)
diff --git a/tests/packages.scm b/tests/packages.scm
index 613b2f1..836d446 100644
--- a/tests/packages.scm
+++ b/tests/packages.scm
@@ -1227,6 +1227,38 @@
     (lambda (key . args)
+(test-equal "specification->package+output"
+  `((,coreutils "out") (,coreutils "debug"))
+  (list (call-with-values (lambda ()
+                            (specification->package+output "coreutils"))
+          list)
+        (call-with-values (lambda ()
+                            (specification->package+output "coreutils:debug"))
+          list)))
+(test-equal "specification->package+output invalid output"
+  'error
+  (catch 'quit
+    (lambda ()
+      (specification->package+output "coreutils:does-not-exist"))
+    (lambda _
+      'error)))
+(test-equal "specification->package+output no default output"
+  `(,coreutils #f)
+  (call-with-values
+    (lambda ()
+      (specification->package+output "coreutils" #f))
+    list))
+(test-equal "specification->package+output invalid output, no default"
+  'error
+  (catch 'quit
+    (lambda ()
+      (specification->package+output "coreutils:does-not-exist" #f))
+    (lambda _
+      'error)))
 (test-equal "find-package-locations"
   (map (lambda (package)
          (cons (package-version package)

reply via email to

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