Skip to content

Latest version cannot deploy/build due to CoffeeScript error #9217

New issue
New issue
Closed
Closed
Latest version cannot deploy/build due to CoffeeScript error#9217
Assignees
benjamn
Labels
Impact:someType:BugconfirmedWe want to fix or implement it
Milestone
Package Patches

Description

@not-ai-yet
not-ai-yet
opened on Oct 12, 2017

Trying to deploy to Galaxy. Error:

/home/ubuntu/.meteor/packages/coffeescript/.1.12.7_2.6mk04s++os+web.browser+web.cordova/plugin.compile-coffeescript.os/npm/node_modules/meteor/coffeescript-compiler/node_modules/source-map/lib/source-map-generator.js:276:15:
original.line and original.column are not numbers -- you probably meant to omit the original mapping entirely and only map the generated
position. If so, pass null for the original mapping instead of an object with empty or null values.
at SourceMapGenerator_validateMapping
(/home/ubuntu/.meteor/packages/coffeescript/.1.12.7_2.6mk04s++os+web.browser+web.cordova/plugin.compile-coffeescript.os/npm/node_modules/meteor/coffeescript-compiler/node_modules/source-map/lib/source-map-generator.js:276:15)
at SourceMapGenerator_addMapping
(/home/ubuntu/.meteor/packages/coffeescript/.1.12.7_2.6mk04s++os+web.browser+web.cordova/plugin.compile-coffeescript.os/npm/node_modules/meteor/coffeescript-compiler/node_modules/source-map/lib/source-map-generator.js:110:12)
at
/home/ubuntu/.meteor/packages/coffeescript/.1.12.7_2.6mk04s++os+web.browser+web.cordova/plugin.compile-coffeescript.os/npm/node_modules/meteor/coffeescript-compiler/node_modules/source-map/lib/source-map-generator.js:72:17
at Array.forEach (native)
at SourceMapConsumer_eachMapping
(/home/ubuntu/.meteor/packages/coffeescript/.1.12.7_2.6mk04s++os+web.browser+web.cordova/plugin.compile-coffeescript.os/npm/node_modules/meteor/coffeescript-compiler/node_modules/source-map/lib/source-map-consumer.js:157:14)
at Function.SourceMapGenerator_fromSourceMap
(/home/ubuntu/.meteor/packages/coffeescript/.1.12.7_2.6mk04s++os+web.browser+web.cordova/plugin.compile-coffeescript.os/npm/node_modules/meteor/coffeescript-compiler/node_modules/source-map/lib/source-map-generator.js:48:24)
at CoffeeScriptCompiler.compileOneFile (packages/coffeescript-compiler/coffeescript-compiler.js:92:40)
at CachedCoffeeScriptCompiler.compileOneFile (packages/compile-coffeescript/compile-coffeescript.js:31:38)
at packages/caching-compiler/caching-compiler.js:299:32
at
/home/ubuntu/.meteor/packages/coffeescript/.1.12.7_2.6mk04s++os+web.browser+web.cordova/plugin.compile-coffeescript.os/npm/node_modules/meteor/caching-compiler/node_modules/async/lib/async.js:182:20
at replenish
(/home/ubuntu/.meteor/packages/coffeescript/.1.12.7_2.6mk04s++os+web.browser+web.cordova/plugin.compile-coffeescript.os/npm/node_modules/meteor/caching-compiler/node_modules/async/lib/async.js:317:21)
at
/home/ubuntu/.meteor/packages/coffeescript/.1.12.7_2.6mk04s++os+web.browser+web.cordova/plugin.compile-coffeescript.os/npm/node_modules/meteor/caching-compiler/node_modules/async/lib/async.js:324:29
at
/home/ubuntu/.meteor/packages/coffeescript/.1.12.7_2.6mk04s++os+web.browser+web.cordova/plugin.compile-coffeescript.os/npm/node_modules/meteor/caching-compiler/node_modules/async/lib/async.js:44:16
at packages/caching-compiler/caching-compiler.js:316:9
at
/home/ubuntu/.meteor/packages/coffeescript/.1.12.7_2.6mk04s++os+web.browser+web.cordova/plugin.compile-coffeescript.os/npm/node_modules/meteor/caching-compiler/node_modules/async/lib/async.js:182:20
at replenish
(/home/ubuntu/.meteor/packages/coffeescript/.1.12.7_2.6mk04s++os+web.browser+web.cordova/plugin.compile-coffeescript.os/npm/node_modules/meteor/caching-compiler/node_modules/async/lib/async.js:317:21)
at
/home/ubuntu/.meteor/packages/coffeescript/.1.12.7_2.6mk04s++os+web.browser+web.cordova/plugin.compile-coffeescript.os/npm/node_modules/meteor/caching-compiler/node_modules/async/lib/async.js:324:29
at
/home/ubuntu/.meteor/packages/coffeescript/.1.12.7_2.6mk04s++os+web.browser+web.cordova/plugin.compile-coffeescript.os/npm/node_modules/meteor/caching-compiler/node_modules/async/lib/async.js:44:16
at packages/caching-compiler/caching-compiler.js:316:9
at
/home/ubuntu/.meteor/packages/coffeescript/.1.12.7_2.6mk04s++os+web.browser+web.cordova/plugin.compile-coffeescript.os/npm/node_modules/meteor/caching-compiler/node_modules/async/lib/async.js:182:20
at replenish
(/home/ubuntu/.meteor/packages/coffeescript/.1.12.7_2.6mk04s++os+web.browser+web.cordova/plugin.compile-coffeescript.os/npm/node_modules/meteor/caching-compiler/node_modules/async/lib/async.js:317:21)
at
/home/ubuntu/.meteor/packages/coffeescript/.1.12.7_2.6mk04s++os+web.browser+web.cordova/plugin.compile-coffeescript.os/npm/node_modules/meteor/caching-compiler/node_modules/async/lib/async.js:324:29
at
/home/ubuntu/.meteor/packages/coffeescript/.1.12.7_2.6mk04s++os+web.browser+web.cordova/plugin.compile-coffeescript.os/npm/node_modules/meteor/caching-compiler/node_modules/async/lib/async.js:44:16
at packages/caching-compiler/caching-compiler.js:316:9
at
/home/ubuntu/.meteor/packages/coffeescript/.1.12.7_2.6mk04s++os+web.browser+web.cordova/plugin.compile-coffeescript.os/npm/node_modules/meteor/caching-compiler/node_modules/async/lib/async.js:182:20
at replenish
(/home/ubuntu/.meteor/packages/coffeescript/.1.12.7_2.6mk04s++os+web.browser+web.cordova/plugin.compile-coffeescript.os/npm/node_modules/meteor/caching-compiler/node_modules/async/lib/async.js:317:21)
at
/home/ubuntu/.meteor/packages/coffeescript/.1.12.7_2.6mk04s++os+web.browser+web.cordova/plugin.compile-coffeescript.os/npm/node_modules/meteor/caching-compiler/node_modules/async/lib/async.js:324:29
at
/home/ubuntu/.meteor/packages/coffeescript/.1.12.7_2.6mk04s++os+web.browser+web.cordova/plugin.compile-coffeescript.os/npm/node_modules/meteor/caching-compiler/node_modules/async/lib/async.js:44:16
at packages/caching-compiler/caching-compiler.js:316:9
at
/home/ubuntu/.meteor/packages/coffeescript/.1.12.7_2.6mk04s++os+web.browser+web.cordova/plugin.compile-coffeescript.os/npm/node_modules/meteor/caching-compiler/node_modules/async/lib/async.js:182:20
at replenish
(/home/ubuntu/.meteor/packages/coffeescript/.1.12.7_2.6mk04s++os+web.browser+web.cordova/plugin.compile-coffeescript.os/npm/node_modules/meteor/caching-compiler/node_modules/async/lib/async.js:317:21)
at
/home/ubuntu/.meteor/packages/coffeescript/.1.12.7_2.6mk04s++os+web.browser+web.cordova/plugin.compile-coffeescript.os/npm/node_modules/meteor/caching-compiler/node_modules/async/lib/async.js:328:15
at Object.async.forEachLimit.async.eachLimit
(/home/ubuntu/.meteor/packages/coffeescript/.1.12.7_2.6mk04s++os+web.browser+web.cordova/plugin.compile-coffeescript.os/npm/node_modules/meteor/caching-compiler/node_modules/async/lib/async.js:221:35)
at CachedCoffeeScriptCompiler.processFilesForTarget (packages/caching-compiler/caching-compiler.js:284:11)

I tried to fix it by doing:
sudo rm -rf node_modules/
sudo rm -rf ~/.meteor/packages/coffeescript
sudo rm -rf ~/.meteor/packages/caching-compiler
meteor reset
meteor update
meteor update coffeescript
meteor npm update
meteor npm install
meteor npm i -g npm

Activity

  1. benjamn commented on Oct 12, 2017

    @benjamn
    benjamn
    on Oct 12, 2017
    Contributor

    A small reproduction would be helpful!

  2. not-ai-yet commented on Oct 12, 2017

    @not-ai-yet
    not-ai-yet
    on Oct 12, 2017
    Author

    Sure, I'll get a reproduction ready. Just an update, not updating my app appears to fix it, so it seems to be something released within the last day or two since I have recently deployed.

  3. benjamn commented on Oct 12, 2017

    @benjamn
    benjamn
    on Oct 12, 2017
    Contributor

    Yes, there was a new version of coffeescript released today.

  4. benjamn commented on Oct 12, 2017

    @benjamn
    benjamn
    on Oct 12, 2017
    Contributor

    @GeoffreyBooth We might be able to go back to [email protected], since I think this validation is a new thing in [email protected]. Alternatively, if we can get a repro, we can just fix the problem directly.

  5. benjamn
    added this to the Package Patches milestone on Oct 12, 2017

  6. not-ai-yet commented on Oct 12, 2017

    @not-ai-yet
    not-ai-yet
    on Oct 12, 2017
    Author

    @benjamn then It was definitely the new coffeescript version, as you can see by the log the issue was linked to coffescript.

  7. GeoffreyBooth commented on Oct 12, 2017

    @GeoffreyBooth
    GeoffreyBooth
    on Oct 12, 2017
    Contributor

    Sure, please roll it back.

    I've been struggling with the same error in updating to 2.0.0, but I hadn't seen it in 1.12.7. I'm thinking the 2 version should use the inputSourceMap option in Babel rather than this custom source map merging.

  8. not-ai-yet commented on Oct 12, 2017

    @not-ai-yet
    not-ai-yet
    on Oct 12, 2017
    Author

    Reproduction done. Just try to build/deploy it and you'll see the error. https://github.com/TheRealNate/meteor-9217

  9. not-ai-yet
    changed the title [-]Latest version cannot deploy to galaxy due to CoffeeScript error[/-] [+]Latest version cannot deploy/build due to CoffeeScript error[/+] on Oct 12, 2017
  10. benjamn
    self-assigned this
    on Oct 12, 2017
  11. benjamn
    added
    Type:Bug
    confirmedWe want to fix or implement it
    Impact:some
    known-fix
    on Oct 12, 2017
  12. benjamn
    closed this as completedin 8b65b6eon Oct 12, 2017

  13. benjamn commented on Oct 12, 2017

    @benjamn
    benjamn
    on Oct 12, 2017
    Contributor

    If you run meteor update coffeescript you should get version 1.12.7_3, which contains a fix for this problem.

  14. not-ai-yet commented on Oct 12, 2017

    @not-ai-yet
    not-ai-yet
    on Oct 12, 2017
    Author

    Can confirm its resolved. Thank you very much!

  15. GeoffreyBooth commented on Oct 12, 2017

    @GeoffreyBooth
    GeoffreyBooth
    on Oct 12, 2017
    Contributor

    How was this not caught by the tests? They just didn’t happen to generate any source maps that failed source-map‘s validation?

  16. benjamn commented on Oct 12, 2017

    @benjamn
    benjamn
    on Oct 12, 2017
    Contributor

    I think that’s right. For what it may be worth, Babel’s test suite fails with [email protected] in the same way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Impact:someType:BugconfirmedWe want to fix or implement it

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions