Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for sourceURL / source mapping #207

Open
ilovett opened this issue Apr 1, 2013 · 15 comments
Open

Support for sourceURL / source mapping #207

ilovett opened this issue Apr 1, 2013 · 15 comments

Comments

@ilovett
Copy link

ilovett commented Apr 1, 2013

This would be nice to have an option to append the sourceURL / sourcemap for the compressed script in production.

@vshjxyz
Copy link

vshjxyz commented Apr 23, 2013

+1 now that coffeescript supports source maps this would be a nice enhancement

@bcooksey
Copy link

👍

@toabi
Copy link

toabi commented May 26, 2013

👍

edit: I played around with it a bit, but one thing is that for example uglifyjs doesn't support source-map generation from stdin. I guess this feature is kind of hard to add. I still would like it very much.

@cyberdelia
Copy link
Member

It's on my radar since coffee-script announced support, but it's still a complicated feature to add, and it also work for only some compiler, not speaking of compressor...
So please refrain to +1 a bit, and give some time for source mapping to mature a bit :bowtie:

@santagada
Copy link

I would like to help with this, can I try to tackle it or are you working on it on some branch/fork?

@ilovett
Copy link
Author

ilovett commented Dec 4, 2013

I have not done any work on this. How about you @cyberdelia ?

Sass CSS compiler supports css sourcemapping but I have not played with it. Stylus is in development... How would pipeline handle this?

@santagada
Copy link

for javascript it is reasonably straight forward, then we can see all the other compilers and minifiers. The biggest problem for something generating css is that in production cssmin would need to update the map...

@santagada
Copy link

Has this matured? Uglifyjs, less, sass, coffeescript and most tools support source maps now... any progress on this?

@aehlke
Copy link

aehlke commented Aug 12, 2014

@santagada #346

@santagada
Copy link

@aehlke This pull request contains merge conflicts. Please resolve them.

also it hasn't been touched in a while, and needs a patch in django to work.

@aehlke
Copy link

aehlke commented Aug 12, 2014

@santagada Indeed, just wanted to give you an update on progress.

I submitted a patch to Django but haven't heard anything yet. We should probably override this broken behavior inside of the django-pipeline post-processor for now since it'll be a long time until the fix is in Django.

@aehlke
Copy link

aehlke commented Aug 12, 2014

I'm swamped now though so would appreciate any help on all this. LMK if you want to help and we can talk more.

@tobz
Copy link

tobz commented Jan 4, 2016

Is there any hope of getting that PR merged, ignoring merge conflicts, to add this support? Mangling provides significant size reduction in JS files compared to basic minification, but is untenable without having sourcemaps for debugging.

@fdintino
Copy link
Member

@tobz I've got a cleaned up and merge-safe pull request open at #490. I'm working on adding the documentation and unit tests required so that it can be merged in, but if it's an immediate need it's stable; we're using it in production at theatlantic.com.

@natecox
Copy link
Contributor

natecox commented Mar 3, 2016

Any updates on this? Source mapping would be great.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

10 participants