scicloj / wolframite

An interface between Clojure and Wolfram Language (the language of Mathematica)
https://scicloj.github.io/wolframite/
Mozilla Public License 2.0
56 stars 2 forks source link

Does not support java 22 when trying to render notebooks #56

Closed light-matters closed 1 month ago

light-matters commented 4 months ago

Following on from a delayed pull request. It is still the case that manjaro linux fails to initialize a REPL under java22-openjdk with the following deps:

{:paths ["src" "resources"]
 :deps {org.clojure/clojure {:mvn/version "1.12.0-beta1"}

        scicloj/wolframite {:git/url "https://github.com/scicloj/wolframite"
                            :git/sha "d9638186076bdd123991a58242ac1809c9d9f2f2"}

        babashka/fs {:mvn/version "0.5.21"}
        io.jhdf/jhdf {:mvn/version "0.6.10"}
        scicloj/tablecloth {:mvn/version "7.021"}
        cnuernber/dtype-next {:mvn/version "10.000-beta-48"}
        ;; tech/core.parallel {:mvn/version "0.1.5"}
        generateme/fastmath {:mvn/version "2.4.0"}
        org.mentat/emmy {:mvn/version "0.31.0"}

        ;; visual tools
        aerial.hanami/aerial.hanami {:mvn/version "0.19.0"}

        org.scicloj/clay {:mvn/version "2-beta11"}
        org.scicloj/kindly {:mvn/version "4-beta5"}
        org.scicloj/kind-clerk {:mvn/version "RELEASE"}
        org.scicloj/kindly-default {:mvn/version "RELEASE"}}

 :aliases
 {:test
  {:extra-paths ["test"]
   :extra-deps {org.clojure/test.check {:mvn/version "1.1.1"}
                io.github.cognitect-labs/test-runner
                {:git/tag "v0.5.1" :git/sha "dfb30dd"}}}
  :build {:deps {io.github.clojure/tools.build
                 {:mvn/version "0.9.6"}
                 slipset/deps-deploy {:mvn/version "0.2.0"}}
          :ns-default build}}}

Switching to java21 solves the problem.

holyjak commented 4 months ago

I am running with openjdk 22 without any problems, on MacOS.

holyjak commented 4 months ago

This is caused by https://github.com/nextjournal/markdown/issues/23. While GraalVM has been fixed, Nextjournal either needs to finish its upgrade to the latest graalvm in https://github.com/nextjournal/markdown/pull/24 or get rid of the graalvm polyglot dependency via https://github.com/nextjournal/markdown/pull/25

Until either is finished, I suggest sticking to Java 21.

Alternatively, we could follow https://github.com/nextjournal/clerk/issues/642#issuecomment-2032310246 and use the dependency io.github.nextjournal/markdown {:git/sha "6683c48dfdb23404a23057817b6ac3acf0310bca"} until the problem gets fixed.

holyjak commented 1 month ago

nextjournal markdown 0.6.157 drops GraalJS.