Skip to content
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

Cesium does not build with npm 1.3.10 #4080

Closed
gberaudo opened this issue Jul 4, 2016 · 3 comments
Closed

Cesium does not build with npm 1.3.10 #4080

gberaudo opened this issue Jul 4, 2016 · 3 comments

Comments

@gberaudo
Copy link
Contributor

gberaudo commented Jul 4, 2016

There is no compilation error, but the build only takes half a minute and Cesium.js is missing from Build/Cesium. The build guide does not mention a minimal version of npm.

With npm 1.3.10

$ npm run minifyRelease

> [email protected] minifyRelease /home/gberaudo/cesium
> gulp minifyRelease

[08:31:22] Using gulpfile ~/cesium/gulpfile.js
[08:31:22] Starting 'build'...
[08:31:23] Finished 'build' after 1.25 s
[08:31:23] Starting 'generateStubs'...
[08:31:24] Finished 'generateStubs' after 268 ms
[08:31:24] Starting 'minifyRelease'...
Building Cesium.js
Building Source/Workers/cesiumWorkerBootstrapper.js
Finished Cesium.js
Finished Source/Workers/cesiumWorkerBootstrapper.js
Building Source/ThirdParty/Workers/inflate.js
Finished Source/ThirdParty/Workers/inflate.js
...
[08:32:00] Finished 'minifyRelease' after 36 s

Then
$ ls Build/Cesium/
Assets Widgets

With npm 2.15.5

$ npm run minifyRelease

> [email protected] minifyRelease /tmp/cesium
> gulp minifyRelease

[08:43:33] Using gulpfile /tmp/cesium/gulpfile.js
[08:43:33] Starting 'build'...
[08:43:33] Finished 'build' after 339 ms
[08:43:33] Starting 'generateStubs'...
[08:43:33] Finished 'generateStubs' after 46 ms
[08:43:33] Starting 'minifyRelease'...
Building Cesium.js
Building Source/Workers/cesiumWorkerBootstrapper.js
Building Source/Workers/transferTypedArrayTest.js
Building Source/ThirdParty/Workers/deflate.js
Building Source/ThirdParty/Workers/inflate.js
Finished Source/Workers/transferTypedArrayTest.js
Finished Source/Workers/cesiumWorkerBootstrapper.js
Finished Source/ThirdParty/Workers/deflate.js
Finished Source/ThirdParty/Workers/inflate.js
Building Source/Workers/combineGeometry.js
Building Source/Workers/createBoxGeometry.js
Building Source/Workers/createBoxOutlineGeometry.js
Building Source/Workers/createCircleGeometry.js
...
@hpinkos
Copy link
Contributor

hpinkos commented Jul 5, 2016

Thanks @gberaudo! We all use node version 4.2.2

@mramato is this a bug or do we need to specify a version of Node in the build guide?

@mramato
Copy link
Contributor

mramato commented Jul 5, 2016

We should specify Node 4.x or newer in the build guide. All earlier versions are soon to be end-of-life.

@gberaudo
Copy link
Contributor Author

gberaudo commented Jul 5, 2016

Thanks

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

No branches or pull requests

3 participants