Clojurians Log v2

@lilactown hey do you have a minimal repro for https://clojure.atlassian.net/browse/CLJS-3413 ? I looked at this a bit but it's still not clear to me how this is failing, I have branch where first I verify that normally it works (not in :preloads context - then examining :preloads I don't see how :preloads compilation would cause any change here. maybe @thheller can clarify how the machinery is being skipped?

far as I can tell, we will compile a single segment preload (with macros), the ns form will be analyzed, the macro load side effect run, and then later when when get-expander-ns is invoked env should have :require-macros.

oh nevermind - I get it now

it won't be in the requires for the current namespace

I linked a repo above https://github.com/lilactown/macro-repl-bug

@lilactown ok all previous tests passing, https://github.com/clojure/clojurescript/pull/229 - I still need to convert yours into test case - maybe give this spin and see if it resolves your issue (and doesn't create new ones)

I probably won't have time to give it a spin today but I'll try tomorrow morning

No rush