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.

Sign up for GitHub

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

Linux tweaks #206

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
+27 −10
Open

Linux tweaks #206

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
7af8bc1
add variable substitution for mac/linux
privatezero Nov 27, 2017
5a71093
tweak to tweak
privatezero Nov 27, 2017
883ed2f
change file query
privatezero Nov 27, 2017
d5505da
make system check more specific
privatezero Dec 5, 2017
749309e
make guidance more explicit
privatezero Dec 6, 2017
81400eb
fixes case difference in command
privatezero Dec 8, 2017
115088f
unset VIDEOCHECK
privatezero Dec 8, 2017
f9878b4
universalize command
privatezero Dec 11, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion searchfingerprint
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ while [ "${*}" != "" ] ; do
shift

#Confirm input is a video file
if [[ -z $(file -Ib "${INPUT}" | grep video) ]] ; then
if [[ -z $(file -i "${INPUT}" | grep video) ]] ; then
Copy link
Collaborator

Choose a reason for hiding this comment

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

This breaks for me. Maybe using ffmpeg is safer.

file -i TEST.mov 
/Users/daverice/Desktop/TEST.mov: regular file

vs

file -Ib TEST.mov 
video/quicktime; charset=binary

Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe using ffmpeg is safer.

+1

echo "Input is not a video file" && continue
fi

Expand Down