emacs-bug-tracker
[Top][All Lists]
Advanced

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

[debbugs-tracker] bug#26075: closed ([PATCH -v3 1/2] build: union: Add c


From: GNU bug Tracking System
Subject: [debbugs-tracker] bug#26075: closed ([PATCH -v3 1/2] build: union: Add create-all-directories? parameter to union-build)
Date: Sun, 26 Mar 2017 10:56:02 +0000

Your message dated Sun, 26 Mar 2017 12:55:22 +0200
with message-id <address@hidden>
and subject line Re: bug#26075: [PATCH -v3 1/2] build: union: Add 
create-all-directories? parameter to union-build
has caused the debbugs.gnu.org bug report #26075,
regarding [PATCH -v3 1/2] build: union: Add create-all-directories? parameter 
to union-build
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
26075: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=26075
GNU Bug Tracking System
Contact address@hidden with problems
--- Begin Message --- Subject: [PATCH -v3 1/2] build: union: Add create-all-directories? parameter to union-build Date: Sun, 12 Mar 2017 19:53:58 +0800
* guix/build/union.scm (union-build): Add create-all-directories? keyword
  parameter.
---
 guix/build/union.scm | 16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

diff --git a/guix/build/union.scm b/guix/build/union.scm
index 6640b5652..b852bc66e 100644
--- a/guix/build/union.scm
+++ b/guix/build/union.scm
@@ -73,9 +73,12 @@ identical, #f otherwise."
                                   (loop)))))))))))))
 
 (define* (union-build output inputs
-                      #:key (log-port (current-error-port)))
-  "Build in the OUTPUT directory a symlink tree that is the union of all
-the INPUTS."
+                      #:key (log-port (current-error-port))
+                      (create-all-directories? #f))
+  "Build in the OUTPUT directory a symlink tree that is the union of all the
+INPUTS.  As a special case, if CREATE-ALL-DIRECTORIES?, creates the
+subdirectories in the output directory to make sure the caller can modify them
+later."
 
   (define (symlink* input output)
     (format log-port "`~a' ~~> `~a'~%" input output)
@@ -104,8 +107,11 @@ the INPUTS."
   (define (union output inputs)
     (match inputs
       ((input)
-       ;; There's only one input, so just make a link.
-       (symlink* input output))
+       ;; There's only one input, so just make a link unless
+       ;; create-all-directories?.
+       (if (and create-all-directories? (file-is-directory? input))
+           (union-of-directories output inputs)
+           (symlink* input output)))
       (_
        (call-with-values (lambda () (partition file-is-directory? inputs))
          (match-lambda*
-- 
2.12.0





--- End Message ---
--- Begin Message --- Subject: Re: bug#26075: [PATCH -v3 1/2] build: union: Add create-all-directories? parameter to union-build Date: Sun, 26 Mar 2017 12:55:22 +0200 User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux)
Hi!

I’ve pushed it with a copyright line for you, and with a unit test for
‘union-build’ with #:create-all-directories? #t:

  
https://git.savannah.gnu.org/cgit/guix.git/commit/?id=addce19e2d38a197f5ea10eefb5f3cd25c3a52e7
  
https://git.savannah.gnu.org/cgit/guix.git/commit/?id=0a5ce0d1df3befa2c4e018e84da3bd66c9eac48d

Thank you!

Ludo’.


--- End Message ---

reply via email to

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