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

pngcrush-bin dependency outdated #9

Closed
mechanicalduck opened this issue Sep 13, 2014 · 3 comments
Closed

pngcrush-bin dependency outdated #9

mechanicalduck opened this issue Sep 13, 2014 · 3 comments

Comments

@mechanicalduck
Copy link

The pngcrush-bin dependency in package.json is outdated.
npm module node-grunt is also affected as it got npm module (node-)iconizr as depenceny.

The outdated pngcrush-bin expects a specific GLIBC library which is not present on many systems and fail, also the build source url is incorrect.

[...]
Creating PNG versions of 2 SVG images (including the sprite) ...
{ [Error: Command failed: [...]/node_modules/iconizr/node_modules/pngcrush-bin/vendor/pngcrush: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.14' not found (required by [...]/node_modules/iconizr/node_modules/pngcrush-bin/vendor/pngcrush)] killed: false, code: 1, signal: null }

Done, without errors.
[...]

Although no error was indicated in the end of the task, only one PNG was generated, the others missing and the style files weren't written.

This can be solved however by forcing iconizr to use the latest stable pngcrush-bin dependency:

(cwd is project dir)
$ rm -r ./node_modules/iconizr/node_modules/pngcrush-bin/
$ npm install pngcrush-bin
Creating PNG versions of 2 SVG images (including the sprite) ...
{ [Error: Command failed: ] killed: false, code: 1, signal: null }

The pngcrush-bin pngcrush command would work again, the PNGs are created -
however, there still seems to be at least some misinterpretation of its exit code as the command is remarked as 'failed' during iconizr run and the subsequent css/sass/stylus/... files aren't written.

So bumping the pngcrush-bin dependency to latest stable + adjusting its invocation in iconizr will solve this issue.

Thank you in advance.
With best regards.

@jkphl
Copy link
Owner

jkphl commented Sep 13, 2014

@mechanicalduck Thanks for this great report! pngcrush has always been problematic for a lot of people, so this might really help. Will try to find some time soon to update the dependency. Thanks again, cheers, Joschi

@mechanicalduck
Copy link
Author

In the meantime one can work around the issue with the pngcrusher binary
by rebuilding it: imagemin/pngcrush-bin#6

jkphl added a commit that referenced this issue Jan 2, 2015
@jkphl
Copy link
Owner

jkphl commented Jan 2, 2015

Hey @mechanicalduck,

I know a lot of time has passed — thanks for your patience! I just updated the dependencies and released a new version. Could you please check if this works for you (again)? I'll close the issue for now, but please feel free to re-open in case of any further problems.

I'll start rewriting iconizr from scratch during the next days, so stay tuned in any case.

Besides that, Happy New Year! :)

Cheers,
Joschi

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

No branches or pull requests

2 participants