IndicoIo-node icon indicating copy to clipboard operation
IndicoIo-node copied to clipboard

LWIP installation issues

Open madisonmay opened this issue 10 years ago • 13 comments

The lwip package is prone to installation issues, should see what we can do to remedy it on our end.

madisonmay avatar Sep 22 '15 17:09 madisonmay

This should be fixed once LWIP migrates NAN to 2.0 which is what Node 4.0.0 uses. There is currently an open PR to fix this in the LWIP repo.

JoseRoman avatar Sep 27 '15 17:09 JoseRoman

Awesome. Thanks for letting us know @JoseRoman

sihrc avatar Sep 27 '15 17:09 sihrc

@madisonmay, @sihrc this issue should be fixed with the new version of LWIP 0.0.8.

JoseRoman avatar Oct 04 '15 16:10 JoseRoman

@JoseRoman Awesome, thank you!

sihrc avatar Oct 04 '15 18:10 sihrc

Is this the issue that I am running into?

s4_1 | Error: Cannot find module '../build/Release/lwip_image' s4_1 | at Function.Module._resolveFilename (module.js:337:15) s4_1 | at Function.Module._load (module.js:287:25) s4_1 | at Module.require (module.js:366:17) s4_1 | at require (module.js:385:17) s4_1 | at /code/node_modules/indico.io/node_modules/lwip/lib/Image.js:3:22 s4_1 | at Object. (/code/node_modules/indico.io/node_modules/lwip/lib/Image.js:16:3) s4_1 | at Module._compile (module.js:435:26) s4_1 | at Object.Module._extensions..js (module.js:442:10) s4_1 | at Module.load (module.js:356:32) s4_1 | at Function.Module._load (module.js:311:12) s4_1 | at Module.require (module.js:366:17) s4_1 | at require (module.js:385:17) s4_1 | at /code/node_modules/indico.io/node_modules/lwip/lib/ImagePrototypeInit.js:3:17 s4_1 | at Object. (/code/node_modules/indico.io/node_modules/lwip/lib/ImagePrototypeInit.js:620:3) s4_1 | at Module._compile (module.js:435:26) s4_1 | at Object.Module._extensions..js (module.js:442:10) s4_1 | at Module.load (module.js:356:32) s4_1 | at Function.Module._load (module.js:311:12) s4_1 | at Module.require (module.js:366:17) s4_1 | at require (module.js:385:17) s4_1 | at Object. (/code/node_modules/indico.io/node_modules/lwip/index.js:1:63) s4_1 | at Module._compile (module.js:435:26) s4_1 | at Object.Module._extensions..js (module.js:442:10) s4_1 | at Module.load (module.js:356:32) s4_1 | at Function.Module._load (module.js:311:12) s4_1 | at Module.require (module.js:366:17) s4_1 | at require (module.js:385:17) s4_1 | at Object. (/code/node_modules/indico.io/lib/image.js:1:74)

package.json below...

{ "name": "indico.io", "description": "A Node.js wrapper for the Indico’s API", "version": "0.4.6", "homepage": "https://github.com/IndicoDataSolutions/IndicoIo-node", "author": { "name": "Joseph Núñez", "email": "[email protected]" }, "contributors": [ { "name": "Madison May", "email": "[email protected]" } ], "keywords": [], "repository": { "type": "git", "url": "git://github.com/IndicoDataSolutions/IndicoIo-node.git" }, "bugs": { "url": "https://github.com/IndicoDataSolutions/IndicoIo-node/issues" }, "licenses": [ { "type": "MIT", "url": "https://github.com/IndicoDataSolutions/IndicoIo-node/blob/master/LICENSE" } ], "main": "./lib/indico", "dependencies": { "bluebird": "^2.9.24", "config-ini": "^0.2.2", "expand-tilde": "^1.2.0", "file-type": "^2.11.0", "lwip": "0.0.8", "request": "^2.36.0", "valid-url": "^1.0.9" }, "devDependencies": { "should": "~3.0.1", "mocha": "~1.17.0", "chai": "1.9.1" }, "scripts": { "test": "mocha test/.js test/integration/.js" }, "js-flags": "--harmony", "_id": "[email protected]", "_shasum": "f6fb1d899427bfe50803b9580b1f8d89935c73be", "_resolved": "https://registry.npmjs.org/indico.io/-/indico.io-0.4.6.tgz", "_from": "indico.io@>=0.4.6 <0.5.0", "_npmVersion": "2.11.2", "_nodeVersion": "4.2.0", "_npmUser": { "name": "madisonmay", "email": "[email protected]" }, "dist": { "shasum": "f6fb1d899427bfe50803b9580b1f8d89935c73be", "tarball": "http://registry.npmjs.org/indico.io/-/indico.io-0.4.6.tgz" }, "maintainers": [ { "name": "slatervictoroff", "email": "[email protected]" }, { "name": "madisonmay", "email": "[email protected]" }, { "name": "indicodatasolutions", "email": "[email protected]" } ], "directories": {} }

meticulo3366 avatar Nov 23 '15 21:11 meticulo3366

perhaps we can use a precompiled binary option?

https://github.com/EyalAr/lwip/issues/83

meticulo3366 avatar Nov 23 '15 22:11 meticulo3366

And it is especially difficult to deploy this on AWS Lambda due to lwip compiling/installation issues...

xraywu avatar Feb 03 '16 08:02 xraywu

+1 for Google Could Compute instances as well.

pondre-brandon avatar Apr 30 '16 04:04 pondre-brandon

I've created a fork of this that removes all the image processing, as we didn't need it in our current project and were having similar installation issues. Could this be broken up into a separate indico.io-text and indico.io-image package that the main indico.io package references?

Didericis avatar May 25 '16 17:05 Didericis

Same error here. I am unable to install. The lwip package throws a build error. Any news, when this can be fixed?

3 warnings and 4 errors generated.
make: *** [Release/obj.target/lwip_decoder/src/decoder/init.o] Error 1
gyp ERR! build error 

suissemaxx avatar Oct 16 '16 19:10 suissemaxx

LWIP is currently an optional dependency. If you are not working with images, a current workaround would be to install the package without the optional dependencies. http://unix.stackexchange.com/a/256960

sihrc avatar Oct 16 '16 19:10 sihrc

I believe I've reproduced the error with: node -> v6.8.1 npm -> 3.3.8

A new 0.10.4 indico.io package has been deployed in #123 that updates the LWIP dependency which fixes the error that was reproduced.

Thanks to everyone who reported this bug, and sorry for not addressing this sooner.

sihrc avatar Oct 16 '16 20:10 sihrc

We'll leave this issue open until a more sustainable solution is implemented. Until then, if anyone begins to have issues with LWIP installation again, we will again update the lib as necessary.

sihrc avatar Oct 16 '16 20:10 sihrc