Skip to content

Platform specific prepare breaks source map paths #485

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

Closed
jasssonpet opened this issue May 18, 2015 · 3 comments
Closed

Platform specific prepare breaks source map paths #485

jasssonpet opened this issue May 18, 2015 · 3 comments
Assignees
Labels
Milestone

Comments

@jasssonpet
Copy link
Contributor

Create file test.ios.coffee:

class MyClass

Compile with --map and tns prepare ios:

File platforms/ios/TestApp/app/test.js:

(function() {
  var MyClass;
  MyClass = (function() {
    function MyClass() {}
    return MyClass;
  })();
}).call(this);

//# sourceMappingURL=test.ios.js.map

File platforms/ios/TestApp/app/test.js.map:

{
  "version": 3,
  "file": "test.ios.js",
  "sourceRoot": "",
  "sources": [
    "test.ios.coffee"
  ],
  "names": [],
  "mappings": ";AAAA;AAAA,MAAA,OAAA;;AAAA,EAAM;yBAAN;;mBAAA;;MAAA,CAAA;AAAA"
}

The source mapping is broken.

This should be also happening with Babel, TypeScript and others.

Are there any inline source maps, that can be used only in debugging?

@jasssonpet jasssonpet added the bug label May 18, 2015
@jasssonpet jasssonpet changed the title Platform specific prepare breaks source maps Platform specific prepare breaks source map paths May 18, 2015
@burkeholland
Copy link

I'm not able to debug typescript and I assume this is related to this issue? My issue is that the .ts files simply don't show up in Safari Web Inspector, but I'm not completely sure why.

@enchev
Copy link
Contributor

enchev commented Jun 13, 2016

Still valid with 2.0.1

@enchev enchev self-assigned this Jun 14, 2016
@enchev enchev added this to the 2.1 milestone Jun 15, 2016
@enchev enchev closed this as completed Jun 15, 2016
vchimev pushed a commit to NativeScript/nativescript-cli-tests that referenced this issue Jun 20, 2016
@vchimev
Copy link

vchimev commented Jun 20, 2016

Tests added: NativeScript/nativescript-cli-tests@172ac70.

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

No branches or pull requests

4 participants