search

andrewrk / juice

Juice inlines CSS stylesheets into your HTML source.
MIT License
60 stars 13 forks source link

Juice not working in Node.js v0.12.0 #26

Open bmcminn opened 9 years ago

bmcminn commented 9 years ago

Mac OSx 10.9.5 64bit Node: v0.12.0 Npm: 2.5.1

Installing juice results in the follow messaging:

> $ npm install grunt-juice-email --save-dev
npm WARN package.json seismic-audio@2.1.0 No repository field.

> contextify@0.1.13 install /Users/username/project-path/node_modules/grunt-juice-email/node_modules/juice/node_modules/jsdom/node_modules/contextify
> node-gyp rebuild

child_process: customFds option is deprecated, use stdio instead.
  CXX(target) Release/obj.target/contextify/src/contextify.o
  SOLINK_MODULE(target) Release/contextify.node
  SOLINK_MODULE(target) Release/contextify.node: Finished
grunt-juice-email@0.1.1 node_modules/grunt-juice-email
└── juice@0.4.0 (batch@0.3.2, cssom@0.2.5, slick@1.10.4, commander@1.1.1, superagent@0.14.9, jsdom@0.6.5)

Seems this my be an issue upstream in contextify where it hasn't fully supported the newer Node versions above 0.10.12... This may be related to #16

EDIT: Determined that the install works in Node versions up to 0.11.12, after that there seems to be some conflict or API issue that is no longer supported.

afloyd commented 9 years ago

I got the above error too when installingon windows 8 w/ node 0.12

Then if you try to run it anyway I get the error:

TypeError: Cannot read property 'removeChild' of null] 'TypeError: Cannot read property \'removeChild\' of null\n    at removeStyleTags (...\\jt-mailer\\node_modules\\juice2\\lib\\juice.js:272:32)