<< Back to previous view

[CLJS-1902] Add support for compiler option :inline-source-maps Created: 24/Jan/17  Updated: 25/Apr/19

Status: Open
Project: ClojureScript
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Enhancement Priority: Minor
Reporter: Antonin Hildebrand Assignee: Antonin Hildebrand
Resolution: Unresolved Votes: 6
Labels: None

Attachments: Text File CLJS-1902-2.patch     Text File CLJS-1902.patch    
Patch: Code and Test


1. refactor `emit-source-map` and break it into multiple functions
2. fix logic for relative path computation (see `strip-prefix-path`)
3. add support for `:inline-source-maps` option
4. add tests

Related: CLJS-1402, CLJS-1901

Comment by Antonin Hildebrand [ 24/Jan/17 8:09 PM ]


Comment by Antonin Hildebrand [ 24/Jan/17 8:27 PM ]

Full review: https://github.com/clojure/clojurescript/compare/darwin:inline-source-maps~3...darwin:inline-source-maps

Also please note that the first patch testing original functionality fails in one test because there was a bug in timestamp formatting in :source-map-url case:

Comment by Antonin Hildebrand [ 25/Jan/17 10:25 AM ]

Today when testing Dirac I realised we need to embed sources contents as well.

Additional patch:

New review URL: https://github.com/darwin/clojurescript/compare/inline-source-maps~4...darwin:inline-source-maps

Tested in DevTools and works like a charm.

Comment by Dusan Maliarik [ 06/Feb/17 10:00 AM ]

This would be helpful for us as well.

Comment by Andrea Richiardi [ 01/May/17 3:49 PM ]

I have run across this one as well by following this tutorial.

Without either this patch or Dirac's complicated setup it is not currently possible to use node --inspect and debug correctly. The symptom I see on our side is that source maps are detected but for some reason Chrome DevTools does not show them in the Tree View.

The content of one of it is:

{"version":3,"file":"\/Users\/user\/cqrs-engine-cljs\/out\/cqrs\/event_store.js","sources":["event_store.cljs"], ...
Comment by David Nolen [ 16/Jun/17 10:29 AM ]

Linking to patches outside of JIRA is not proper for tickets. Please add a single squashed patch to this ticket directly.

Comment by Antonin Hildebrand [ 16/Jun/17 2:25 PM ]

Attached it as a patch file.

Took https://github.com/darwin/clojurescript/compare/inline-source-maps~4...darwin:inline-source-maps.diff and applied it to current master. It applied cleanly without conflicts. Tests are still passing on my machine.

Comment by Mike Fikes [ 19/Nov/17 7:36 PM ]

Patch no longer applies; needs re-baseline.

Comment by Antonin Hildebrand [ 05/Apr/19 8:18 AM ]

Newly when developing chrome extensions, source-maps got broken again[1]. Probably due to some added security restrictions in what Chrome allows Chrome DevTools to "see".

Some people on the interwebs claim that inlined source-maps are a possible workaround:

[1] https://bugs.chromium.org/p/chromium/issues/detail?id=931675

Comment by David Nolen [ 22/Apr/19 1:37 PM ]

Just noting that I'm ok with what's proposed. Please re-baseline the patch and I can review in the near future.

Comment by Antonin Hildebrand [ 22/Apr/19 1:39 PM ]

Thanks. I will try to look into it during this week.

Comment by Antonin Hildebrand [ 25/Apr/19 10:07 AM ]
Generated at Fri Apr 26 04:47:52 CDT 2019 using JIRA 4.4#649-r158309.