Thanks

We looked at this before, at that point it didn't seem like we could make it work because of the interface implementations (IFn and Object), but turns out if we leave those out it's still a pretty useful library

> On babashka implementing interfaces is not supported More specifically: this is supported for only one interface at a time + an optional amount of protocols

@U04V15CAJ I tried doing just the Object (toString []) implementation, leaving out the IFn but that didn't work. If that could be made to work that would be fantastic.

(not an interface in this case but a base class override)

ah hmm, I see interfaces aren't supported on records, but only a single one in reify

or alternatively if sci could provide an alternative way to override str for custom types

ok, feel free to post issue(s) at babashka and when I have time I'll look into it

👍

you can for now use bb reader conditionals as well to work around things

(which is what you're doing I see now)

@U04V15CAJ is there a guide on how to check if a library is bb-compatible and what are the steps needed to make it compatible if it isn't?

This is documented here: https://book.babashka.org/#differences-with-clojure

Most of it has to do with the limitation that you cannot create new Java types at runtime in a native image

Gotcha. I'm trying to check if https://github.com/cgrand/xforms is compatible

Looks like it isn't atm 😞

; export BABASHKA_CLASSPATH="$(clojure -Sdeps '{:deps {net.cgrand/xforms {:mvn/version "0.19.2"}}}' -Spath)"

bb -e "
(require '[net.cgrand.xforms :as x])
(x/into [] cat [[1][2]])
"
----- Error --------------------------------------------------------------------
Type:     clojure.lang.ExceptionInfo
Message:  Unable to resolve classname: java.util.Comparator
Phase:    analysis

# ~  1 ↵

I was looking into as well. We could just add that class.

The next one is:

128:  (satisfies? #?(:clj clojure.core.protocols/IKVReduce :cljs IKVReduce) coll))
                  ^--- Could not resolve symbol: clojure.core.protocols/IKVReduce

That can also be accounted for, but takes some more work

Welcome to make an issue about trying to make xforms run in bb

What are you trying to use it for, for the library itself or is there another lib which depends on it?

I use it heavily in clojure (❤️ xducers) and I'd miss it from my utility belt. It hasn't been updated in a while and has a few https://github.com/cgrand/xforms/pulls, fixing reflection warnings among others so I have been thinking about merging those in my own fork. And when one of my colleagues asked whether it's available in bb I decided to check whether I could patch it so it's compatible. But it sounds like bb itself (or sci?) will need some additions for that, right?

adding classes is fairly easy, adding protocols requires more work, but can be done

Thanks, I'll add an issue there!

cool. there may be some other dragons, I don't expect this to work soon, but long term it'd be interesting to see what can be done

(after seeing this: https://github.com/cgrand/xforms/blob/62375212a8604daad631c9024e9dbe1db4ec276b/src/net/cgrand/xforms.cljc#L101, this is currently not possible in SCI, reify with multiple classes)

Thanks for the quick eyeballing.

Christophe writes some pretty advanced Clojure... might be worth considering making a spin-off library which implements the functions you use most in a bb-compatible way

He's currently busy with creating ClojureDart so I wouldn't expect him to provide too much support on this...

and since the lib is already .cljc it might not be too much work to introduce a few :bbs