An issue to be sorted: Quick Start page should be updated to 1.10.439, but we don’t have a standalone cljs.jar for that release.

huh I thought I uploaded that

hopefully fixed now

@mfikes ^

Cool. https://github.com/clojure/clojurescript-site/pull/275

I think I found an analyzer regression in 1.10.439: https://gist.github.com/martinklepsch/9f885feb061ec3f03f365e22d0d9bf5b

@martinklepsch this issue doesn't really make any sense to me

I don't see how this could have anything to do w/ the analyzer

but I'm assuming that your gist is a minimal repro?

@dnolen the gist shows how ana/analyze-file throws an exception in 439 that isn't thrown in 339

my description (macros, dynvars etc.) might be absolutely wrong, those are more guesswork

@martinklepsch yeah that won't work

if it did work before, you were getting lucky

if you analyze a file with analyzing deps you won't load macros

so it won't work

Also, if you set *analyze-deps* to true it works.

right

that will actually load the files

so I wouldn't consider this a regression

duh, interesting I just copied that *analyze-deps* thing mindlessly

Thanks, and sorry for the false alarm 😅

np

I changed *analyze-deps* to true but now I'm having issues analyzing any namespace that transitively requires something like cljsjs.react — is there something special that needs to be done for this to work?

No such namespace: cljsjs.react, could not locate cljsjs/react.cljs, cljsjs/react.cljc, or JavaScript source providing "cljsjs.react" in file file:/Users/martinklepsch/.m2/repository/reagent/reagent/0.7.0/reagent-0.7.0.jar!/reagent/impl/util.cljs {:tag :cljs/analysis-error}

you need to provide foreign deps entries for that stuff

since you can't resolve directly via the classpath

@martinklepsch ^

@dnolen do build tools handle this by scanning the classpath for deps.cljs? I guess I'll need to implement this in a generic way somehow (i.e. classpath->foreign-deps-entries) — if you have any pointers around that I'd appreciate them 🙂

@martinklepsch that should be fairly easy https://github.com/clojure/clojurescript/blob/master/src/main/clojure/cljs/closure.clj#L2140