-
Notifications
You must be signed in to change notification settings - Fork 314
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
Suggestion: throw error when resize option is not present #60
Comments
Hi @chasegiunta --resize is not always needed strangely it seems. Only in some cases, would you mind digging a bit more to see why? |
Unfortunately I'm not skilled enough to do so, ha. I can provide the video files (.mp4) if that is of some use. |
Yes drop them here so that someone can have a look. I have really no idea on what's going on |
Apologies, it looks like it does create a temp file... But it will continue to run indefinitely (I think, I didn't leave it go to because it quickly ate up 5GB for this 7mb .mp4) https://cl.ly/0F3C2S3G1a1I |
Maybe there's an underlying bug somewhere in the dependencies stack |
As it seems that the
--resize
option is a requirement (seen in my experience and #23) , maybe throw an error if the parameter is not present in command. Unless this is a bug? Gifify will continue to run indefinitely, and not create a write file in location, if not the--resize
option is not definedThe text was updated successfully, but these errors were encountered: