cljsjs / packages

DEPRECATED: Javascript libraries packaged up with Google Closure externs
http://cljsjs.github.io
787 stars 816 forks source link

Including sourcemaps #331

Open atroche opened 8 years ago

atroche commented 8 years ago

Hi!

I'm using https://github.com/cljsjs/packages/tree/master/react-day-picker, and Chrome complains about not being able to parse the source map. I've tried updating the package to include the source map (downloading and moving it to cljsjs/react-day-picker/development/DayPicker.js.map), but the compiler isn't including the file in the compiled JS folder.

Is there any way to do this?

Cheers,

Alistair

Deraen commented 8 years ago

I don't think so.

Foreign deps are repsented as a maps with values read from deps.cljs: https://github.com/clojure/clojurescript/blob/c72e9c52156b3b348aa66857830c2ed1f0179e8c/src/main/clojure/cljs/js_deps.cljc#L191-L203

Maps don't implement ISourceMap: https://github.com/clojure/clojurescript/blob/master/src/main/clojure/cljs/closure.clj#L325-L334

Deraen commented 8 years ago

Related: http://dev.clojure.org/jira/browse/CLJS-1410