galkahana / HummusJS

Node.js module for high performance creation, modification and parsing of PDF files and streams
http://www.pdfhummus.com
Other
1.14k stars 169 forks source link

warning "hummus@1.0.108" is missing a bundled dependency "node-pre-gyp". #444

Open PaulMest opened 4 years ago

PaulMest commented 4 years ago

Got this in my latest yarn install:

warning "hummus@1.0.108" is missing a bundled dependency "node-pre-gyp". This should be reported to the package maintainer.

Os: macOS Catalina Node: v12.13.0

linusphan commented 4 years ago

I'm getting the same warning. Could we resolve this issue?

DanielRuf commented 4 years ago

Hi @galkahana,

are you looking for new maintainers to take over the project or is it best if I create a fork?

quinn-diesel commented 4 years ago

did anyone figure out a solution to this issue?

DanielRuf commented 4 years ago

You can optionally add / install node-pre-gyp.

Kodiakkb commented 4 years ago

Just do an npm install

DanielRuf commented 4 years ago

Just do an npm install

This is the same like yarn install.

quinn-diesel commented 4 years ago

sweet cheers guys. forgot to post but we ended up bumping our node version back to 10.0.17 and that did the trick for now.

DanielRuf commented 4 years ago

This is just a workaround, not a longterm solution.

DanielRuf commented 4 years ago

Seems this is a known problem. It has to be bundled with the release or added as normal dependency.

Felipedelima123 commented 4 years ago

I'm having the same problem. Started happening after change the servers and upgrade the NodeJS version of the Stable 12.16.1 for 13.10.1. As @DanielRuf said we need to find a long term support fix for this, the author will not help, because it end support.

Any ideas?

IMarty commented 4 years ago

Thx @Felipedelima123, you just saved my project... definilty not the best solution but will do the trick for some months... Somebody need to do a fork and manage at least the dependencies :) I fixed it by running that command npm install node@12.16.1--save-exact

PaulMest commented 3 years ago

It seems like this fork https://github.com/julianhille/muhammarajs at least builds with Node v14 and does not show a warning. I have no affiliation with the project or author. So I don't know the quality of the fork or how maintained that fork will be over time.

PradeepNooney commented 3 years ago

hello , Build failed after upgrading node from 14.x to 15.x (docker - ubuntu) getting below error. Can anyone please help me in this.

17 191.0 npm ERR! V8_WARN_UNUSED_RESULT Maybe Set(Local context,

17 191.0 npm ERR! ^~~

17 191.0 npm ERR! In file included from ../src/hummus.cpp:23:0:

17 191.0 npm ERR! ../src/nodes.h:93:51: warning: ignoring return value of 'v8::Maybe v8::Object::Set(v8::Local, v8::Local, v8::Local)', declared with attribute warn_unused_result [-Wunused-result]

17 191.0 npm ERR! #define EXPORTS_SET(e,k,v) e->Set(context, k,v);

17 191.0 npm ERR! ^

17 191.0 npm ERR! ../src/hummus.cpp:645:5: note: in expansion of macro 'EXPORTS_SET'

17 191.0 npm ERR! EXPORTS_SET(exports,NEW_SYMBOL("EInfoTrappedFalse"),NEW_NUMBER(EInfoTrappedFalse))

17 191.0 npm ERR! ^~~

17 191.0 npm ERR! In file included from /root/.cache/node-gyp/15.14.0/include/node/node.h:63:0,

17 191.0 npm ERR! from ../src/hummus.cpp:20:

17 191.0 npm ERR! /root/.cache/node-gyp/15.14.0/include/node/v8.h:3716:37: note: declared here

17 191.0 npm ERR! V8_WARN_UNUSED_RESULT Maybe Set(Local context,

17 191.0 npm ERR! ^~~

17 191.0 npm ERR! In file included from ../src/hummus.cpp:23:0:

17 191.0 npm ERR! ../src/nodes.h:93:51: warning: ignoring return value of 'v8::Maybe v8::Object::Set(v8::Local, v8::Local, v8::Local)', declared with attribute warn_unused_result [-Wunused-result]

17 191.0 npm ERR! #define EXPORTS_SET(e,k,v) e->Set(context, k,v);

17 191.0 npm ERR! ^

17 191.0 npm ERR! ../src/hummus.cpp:646:5: note: in expansion of macro 'EXPORTS_SET'

17 191.0 npm ERR! EXPORTS_SET(exports,NEW_SYMBOL("EInfoTrappedUnknown"),NEW_NUMBER(EInfoTrappedUnknown))

17 191.0 npm ERR! ^~~

17 191.0 npm ERR! In file included from /root/.cache/node-gyp/15.14.0/include/node/node.h:63:0,

17 191.0 npm ERR! from ../src/hummus.cpp:20:

17 191.0 npm ERR! /root/.cache/node-gyp/15.14.0/include/node/v8.h:3716:37: note: declared here

17 191.0 npm ERR! V8_WARN_UNUSED_RESULT Maybe Set(Local context,

dietrichg commented 1 year ago

Any update?