Tweaked makemkv rip logic to only copy back the largest created mkv file
authorAlan J. Pippin <alan@pippins.net>
Wed, 1 Feb 2012 05:31:51 +0000 (22:31 -0700)
committerAlan J. Pippin <ajp@pippins.net>
Wed, 1 Feb 2012 05:31:51 +0000 (22:31 -0700)
rip_dvd

diff --git a/rip_dvd b/rip_dvd
index af108dd424d5a7bca2c2474e6e255faf71ee7eec..0035a2d284ad8ec25a20e895a4296027cce22503 100755 (executable)
--- a/rip_dvd
+++ b/rip_dvd
@@ -1164,14 +1164,23 @@ function make_dvd_mkv_image {
        fatal_and_exit "-E- Unhandled makemkv error"
     fi
 
-    # Move the created MKV file into place
-    dstfile=${dstmkv%.[^.]*}
-    typeset -i num=0
-    for i in `/bin/ls -1 "$dstdir"/*.mkv`; do
-       echo -e "\n   Moving $i -> $dstfile.$num.mkv" | tee -a "$ddrescuelog"
-       mv "$i" "$dstfile.$num.mkv"
-       ((num = num + 1 ))
-    done
+    # Move the largest created MKV file into place
+    # There might be multiple ones, so just grab the largest one
+    largest_mkv_file=`/bin/ls -rS "$dstdir"/*.mkv | head -1`
+    echo -e "\n   Moving largest mkv file $largest_mkv_file -> $dstmkv\n" | tee -a "$ddrescuelog"
+    mv "$largest_mkv_file" "$dstmkv"
+
+    # Move all of the created MKV file into place
+    #echo "\n" | tee -a "$ddrescuelog"
+    #dstfile=${dstmkv%.[^.]*}
+    #typeset -i num=0
+    #for i in `/bin/ls -1 "$dstdir"/*.mkv`; do
+    #echo -e "   Moving $i -> $dstfile.$num.mkv" | tee -a "$ddrescuelog"
+    #mv "$i" "$dstfile.$num.mkv"
+    #((num = num + 1 ))
+    #done
+    #echo "\n" | tee -a "$ddrescuelog"
+
     if [ $? != 0 ]; then
        fatal_and_exit "-E- Unhandled mv error"
     fi