-
Notifications
You must be signed in to change notification settings - Fork 3
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
Comments
@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 |
In the meantime one can work around the issue with the pngcrusher binary |
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, |
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.
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:
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.
The text was updated successfully, but these errors were encountered: