diff options
author | Ludovic Courtès <ludo@gnu.org> | 2015-04-19 18:49:29 +0200 |
---|---|---|
committer | Ludovic Courtès <ludo@gnu.org> | 2015-04-19 23:34:53 +0200 |
commit | ee8591990fd38ee2860f0ab659b05052b10f14c6 (patch) | |
tree | a6cae89469853393267548e63d6c5bdf82e92c5b | |
parent | 07c0b6e08264f62d0e55ac16be6d313925badfd9 (diff) | |
download | guix-ee8591990fd38ee2860f0ab659b05052b10f14c6.tar guix-ee8591990fd38ee2860f0ab659b05052b10f14c6.tar.gz |
guix package: Fix 'readlink*' implementation.
* guix/scripts/package.scm (readlink*): Fix to handle symlinks with
relative targets. Taken from ld-wrapper2.in.
-rw-r--r-- | guix/scripts/package.scm | 32 |
1 files changed, 25 insertions, 7 deletions
diff --git a/guix/scripts/package.scm b/guix/scripts/package.scm index a42452ae70..1e724b4e19 100644 --- a/guix/scripts/package.scm +++ b/guix/scripts/package.scm @@ -730,13 +730,31 @@ doesn't need it." (define (readlink* file) "Call 'readlink' until the result is not a symlink." - (catch 'system-error - (lambda () - (readlink* (readlink file))) - (lambda args - (if (= EINVAL (system-error-errno args)) - file - (apply throw args))))) + (define %max-symlink-depth 50) + + (let loop ((file file) + (depth 0)) + (define (absolute target) + (if (absolute-file-name? target) + target + (string-append (dirname file) "/" target))) + + (if (>= depth %max-symlink-depth) + file + (call-with-values + (lambda () + (catch 'system-error + (lambda () + (values #t (readlink file))) + (lambda args + (let ((errno (system-error-errno args))) + (if (or (= errno EINVAL)) + (values #f file) + (apply throw args)))))) + (lambda (success? target) + (if success? + (loop (absolute target) (+ depth 1)) + file)))))) ;;; |