guix-commits
[Top][All Lists]
Advanced

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

01/02: build: ruby: Avoid long build directory names.


From: David Thompson
Subject: 01/02: build: ruby: Avoid long build directory names.
Date: Mon, 07 Sep 2015 13:18:18 +0000

davexunit pushed a commit to branch master
in repository guix.

commit b500e7182d3278cbbe3ba3cb5c83ba985554aff7
Author: David Thompson <address@hidden>
Date:   Sat Aug 29 21:55:12 2015 -0400

    build: ruby: Avoid long build directory names.
    
    Having the hash of the source gem in the source directory file name proved 
to
    be problematic when running the test suite for the 'pg' gem that creates
    UNIX-domain sockets in the source directory and exceeded the 108 character
    limit on GNU/Linux systems.
    
    * guix/build/ruby-build-system.scm (unpack): Rename unpacked gem directory 
to
      "gem".
---
 guix/build/ruby-build-system.scm |   16 ++++++++++------
 1 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/guix/build/ruby-build-system.scm b/guix/build/ruby-build-system.scm
index 90fab92..4184ccc 100644
--- a/guix/build/ruby-build-system.scm
+++ b/guix/build/ruby-build-system.scm
@@ -44,12 +44,16 @@ directory."
 (define* (unpack #:key source #:allow-other-keys)
   "Unpack the gem SOURCE and enter the resulting directory."
   (and (zero? (system* "gem" "unpack" source))
-       (begin
-         ;; The unpacked gem directory is named the same as the archive, sans
-         ;; the ".gem" extension.
-         (chdir (match:substring (string-match "^(.*)\\.gem$"
-                                               (basename source))
-                                 1))
+       ;; The unpacked gem directory is named the same as the archive, sans
+       ;; the ".gem" extension.  It is renamed to simply "gem" in an effort to
+       ;; keep file names shorter to avoid UNIX-domain socket file names and
+       ;; shebangs that exceed the system's fixed maximum length when running
+       ;; test suites.
+       (let ((dir (match:substring (string-match "^(.*)\\.gem$"
+                                                 (basename source))
+                                   1)))
+         (rename-file dir "gem")
+         (chdir "gem")
          #t)))
 
 (define* (build #:key source #:allow-other-keys)



reply via email to

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