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

add from/to, fix $speed, rm $temp #36

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

add from/to, fix $speed, rm $temp #36

wants to merge 1 commit into from

Conversation

jgreely
Copy link

@jgreely jgreely commented Oct 8, 2016

bug: the logic for parsing $speed was backwards.
bug: temp file not removed on exit

enhancement: added -f and -t options to extract short clips from a long video

bug: the logic for parsing $speed was backwards.
bug: temp file not removed on exit

enhancement: added -f and -t options to extract short clips from a long video
speed=1
fi

if [ ! -z "$to" ]; then
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You'll need to add the to= and from= vars, otherwise you'll get:

/usr/local/bin/gifify: line 104: to: unbound variable
/usr/local/bin/gifify: line 104: temp: unbound variable

speed=1
fi

if [ ! -z "$to" ]; then
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See @STRML's comment

@@ -16,12 +17,24 @@ function printHelpAndExit {
echo ' below approximately 60.'
echo ' p SCALE: Rescale the output, e.g. 320:240'
echo ' x: Remove the original file and resulting .gif once the script is complete'
echo ' f from: The start time of the clip, in seconds'
echo ' t to: The end time of the clip, in seconds'
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These should be formatted as f FROM and t TO to indicate flags vs values

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

Successfully merging this pull request may close these issues.

3 participants