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

LWIP installation issues #67

Open
madisonmay opened this issue Sep 22, 2015 · 13 comments
Open

LWIP installation issues #67

madisonmay opened this issue Sep 22, 2015 · 13 comments

Comments

@madisonmay
Copy link
Contributor

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

@JoseRoman
Copy link
Contributor

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.

@sihrc
Copy link
Member

sihrc commented Sep 27, 2015

Awesome. Thanks for letting us know @JoseRoman

@JoseRoman
Copy link
Contributor

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

@sihrc
Copy link
Member

sihrc commented Oct 4, 2015

@JoseRoman Awesome, thank you!

@meticulo3366
Copy link

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": "toctochello@gmail.com"
},
"contributors": [
{
"name": "Madison May",
"email": "madison@indico.io"
}
],
"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": "indico.io@0.4.6",
"_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": "madison@indico.io"
},
"dist": {
"shasum": "f6fb1d899427bfe50803b9580b1f8d89935c73be",
"tarball": "http://registry.npmjs.org/indico.io/-/indico.io-0.4.6.tgz"
},
"maintainers": [
{
"name": "slatervictoroff",
"email": "slater@indico.io"
},
{
"name": "madisonmay",
"email": "madison@indico.io"
},
{
"name": "indicodatasolutions",
"email": "contact@indico.io"
}
],
"directories": {}
}

@meticulo3366
Copy link

perhaps we can use a precompiled binary option?

EyalAr/lwip#83

@xraywu
Copy link

xraywu commented Feb 3, 2016

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

@pondre-brandon
Copy link

+1 for Google Could Compute instances as well.

@Didericis
Copy link

Didericis commented May 25, 2016

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?

@suissemaxx
Copy link

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 

@sihrc
Copy link
Member

sihrc commented Oct 16, 2016

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
Copy link
Member

sihrc commented Oct 16, 2016

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
Copy link
Member

sihrc commented Oct 16, 2016

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.

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

No branches or pull requests

8 participants