CoreJS build fails during deploy about meteor-webpack-react

[jedwards1211]

Huh, I don't think it's environment variables, maybe it failed to npm install within the core-js folder. In any case I need to investigate if the core-js custom build is even necessary anymore, since Meteor now has some builtin ES2015 support they may have fixed the Number constructor issue.

[jthomaschewski]

By looking at the error message I would think the same. But When running npm install and then node dev.js the corejs build runs fine.
So this happens only when using deploy.js instead of dev.js - on the same environment with the git repository cleaned before running the script.

[jedwards1211]

I'm still getting the match errors when doing something like collection.find({}, {limit: 3}) due to the Number constructor polyfill with the latest version of Meteor...which would mean they're doing something weird to support it with their ES2015 plugin that doesn't also apply to externally transpiled code.
I should log an issue in their tracker about this...

[jedwards1211]

Have you tried this on clean install? This would certainly indicate that the npm install call from core-js-custom-build.js isn't working on your CI server

> npm install
> cd node_modules/core-js
> npm install
> cd ../..
> node deploy.js meteor.com

[jthomaschewski]

Yes, this happens on a clean install of the current master of this skeleton.
And not only on my CI server (Debian) but also on my local machine (Arch Linux) - both node 4.1 and npm 3.
Running npm install manually on the core-js folder as you suggested helped. So there seems to be a problem with the "core-js-custom-build.js" call - but only in deploy.js. As it's called the same way there must be some difference somewhere else...

[jedwards1211]

Weird. I do need to start using the latest versions of Node. I wonder if shelljs can cause any problems...I've noticed that exec calls swallow dynamic things printed to the terminal, like download progress meters that would show when running the core-js custom build via bash script.
I may not have much more time leftover to work on this project this week...but feel free to make a PR if you figure out what's wrong

[tomitrescak]

Pardon my off topic. But which version of node is recommended? I am using 0.10.40 for compatibility issues with Meteor.

[jthomaschewski]

I'm using latest node 4.1.2 and npm 3.3.5 and don't have any issues.

jedwards1211 wrote:

Weird. I do need to start using the latest versions of Node

I assume this skeleton is tested with older versions of node

@tomitrescak what kind of compatibility issues do you have with meteor? I thought the meteor install includes node and is in independed from the system wide installed version.

[tomitrescak]

@JBBr Not even Node 0.12.x was recommended for meteor. If you use MUP or some other deploy tools, the 0.10.40 is the latest they install. I did not try to discover why is that, just accepted it as a fact.

[jthomaschewski]

@tomitrescak Yes that's true for running meteor bundles directly with node (e.g. using meteor bundle or demeteorizer).
But in the skeleton the meteor command is executed in the background. Node is only used to run some scripts and webpack/webpack-dev-server.

The meteor server is still using the included node version (maybe still 0.10.x, maybe updated in Meteor 1.2)

[jthomaschewski]

Back to topic @jedwards1211:

I've found a way to fix the problem. It does only occur if core-js-custom-build is required inside a exported function (module.exports).
If it's required before module.exports build runs fine.

I'll do a PR