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

1.2.4 fails build on nan.h #588

Closed
arvind opened this issue Jul 23, 2015 · 11 comments
Closed

1.2.4 fails build on nan.h #588

arvind opened this issue Jul 23, 2015 · 11 comments

Comments

@arvind
Copy link

arvind commented Jul 23, 2015

node-canvas v1.2.4 was just released but it's failing to build on nodejs 0.10 and 0.12 for me with the following error:

> canvas@1.2.4 install /home/travis/build/vega/vega/node_modules/vega-scenegraph/node_modules/canvas
> node-gyp rebuild
make: Entering directory `/home/travis/build/vega/vega/node_modules/vega-scenegraph/node_modules/canvas/build'
  SOLINK_MODULE(target) Release/obj.target/canvas-postbuild.node
  SOLINK_MODULE(target) Release/obj.target/canvas-postbuild.node: Finished
  COPY Release/canvas-postbuild.node
  CXX(target) Release/obj.target/canvas/src/Canvas.o
In file included from ../src/Canvas.cc:7:0:
../src/Canvas.h:22:17: fatal error: nan.h: No such file or directory
compilation terminated.
make: *** [Release/obj.target/canvas/src/Canvas.o] Error 1
make: Leaving directory `/home/travis/build/vega/vega/node_modules/vega-scenegraph/node_modules/canvas/build'
gyp ERR! build error 
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/home/travis/.nvm/v0.10.40/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:267:23)
gyp ERR! stack     at ChildProcess.emit (events.js:98:17)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (child_process.js:820:12)
gyp ERR! System Linux 3.13.0-40-generic
gyp ERR! command "node" "/home/travis/.nvm/v0.10.40/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /home/travis/build/vega/vega/node_modules/vega-scenegraph/node_modules/canvas
gyp ERR! node -v v0.10.40
gyp ERR! node-gyp -v v1.0.1
gyp ERR! not ok 
@arvind
Copy link
Author

arvind commented Jul 23, 2015

This has broken our Travis CI build (https://travis-ci.org/vega/vega/builds/72344251). I'm wondering if some additional dependencies need to be included or added to .travis.yml that have been introduced since v1.2.3?

@mathiasbynens
Copy link
Contributor

Same issue here. This breaks io.js v2.4.0 as well.

mathiasbynens added a commit to tibiamaps/tibia-maps-script that referenced this issue Jul 23, 2015
v1.2.4 was just released and is broken: Automattic/node-canvas#588
@jheer
Copy link

jheer commented Jul 23, 2015

There are discrepancies in binding.gyp between the github repo and the version published on npm. In particular, this segment appears to clobber the earlier include_dirs directive, preventing the inclusion of nan.h:

      'include_dirs': [
        '/opt/X11/include',
      ],
      'library_dirs': [
        '/opt/X11/lib',
      ],

If you remove these additions, canvas builds without issue for me. Similarly, the github version builds fine for me. Perhaps an errant edit made it into the file prior to npm publish?

@Crotery
Copy link

Crotery commented Jul 24, 2015

+1, 1.2.4 fails, 1.2.3 npm install ok

@arusakov
Copy link
Contributor

+1, node-canvas 1.2.4 fails, 1.2.3 ok
use Mac OS X 10.10.4

@frnk94
Copy link

frnk94 commented Jul 24, 2015

+1 Same issue

@paulborza
Copy link

+1

@unlucio
Copy link

unlucio commented Jul 26, 2015

+1 fails for me too

qrohlf added a commit to qrohlf/trianglify that referenced this issue Jul 26, 2015
@miguelmota
Copy link
Contributor

+1 fails

@joemaller
Copy link

Confirming, on OS X 1.2.4 fails to install, 1.2.3 installs correctly. Rolling back for now.

@LinusU
Copy link
Collaborator

LinusU commented Jul 28, 2015

This is republished as 1.2.5 now, that one should work.

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

No branches or pull requests