Add video groups

Signed-off-by: James Ketrenos <jketreno@media.ketrenos.com>

video-convert

@@ -2,6 +2,19 @@
 # Look into HW video encoding to vp9 instead of h264:
 # https://www.reddit.com/r/VP9/comments/g9uzzv/hardware_encoding_vp9_on_intel/
 
+fail() {
+  echo "$*" >&2
+  exit -1
+}
+
+VIDEO=$(getent group video | sed -E 's,^video:[^:]*:([^:]*):.*$,\1,')
+RENDER=$(getent group render | sed -E 's,^render:[^:]*:([^:]*):.*$,\1,')
+
+[[ "${VIDEO}" != "" ]] || fail "No video group found."
+
+ADD_GROUPS="--group-add ${VIDEO}"
+[[ "${RENDER}" != "" ]] && ADD_GROUPS+=" --group-add ${RENDER}"
+
 video_detect() {
   for file in "${*}"; do
     INFO=($(ffprobe -v error \
@@ -39,8 +52,8 @@ function move {
   base="$(dirname "${IN}")"
   [ ! -d "${base}" ] && continue
   base=/remote/media/backup/"${base}"
-  [ ! -d "${base}" ] && mkdir -p "${base}"
+  [ ! -d "${base}" ] && mkdir -p "${base}" || fail "Unable to mkdir '${base}'"
-  [ -e "${base}" ] && mv "${IN}" "${base}"/
+  [ -e "${base}" ] && mv "${IN}" "${base}"/ || fail "Unable to move '$IN'"
 }
 
 function convert {
@@ -86,7 +99,7 @@ function convert {
   fi
 
   echo -n "Counting frames in ${IN}: "
-  FRAMES=$(ffprobe -v error -select_streams v:0 -count_packets -show_entries stream=nb_read_packets -of csv=p=0 "${IN}")
+  FRAMES=$(ffprobe -v error -select_streams v:0 -count_packets -show_entries stream=nb_read_packets -of csv=p=0 "${IN}") || fail "Unable to count frames"
   echo "${FRAMES} frames."
 
   # Use stdbuf to flush stdout/stderr every line
@@ -97,16 +110,26 @@ function convert {
   OUT=$(realpath "${OUT}")
   DIN=$(dirname "${IN}")
   DOUT=$(dirname "${OUT}")
-  NO_COLOR=1 \
+  if false; then
-    docker run --user=1000 --device=/dev/dri \
+    quiet="
+      -v quiet
+      -loglevel fatal
+    "
+  else
+    quiet=""
+  fi
+
+  docker run \
+    --device=/dev/dri \
+    --user=$(id -u) \
     --rm \
+    ${ADD_GROUPS} \
     -v "${DIN}:${DIN}" \
     -v "${DOUT}:${DOUT}" \
     intel-media-ffmpeg \
     ffmpeg \
+    ${quiet} \
       -nostdin \
-      -v quiet \
-      -loglevel fatal \
     ${input_flags} \
       -i "${IN}" \
       -progress /dev/stdout \
@@ -114,12 +137,23 @@ function convert {
     ${output_flags} \
       -movflags +faststart \
     "${OUT}" \
-      -y | while read line; do
+      -y || fail "ffmpeg failed" 2>&1 |
+    while read line; do
+      if [[ "${line}" == "ffmpeg failed" ]]; then
+        false
+        break
+      fi
       POS=$(echo $line | sed -n 's/^frame=*\(.*\)/\1/p')
-      if [[ "${POS}" != "" ]]; then
+      if [[ "${POS}" == "" ]]; then
+        continue
+      fi
+
       NOW=$(date +%s)
       ELAPSEDTIME=$(( NOW - LASTTIME ))
-        if (( ELAPSEDTIME > 5 )); then
+      if (( ELAPSEDTIME <= 5 )); then
+        continue
+      fi
+
       ELAPSEDFRAMES=$(( POS - LASTPOS ))
       REMAININGFRAMES=$(( FRAMES - POS ))
       REMAININGFRAMES=$(( REMAININGFRAMES * 1000 )) # convert to ms
@@ -132,20 +166,16 @@ function convert {
       fi
       LASTTIME=${NOW}
       LASTPOS=${POS}
-        fi
-      fi
     done && move && {
       if [[ "${ORIG}" == "${IN}" ]]; then
         # If the original file was an '.mkv' then OUT was '.transcoded.mkv'
         # during transcode. Now that the transcode is complete, set the
         # file back to the original name.
-        mv "${OUT}" "${IN}"
+        mv "${OUT}" "${IN}" || fail "Unable to mv '${OUT}' -> '${IN}'"
       fi
     }
     NOW=$(date +%s)
     echo -e "\nCompleted in $(mstodate $(( NOW - STARTTIME ))000)"
-#    -vcodec copy \
-#    -acodec aac \
 }
 
 function check_and_convert {