ForbesLindesay / browserify-middleware

express middleware for browserify, done right
http://browserify.org
MIT License
381 stars 66 forks source link

Source maps broken in firefox #56

Closed glenjamin closed 9 years ago

glenjamin commented 10 years ago

This seems related to https://github.com/substack/node-browserify/issues/681

If i remove the leading "/" from https://github.com/ForbesLindesay/browserify-middleware/blob/master/lib/compile.js#L40 then it works.

It looks like the "/" is there for a chrome issue though, so unsure if this is a neat fix.

In a related note, there doesn't seem to be any way to tweak the source map handling from calling code - so I can't see a way to change this behaviour from my side without an upstream change.

ForbesLindesay commented 9 years ago

We no longer to any munging of source maps, so any remaining bugs should be directed to browserify instead.