diff mbox series

[bug#53765,v5,1/2] gnu: clojure-tools-deps-alpha: Patch unpackageable deps warning.

Message ID f7ddaeaea1a97738393e5e2fa6f02c0ca65fa1c7.1650195554.git.mail@reilysiegel.com
State New
Headers show
Series [bug#53765,v5,1/2] gnu: clojure-tools-deps-alpha: Patch unpackageable deps warning. | expand

Checks

Context Check Description
cbaines/comparison success View comparision
cbaines/git branch success View Git branch
cbaines/applying patch success View Laminar job
cbaines/issue success View issue

Commit Message

Reily Siegel April 16, 2022, 9:59 p.m. UTC
* gnu/packages/clojure.scm (clojure-tools-deps-alpha) [arguments] <#:phases>
{remove-s3-transporter}: Patch out warning for missing S3TransporterFactory,
caused by a missing dependency that cannot be included in Guix.
---
 gnu/packages/clojure.scm | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)
diff mbox series

Patch

diff --git a/gnu/packages/clojure.scm b/gnu/packages/clojure.scm
index e6eb749501..8ec04bd94d 100644
--- a/gnu/packages/clojure.scm
+++ b/gnu/packages/clojure.scm
@@ -501,11 +501,11 @@  (define-public clojure-tools-deps-alpha
        #:tests? #f
        #:phases
        (modify-phases %standard-phases
-         ;; FIXME: Currently, the S3 transporter depends on ClojureScript,
-         ;; which is very difficult to package due to dependencies on Java
-         ;; libraries with non-standard build systems. Instead of actually
-         ;; packaging these libraries, we just remove the S3 transporter that
-         ;; depends on them.
+         ;; FIXME: Currently, the S3 transporter depends on
+         ;; com.cognitect.aws/s3, which is built from Amazon's aws-sdk-js
+         ;; using a non-free script. For more information see:
+         ;; https://issues.guix.gnu.org/53765
+         ;; https://github.com/cognitect-labs/aws-api/issues/116
          (add-after 'unpack 'remove-s3-transporter
            (lambda _
              (for-each delete-file
@@ -519,9 +519,10 @@  (define-public clojure-tools-deps-alpha
                         (string-append
                          "src/test/clojure/clojure/"
                          "tools/deps/alpha/util/test_s3_transporter.clj")))
-             (substitute*
-                 "src/main/clojure/clojure/tools/deps/alpha/util/maven.clj"
+             (substitute* "src/main/clojure/clojure/tools/deps/alpha/util/maven.clj"
                (("clojure.tools.deps.alpha.util.s3-transporter")
+                "")
+               (("(printerrln \"Warning: failed to load the S3TransporterFactory class\")")
                 "")))))))
     (propagated-inputs (list maven-resolver-api
                              maven-resolver-spi