-
Notifications
You must be signed in to change notification settings - Fork 231
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
gifsicle should maybe not exit with status 1 when valid output is produced #89
Comments
I think these particular errors can be characterized as warnings. If you find other ones, let me know what they are. Thanks! |
@kohler what about too many background color, it also exits with status 1.
one source gif: http://cdn.fatu.me/liZRzg1BORgSqoLNtemNf3-u3W_X |
@kohler Can you release new version with this fix. It's causing lot of problems. |
I use a kinda popular image server called Thumbor and it use gifsicle as an optional "engine" for doing all the gif things. Which is great.
However, sometimes people will upload gifs with something that gifsicle doesn't like ... so a
1
exit status is sent. Thumbor assumes if gifsicle exits with1
that the output is invalid and will throw the result away.Turns out that the output in at least the case I am test for is actually completely fine. Is there a way that gifsicle could differentiate between invalid output and just some error that didn't gifsicle from doing it's fine work?
Here is a crude example:
source gif here (sportsball)
It might also be helpful to label the error message with "error:", otherwise it sorta just looks like info/debug level stuff.
Thanks!
The text was updated successfully, but these errors were encountered: