-
Notifications
You must be signed in to change notification settings - Fork 65
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
fails inside a docker container #57
Comments
i updated npm inside the container and re-installed imagemin dep and it worked thx |
it only works if i manually install imagemin inside the container i will have to include that in the dockerfile do you think of another better solution? |
Never worked with imagemin using docker so unfortunately, no. |
@dciccale Hmm, I hit the same issue with docker... I tried to manually install imagemin in the container but didn't have any luck with getting past the newline unexpected issue. What did you do exactly? I tried |
@treejanitor i am using gulp-imagemin which uses this module, what I did in my Dockerfile is |
@dciccale Thanks - I'm using grunt, but hopefully it'll be the same... I'll give it a shot. |
@dciccale I couldn't get this to work for the life of me... the issue is inside |
@treejanitor did you figured out why it was failing ? I also experiencing this issue with a grunt app |
I was able to workaround this issue by switching to the node:6-wheezy container. seems that by default, node:6 was built using the jessie ubuntu. This package appears to have issues running on that flavor |
@manchuck you saved my day! Thanks for the wheezy tip, worked like a charm. |
@kevinkarwaski Seems that jessie ubuntu is using an ARM processor. Something compiled along the way is causing issues. Still have not gone through to figure out why that could be causing this issue. |
thanks breh |
I just ran into this issue. I discovered that my docker setup was copying compiled npm binaries from my host to the guest container. However, you cannot guarantee that a container's cpu architecture will match that of your host. So, to avoid this issue, I would recommend adding |
fails inside a docker container created from
node:latest
The text was updated successfully, but these errors were encountered: