strongloop / modern-syslog

modern-syslog
Other
49 stars 19 forks source link

Build problems on SmartOS (illumos distro) #10

Closed johngrasty closed 8 years ago

johngrasty commented 8 years ago

Node newb here. Sorry for not knowing much, but I am unable to get this to install on recent SmartOS. I will be happy to provide any information necessary to debug.

Node version: 4.1.2 npm version: 2.14.4

Here is the compile error:

npm install -g modern-syslog
-
> modern-syslog@1.1.1 install /opt/local/lib/node_modules/modern-syslog
> node-gyp rebuild

make: Entering directory '/opt/local/lib/node_modules/modern-syslog/build'
  CXX(target) Release/obj.target/core/core.o
../core.cc: In function 'void {anonymous}::Init(Nan::ADDON_REGISTER_FUNCTION_ARGS_TYPE)':
../core.cc:131:5: error: 'LOG_PERROR' was not declared in this scope
core.target.mk:92: recipe for target 'Release/obj.target/core/core.o' failed
make: *** [Release/obj.target/core/core.o] Error 1
make: Leaving directory '/opt/local/lib/node_modules/modern-syslog/build'
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/opt/local/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:270:23)
gyp ERR! stack     at emitTwo (events.js:87:13)
gyp ERR! stack     at ChildProcess.emit (events.js:172:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:200:12)
gyp ERR! System SunOS 5.11
gyp ERR! command "/opt/local/bin/node" "/opt/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /opt/local/lib/node_modules/modern-syslog
gyp ERR! node -v v4.1.2
gyp ERR! node-gyp -v v3.0.3
gyp ERR! not ok
npm ERR! SunOS 5.11
npm ERR! argv "/opt/local/bin/node" "/opt/local/bin/npm" "install" "-g" "modern-syslog"
npm ERR! node v4.1.2
npm ERR! npm  v2.14.4
npm ERR! code ELIFECYCLE
rmg commented 8 years ago

Looks like a real bug to me, I'll submit a pull request for it :-)

Unrelated to it, however.. I can't think of a scenario where you would want to install this particular module globally. It should be an app dependency, or a dependency of a globally installed module.

johngrasty commented 8 years ago

@rmg Incidentally, you are right. I ran into the error while trying to install Haraka. But I was getting ready to walk out of the door, and I didn't have the compile errors handy.

Thanks for the quick response and help!

rmg commented 8 years ago

@johngrasty release modern-syslog@1.1.2, can you update and confirm it works for you now?

johngrasty commented 8 years ago

@rmg Confirmed. I just installed Haraka, and modern-syslog@1.1.2 installed just fine. Thanks!