Are goog-defines supported for simple and/or whitespace? I remember they weren’t with the initial impl

@martinklepsch IIRC they didn't work properly in :none but always worked for everything else

@thheller After a bit more discussion elsewhere I think they should work for none, simple + advanced but never work for whitespace

that seems like a bug then. definitely no reason they shouldn't work in whitespace mode

@thheller AFAIK whitespace == no closure at all. I believe thats the reason why it’s not supported

:none is no closure at all. all other modes are piped through closure

@martinklepsch Didn't you submit a patch for this? https://dev.clojure.org/jira/browse/CLJS-1389

@rauh looks like it 😄

It’s been a while though 🙂 I have vague memory of discussing with @dnolen that not supporting whitespace was fine/ok for some reason.

from the ticket I gather that CLOSURE_UNCOMPILED_DEFINES is used. :whitespace is a compiled mode so CLOSURE_DEFINES should be used?

hmm looks like they are only emitted for :none and :whitespace is setting them via the closure compiler api

so maybe Closure doesn't support them for :whitespace and instead expects CLOSURE_DEFINES in JS?

shadow-cljs always uses CLOSURE_DEFINES which seems to be more reliable