[Bug]: prep-lib is broken

[TimoKramer]

What version of Datahike are you using?

main

What version of Java are you using?

19

What operating system are you using?

Linux

What database EDN configuration are you using?

Describe the bug

When using datahike as a lib I need to run clj -X:deps prep to compile the java classes. This throws Prep function could not be resolved: tools.build/jcompile because the clj -T:build jcompile functionality seems to be broken. Tried from datahike-server repository.

In the datahike-folder when cloned to disk clj -X:deps prep does nothing but clj -T:build compile throws java.lang.Exception: Cyclic load dependency: [ /clojure/main ]->/clojure/spec/alpha->[ /clojure/main ]->/clojure/core/server

What is the expected behaviour?

clj -X:deps prep should compile the java code like it is described here.

How can the behaviour be reproduced?

clj -X:deps prep

[Moocar]

I've also run into this. I tried debugging it, but can't deduce any useful info from the emitted stacktrace for java.lang.Exception: Cyclic load dependency: [ /clojure/main ]->/clojure/spec/alpha->[ /clojure/main ]->/clojure/core/server .

I tried running the build command from the circle ci yaml (clj -A:build -P). It completed successfully, but the prep command still failed with the above error.

[oelrich]

The reason for this seems to be that the [:deps/prep-lib :fn] path points to a babashka build task and not a function. A really ugly hack that seems to provide a workaround for the problem is as follows.

First add a build.clj:

(ns build
  (:require [clojure.java.shell :refer [sh]]))

(defn bb-trigger [_input]
  (sh "bb" "jcompile"))

Then change the ´deps.edn´ to use that file as build namespace and the function, i.e. :build {:ns-default build} and :fn bb-trigger.

The result is really slow for some reason, and someone with more clojure experience may find a neater solution.

[TimoKramer]

A really ugly hack

Haha :) I like it. I will try to test it and solve this issue soonish. Thanks!

[TimoKramer]

I think I will do it like you proposed. Do you want to submit that patch? @oelrich

[oelrich]

Absolutely! I'll try to get it done today.