cljsrn

joshmiller 2024-08-19T17:06:42.381319Z

I wrote up the Hermes engine situation and how to fix it, so now hopefully that info isn’t siloed on Slack: https://increasinglyfunctional.com/2024/08/19/clojurescript-react-native-hermes.html

❤️ 1
hadils 2024-08-19T17:09:36.143319Z

Hi @joshmiller. Thanks for the write-up! I discovered something. I am trying to use com.taoensso/telemere and everything compiles, but when I invoke the REPL and then require the library, it causes Krell to recompile everything and put the ! back in the log file.

hadils 2024-08-19T17:10:10.172739Z

I would like to help with troubleshooting the Krell + CLJS compiler issue. Please tell me where to start.

joshmiller 2024-08-19T17:11:09.820649Z

So the issue now is that we can’t just require a Closure snapshot because the latest CLJS on Krell isn’t transpiling some Closure library sources. I am not sure why that is.

hadils 2024-08-19T17:11:52.561599Z

Ok. I’ll try to look into it.

👍 1
joshmiller 2024-08-19T19:54:30.855569Z

https://x.com/tmikov/status/1824138431959798020?s=46

joshmiller 2024-08-19T19:54:58.318469Z

May end up being a moot point soon

hadils 2024-08-19T19:55:07.073219Z

Indeed.

hadils 2024-08-20T00:30:07.580059Z

Hi @joshmiller. I think I discovered the problem in the CLJS compiler — the files not being transpiled are marked :es8 that is not supported in the closure.clj/lang-level def. I am going to test this now…

hadils 2024-08-20T00:30:27.254399Z

:es8 is :ecmascript2017

hadils 2024-08-20T00:36:30.912409Z

That fixes it!

joshmiller 2024-08-20T00:38:47.588299Z

Nice! So did you have to patch cljs to add that?

hadils 2024-08-20T00:38:59.782359Z

Yes.

hadils 2024-08-20T17:45:40.140319Z

I submitted a bug to the Clojure Q&A as per the guildelines.

👍 1