union: Do not compare directories upon collision.

* guix/build/union.scm (file=?): Return #f if FILE1 and FILE2 are not
  regular files.  Fixes a bug whereby collisions among directories would
  lead to the invocation of 'file=?' and thus 'call-with-input-file' on
  directories.  Reported by Mark H. Weaver <mhw@netris.org>.
This commit is contained in:
Ludovic Courtès 2014-02-10 00:05:39 +01:00
parent 2de227af4b
commit 6ede17ca69
1 changed files with 21 additions and 16 deletions

View File

@ -1,5 +1,5 @@
;;; GNU Guix --- Functional package management for GNU ;;; GNU Guix --- Functional package management for GNU
;;; Copyright © 2012, 2013 Ludovic Courtès <ludo@gnu.org> ;;; Copyright © 2012, 2013, 2014 Ludovic Courtès <ludo@gnu.org>
;;; ;;;
;;; This file is part of GNU Guix. ;;; This file is part of GNU Guix.
;;; ;;;
@ -103,8 +103,13 @@ single leaf."
(leaf leaf)))) (leaf leaf))))
(define (file=? file1 file2) (define (file=? file1 file2)
"Return #t if the contents of FILE1 and FILE2 are identical, #f otherwise." "Return #t if FILE1 and FILE2 are regular files and their contents are
(and (= (stat:size (stat file1)) (stat:size (stat file2))) identical, #f otherwise."
(let ((st1 (stat file1))
(st2 (stat file2)))
(and (eq? (stat:type st1) 'regular)
(eq? (stat:type st2) 'regular)
(= (stat:size st1) (stat:size st2))
(call-with-input-file file1 (call-with-input-file file1
(lambda (port1) (lambda (port1)
(call-with-input-file file2 (call-with-input-file file2
@ -117,7 +122,7 @@ single leaf."
(n2 (get-bytevector-n! port2 buf2 0 len))) (n2 (get-bytevector-n! port2 buf2 0 len)))
(and (equal? n1 n2) (and (equal? n1 n2)
(or (eof-object? n1) (or (eof-object? n1)
(loop))))))))))) (loop))))))))))))
(define* (union-build output directories (define* (union-build output directories
#:key (log-port (current-error-port))) #:key (log-port (current-error-port)))