Skip to main content

Bug Tracker

Side navigation

#13793 closed bug (fixed)

Opened April 19, 2013 04:06PM UTC

Closed May 23, 2013 07:09PM UTC

Last modified May 23, 2013 07:14PM UTC

SourceMap on Google is broken

Reported by: mattrobenolt Owned by: m_gol
Priority: blocker Milestone: 1.10/2.0
Component: build Version: 2.0.0
Keywords: Cc: paul.irish
Blocked by: Blocking:
Attachments (0)
Change History (14)

Changed April 19, 2013 04:12PM UTC by dmethvin comment:1

cc: → paul.irish

I provided the file to Google as jquery-2.0.0.*, the sourcemap would need to be edited manually by Google to retarget the file name. Paul, did you guys do that last time? Or did I provide you one? The ASP.NET CDN uses the name name as we do with the embedded version so it doesn't affect them.

Changed April 19, 2013 04:31PM UTC by dmethvin comment:2

component: unfiledbuild
milestone: None2.0.1
priority: undecidedblocker
status: newopen

Looks like I did the job manually for 1.9.1 and sent them both (with and without version) but it didn't get into the build process; I wasn't sure which one they were using. So as I understand it from Google this can't be fixed, you'll need to use a local copy if you need the sourcemap.

Changed April 19, 2013 04:33PM UTC by dmethvin comment:3

So just in case it's not clear, this will be fixed by us in 2.0.1, but the current Google CDN version cannot be changed.

Changed April 19, 2013 04:34PM UTC by dmethvin comment:4

Whoops, just heard from them. They *are* changing the map, but whether you get it or not will depend on whether you or any intervening proxy has cached the old one forever.

Changed April 19, 2013 06:00PM UTC by mattrobenolt comment:5

FWIW, it seems my validator is able to fetch the right map from Google, but it appears 100% wrong. No idea how it's mapping every token up incorrectly. It's almost like it was built before the comment at the top of jquery.js was added.

Changed May 21, 2013 07:05AM UTC by m_gol comment:6

owner: → m_gol
status: openassigned

Hah, I've noticed the map file generated points to jquery.pre-min.js instead of jquery.js...

I'll look into it.

Changed May 21, 2013 08:11AM UTC by m_gol comment:7

milestone: 2.0.11.10/2.0

Also, the spec changed so we should change the pragma accordingly to //# sourceMappingUrl

Changed May 21, 2013 02:43PM UTC by mattrobenolt comment:8

Hmm, I see the spec was updated May 16th. Is there a discussion around that I can follow?

Changed May 21, 2013 03:13PM UTC by mattrobenolt comment:10

Nice, thank you.

Changed May 21, 2013 04:52PM UTC by dmethvin comment:11

Yes, the -git versions need to have working map files so thanks m_gol. I think it's too early to change from //@ to //# but let's do that for 1.11/2.1 time frame. If Chrome hasn't changed to accept the new format by then, our change will be a motivator for it.

Changed May 22, 2013 01:19PM UTC by m_gol comment:12

Seems reasonable to wait for a pragma change until 1.11/2.1.

I submitted a pull request: https://github.com/jquery/jquery/pull/1275

I really need people to test it before we ship 1.10, this all is way too fragile.

Changed May 23, 2013 07:09PM UTC by Michał Gołębiowski comment:13

resolution: → fixed
status: assignedclosed

Fix #13793. Correct source map generation bugs. Close gh-1275.

Changeset: 997da31121b9d084ccba05a9bb1e258c8c8faaf0