gnu: tadbit: Update to 1.0.1-1.5c4c1dd.

* gnu/packages/bioinformatics.scm (tadbit): Update to 1.0.1-1.5c4c1dd.
[arguments]: Remove trailing #T from build phases; pass -fcommon to test
binaries; run tests conditionally.

gnu/packages/bioinformatics.scm

@@ -9430,66 +9430,72 @@ applications for tackling some common problems in a user-friendly way.")
     (license (list license:lgpl2.1+
                    license:gpl3+))))
 
+;; We use this seemingly arbitrary commit because of
+;; https://github.com/3DGenomes/TADbit/issues/371
 (define-public tadbit
-  (package
+  (let ((commit "5c4c1ddaadfbaf7e6edc58173e46d801093bdc9b")
-    (name "tadbit")
+        (revision "1"))
-    (version "1.0.1")
+    (package
-    (source (origin
+      (name "tadbit")
-              (method git-fetch)
+      (version (git-version "1.0.1" revision commit))
-              (uri (git-reference
+      (source (origin
-                    (url "https://github.com/3DGenomes/TADbit")
+                (method git-fetch)
-                    (commit (string-append "v" version))))
+                (uri (git-reference
-              (file-name (git-file-name name version))
+                      (url "https://github.com/3DGenomes/TADbit")
-              (sha256
+                      (commit commit)))
-               (base32
+                (file-name (git-file-name name version))
-                "0hqrlymh2a2bimcfdvlssy1x5h1lp3h1c5a7jj11hmcqczzqn3ni"))))
+                (sha256
-    (build-system python-build-system)
+                 (base32
-    (arguments
+                  "17nwlvjgqpa7x6jgh56m3di61ynaz34kl1jamyv7r2a5rhfcbkla"))))
-     `(#:phases
+      (build-system python-build-system)
-       (modify-phases %standard-phases
+      (arguments
-         (add-after 'unpack 'fix-problems-with-setup.py
+       `(#:phases
-           (lambda* (#:key outputs #:allow-other-keys)
+         (modify-phases %standard-phases
-             ;; Don't attempt to install the bash completions to
+           (add-after 'unpack 'fix-problems-with-setup.py
-             ;; the home directory.
+             (lambda* (#:key outputs #:allow-other-keys)
-             (rename-file "extras/.bash_completion"
+               (substitute* "src/test/Makefile"
-                          "extras/tadbit")
+                 (("^CFLAGS=") "CFLAGS= -fcommon"))
-             (substitute* "setup.py"
+               
-               (("\\(path.expanduser\\('~'\\)")
+               ;; Don't attempt to install the bash completions to
-                (string-append "(\""
+               ;; the home directory.
-                               (assoc-ref outputs "out")
+               (rename-file "extras/.bash_completion"
-                               "/etc/bash_completion.d\""))
+                            "extras/tadbit")
-               (("extras/\\.bash_completion")
+               (substitute* "setup.py"
-                "extras/tadbit"))
+                 (("\\(path.expanduser\\('~'\\)")
-             #t))
+                  (string-append "(\""
-         (replace 'check
+                                 (assoc-ref outputs "out")
-           (lambda* (#:key inputs outputs #:allow-other-keys)
+                                 "/etc/bash_completion.d\""))
-             (add-installed-pythonpath inputs outputs)
+                 (("extras/\\.bash_completion")
-             (invoke "python3" "test/test_all.py")
+                  "extras/tadbit"))))
-             #t)))))
+           (replace 'check
-    (native-inputs
+             (lambda* (#:key tests? inputs outputs #:allow-other-keys)
-     `(("glib" ,glib "bin")             ;for gtester
+               (when tests?
-       ("pkg-config" ,pkg-config)))
+                 (add-installed-pythonpath inputs outputs)
-    (inputs
+                 (invoke "python3" "test/test_all.py")))))))
-     ;; TODO: add Chimera for visualization
+      (native-inputs
-     `(("imp" ,imp)
+       `(("glib" ,glib "bin")             ;for gtester
-       ("mcl" ,mcl)
+         ("pkg-config" ,pkg-config)))
-       ("python-future" ,python-future)
+      (inputs
-       ("python-h5py" ,python-h5py)
+       ;; TODO: add Chimera for visualization
-       ("python-scipy" ,python-scipy)
+       `(("imp" ,imp)
-       ("python-numpy" ,python-numpy)
+         ("mcl" ,mcl)
-       ("python-matplotlib" ,python-matplotlib)
+         ("python-future" ,python-future)
-       ("python-pysam" ,python-pysam)))
+         ("python-h5py" ,python-h5py)
-    (home-page "https://3dgenomes.github.io/TADbit/")
+         ("python-scipy" ,python-scipy)
-    (synopsis "Analyze, model, and explore 3C-based data")
+         ("python-numpy" ,python-numpy)
-    (description
+         ("python-matplotlib" ,python-matplotlib)
-     "TADbit is a complete Python library to deal with all steps to analyze,
+         ("python-pysam" ,python-pysam)))
+      (home-page "https://3dgenomes.github.io/TADbit/")
+      (synopsis "Analyze, model, and explore 3C-based data")
+      (description
+       "TADbit is a complete Python library to deal with all steps to analyze,
 model, and explore 3C-based data.  With TADbit the user can map FASTQ files to
 obtain raw interaction binned matrices (Hi-C like matrices), normalize and
 correct interaction matrices, identify and compare the so-called
 @dfn{Topologically Associating Domains} (TADs), build 3D models from the
 interaction matrices, and finally, extract structural properties from the
 models.  TADbit is complemented by TADkit for visualizing 3D models.")
-    (license license:gpl3+)))
+      (license license:gpl3+))))
 
 (define-public kentutils
   (package