Fork me on GitHub
#clojure-dev
<
2024-03-11
>
seancorfield17:03:57

These seem to be new reflection warnings in tools.reader:

Reflection warning, clojure/tools/reader.clj:1025:74 - call to method setLength can't be resolved (target class is unknown).
Reflection warning, clojure/tools/reader.clj:1030:74 - call to method setLength can't be resolved (target class is unknown).
I haven't checked whether they are due to changes in tools.reader or Clojure 1.12 Alpha 9.

Alex Miller (Clojure team)17:03:21

so these are new as of ... 1.12.0?

seancorfield17:03:02

Unclear at the moment. We did some dependency updates this morning (but not of Clojure which was already on alpha 9) so I haven't tracked down exactly what causes these new warnings...

seancorfield17:03:26

(I posted mostly as an FYI in case anyone had an immediate response that might save me some work)

Alex Miller (Clojure team)17:03:40

new to me, but we can look into it

Alex Miller (Clojure team)18:03:07

can easily repro in the lib, doesn't seem related to 1.12, will look

Alex Miller (Clojure team)18:03:57

looks like it was in "TRDR-71: source-logging-push-back-reader throws" change from Nov

seancorfield18:03:26

Ah, interesting. I wonder why we've only just started seeing it?

> clojure -Stree  -M:build:cfml-affiliate:dev:+default:test | fgrep tools.reader
      X org.clojure/tools.reader 1.3.6 :older-version
    X org.clojure/tools.reader 1.3.6 :parent-omitted
    . org.clojure/tools.reader 1.3.6
    X org.clojure/tools.reader 1.3.4 :older-version
      X org.clojure/tools.reader 1.3.6 :older-version
    . org.clojure/tools.reader 1.4.0
We don't depend on it directly so it could be due to our round of minor updates this morning bringing in 1.4.0?

seancorfield18:03:56

(we exclude known reflection warnings from all our deps but this was an "unknown") 🙂

seancorfield18:03:22

Ah, it's due to moving from tools.build 0.9.6 to 0.10.0 which brings in an updated tools.namespace which in turn brings in the updated tools.reader.

borkdude19:03:40

hmm, interesting, this should break lots of my graalvm native-image projects :)

borkdude19:03:15

haven't run into it yet though