unofficial mirror of bug-guix@gnu.org 
 help / color / mirror / code / Atom feed
From: Kristiyan Kanchev <skrechy@gmail.com>
To: "Roman Scherer" <roman.scherer@burningswell.com>,
	"Ludovic Courtès" <ludo@gnu.org>
Cc: Rostislav Svoboda <Rostislav.Svoboda@gmail.com>,
	73432@debbugs.gnu.org,
	Andreas Bauer <andreas.wolfgang.bauer@gmail.com>
Subject: bug#73432: clojure package is broken
Date: Fri, 4 Oct 2024 11:33:54 +0300	[thread overview]
Message-ID: <9173f3ce-ba91-42c5-8755-3c9288d7dd71@gmail.com> (raw)
In-Reply-To: <86zfnlqm3e.fsf@burningswell.com>

Hello Roman,

What do you mean that you're getting the same error? Getting the same 
error with Guix Clojure or with Nix Clojure? Metabase or the test 
project by Andreas?

I'm fairly confident that the problem is not caused by the S3 transport 
because alot of transitive dependencies are missing from the class-path 
that are not related to these libraries you cited. I was investigating 
the reported class-path from within Clojure with 
https://github.com/clojure/java.classpath, which is also dependency for 
Metabase, so you could try it in a nREPL session.

I'm currently trying to  maven-resolver-provider from 3.9 to 3.8 and 
will report results, hopefully soon -- I'm new to Guix and need some 
more time. If it works out, I'll try to push patches.

Thanks,
Kriss

On 10/3/24 17:48, Roman Scherer wrote:
> Ludo recently merged some of my patches that I have re-submitted
> recently. There are 4 patches missing, that I haven't submitted
> again. They contain the S3 support.
>
> But it's not clear if they can be included inn Guix, because some of the
> artifacts are Clojure EDN files generated from AWS API docs. The code to
> generate them has not been published by the Clojure team. That was a
> blocker back in the day according to Maxim Devos.
>
> The patches are here:
>
> https://github.com/r0man/guix/commits/clojure-s3-transport/
>
> I really should find that thread ...
>
> Roman Scherer <roman.scherer@burningswell.com> writes:
>
>> Hi,
>>
>> I just tried this and got the same error. I'm not using the Clojure
>> package from Guix because the AWS S3 support has been ripped out. I use
>> the package from Nix.
>>
>> My guess is, this error is also because of the lacking S3 support, or if
>> not it will break later. The deps.edn file of Metabase contains:
>>
>> ```
>> {"athena"   {:url "https://s3.amazonaws.com/maven-athena"}
>>    "redshift" {:url "https://s3.amazonaws.com/redshift-maven-repository/release"}
>>    ;; for metabase/saml20-clj
>>    "opensaml" {:url "https://build.shibboleth.net/nexus/content/repositories/releases/"}}
>> ```
>>
>> So it's using S3 to resolve Maven dependencies. There is an open issue
>> for this https://issues.guix.gnu.org/53765 and I opened a duplicate of
>> this a while ago (which I can't find right now). Unfortunaltly this went
>> nowhwere.
>>
>> Roman
>>
>> Ludovic Courtès <ludo@gnu.org> writes:
>>
>>> Hi Andreas and Kristiyan,
>>>
>>> Cc’ing Roman and Rostislav who are among the recent committers for this
>>> file.
>>>
>>> Andreas Bauer <andreas.wolfgang.bauer@gmail.com> wrote:
>>>
>>>> Now it can be fixed with time machine:
>>>>
>>>> guix time-machine --commit=98e4bfe96f2c99daa0b66b65c1d379bb385301a8 -- guix
>>>> shell --pure --development --check clojure-tools openjdk coreutils
>>>>
>>>> Then in time-machine run again clojure-X:curl and the demo will work.
>>> Kristiyan Kanchev <skrechy@gmail.com> skribis:
>>>
>>>> I'm seeing issues with the project Metabase: https://github.com/metabase/metabase. When `clojure -M:run`, it fails with:
>>>> Execution error (ClassNotFoundException) at java.net.URLClassLoader/findClass (URLClassLoader.java:476).
>>>> jakarta.servlet.AsyncContext
>>>>
>>>> The problem seems to be that Clojure CLI (`clojure-tools` guix package) doesn't construct proper class path including all transitive
>>>> dependencies of the project. I've tried to figure out what might be causing the issue and I suspect `maven-resolver-*` packages. Clojure
>>>> CLI depends on `tools.deps` clojure lib, which itself depends on `maven-resolver-*`. There is a mention in the `tools.deps` changelog of
>>>> reverting `maven-resolver-*` libs in version 0.18.1374 with comment "Revert update of Maven resolver libs - seeing resolution
>>>> differences".
>>>> Here is the relevant commit:
>>>> https://github.com/clojure/tools.deps/commit/04539cd060d41a528794c04b50a6aa2293b672ce
>>>>
>>>> Up until this date, `tools.deps` lists the older versions of maven-resolver-* libs in its deps.end file. Since Guix uses the latest versions of
>>>> maven-resolver-* libs, I think this is causing the issue.
>>> Roman, Rostislav: does that ring a bell?
>>>
>>> Thanks,
>>> Ludo’.




  reply	other threads:[~2024-10-04  8:36 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-09-23  2:21 bug#73432: clojure package is broken Andreas Bauer
2024-09-26 13:50 ` Kristiyan Kanchev
2024-10-03 14:00   ` Ludovic Courtès
2024-10-03 14:41     ` Roman Scherer
2024-10-03 14:48       ` Roman Scherer
2024-10-04  8:33         ` Kristiyan Kanchev [this message]
2024-10-04 10:51           ` Roman Scherer
2024-10-06 12:57             ` Rostislav Svoboda
2024-10-09 12:15               ` Kristiyan Kanchev
2024-10-09 12:18                 ` Roman Scherer
2024-10-15  7:35 ` bug#73432: [PATCH 0/3] Fix clojure-tools-deps: wrong resolution of deps Kristiyan Kanchev
2024-10-15  7:43   ` bug#73432: [PATCH 1/3] gnu: Add maven-resolver-1.6 Kristiyan Kanchev
2024-10-15  7:43   ` bug#73432: [PATCH 2/3] gnu: Add maven-3.8-core Kristiyan Kanchev
2024-10-15  7:43   ` bug#73432: [PATCH 3/3] gnu: clojure-tools-deps: Fix wrong resolution of clojure deps Kristiyan Kanchev

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=9173f3ce-ba91-42c5-8755-3c9288d7dd71@gmail.com \
    --to=skrechy@gmail.com \
    --cc=73432@debbugs.gnu.org \
    --cc=Rostislav.Svoboda@gmail.com \
    --cc=andreas.wolfgang.bauer@gmail.com \
    --cc=ludo@gnu.org \
    --cc=roman.scherer@burningswell.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).