Fixed a subtitle bug, added Windows support.

README.md

@@ -33,6 +33,16 @@ will currently always be mkv:
 
     dve -s .encoded.mkv -e ~/bin/ffmpeg -l host1,host2,host3 media/test.mp4
 
+dve automatically copies the local ffmpeg to remote systems for encoding, but this
+won't work if you want to mix architectures / OSes. If you ensure
+there's a valid ffmpeg install in the $PATH on each remote system, you can
+disable copying over the local encoder:
+
+    dve -d -l win1,win2,lin1,lin2 media/test.mp4
+
+This allows you to mix Windows/cygwin and Linux hosts on the same
+encoding job.
+
 ## Benchmarks
 
 Hosts used for this benchmark were dual Xeon L5520 systems with 24GB of RAM,
@@ -41,14 +51,17 @@ clip, 3:47 in length.
 
 ### ffmpeg on a single host
 
+```bash
 $ time nice -n 10 ./ffmpeg -y -v error -stats -i test.mp4 -c:v libx264 -crf 20.0 -preset medium -c:a libvorbis -aq 5 -f matroska test.mkv
 frame= 5459 fps=7.4 q=-1.0 Lsize=  530036kB time=00:03:47.43 bitrate=19091.2kbits/s    
 real    12m17.177s
 user    182m57.340s
 sys     0m36.240s
+```
 
 ### dve with 3 hosts
 
+```bash
 $ time dve -o "-c:v libx264 -crf 20.0 -preset medium -c:a libvorbis -aq 5" -l c1,c2,c3 test.mp4
 Creating chunks to encode
 
@@ -78,7 +91,7 @@ clip, 3:47 in length.
 real    6m17.075s
 user    1m29.630s
 sys     0m22.697s
-
+```
 
 ### Summary
 
@@ -112,12 +125,52 @@ on the target system.
 It's recommended that you use the latest ffmpeg statically linked binaries (above),
 which will also improve your chances of transcoding new / strange video formats.
 
+### bash
+
+If you're using the statically linked ffmpeg binaries in ~/bin,
+you'll need to ensure bash adds ~/bin to your $PATH for non-interactive shells.
+Including the following at the top of your ~/.bashrc should be sufficient:
+
+```bash
+# User dependent .bashrc file
+
+# Set PATH so it includes user's private bin if it exists
+if [ -d "${HOME}/bin" ] ; then
+  PATH="${HOME}/bin:${PATH}"
+fi
+```
+
+### Windows
+
+dve can be run on Windows via [cygwin](http://www.cygwin.com/).
+
+To do so, you'll need to:
+
+- build GNU parallel manually from source (requires make).
+- install a static build of [ffmpeg for Windows](http://ffmpeg.zeranoe.com/builds/).
+- install (or symlink) above into your $PATH, usually ~/bin.
+
+You'll also need to do the following if you want to use the host to render with:
+
+- [configure sshd](http://www.noah.org/ssh/cygwin-sshd.html)
+- alter ~/.bashrc as mentioned above.
+
 ## Restrictions
 
 - currently only generates mkv containers on output.
 
 ## ⚠ Known Issues
 
+### chunk sizing
+
+Sometimes the last chunk to be split out will be too small to be properly encoded,
+and this will cause the whole encode to fail when the pieces are joined at the end
+of the job.
+
+Currently the only solution is to adjust the -m and -c options so that the final
+chunk of the file is reasonably large.
+
+### ffmpeg / libav
 Ubuntu ships the libav fork version ffmpeg wrapper instead of the actual ffmpeg
 release. This version is missing many necessary features, including the
 [concat demuxer](https://www.ffmpeg.org/faq.html#How-can-I-concatenate-video-files_003f)

dve

@@ -1,13 +1,15 @@
 #!/bin/bash
 
 ENC="ffmpeg"
-OPTS="-c:v libx264 -crf 20.0 -preset veryslow -c:a libvorbis -aq 5"
+ENCPATH=""
+OPTS="-c:v libx264 -crf 20.0 -preset veryslow -c:a libvorbis -aq 5 -c:s copy"
 SUFFIX="_new.mkv"
 SERVERS="localhost"
 CHUNKS=100
 MINLEN=30
 OUTDIR="$HOME/.dve"
-VERBOSE="quiet"
+VERBOSE="error"
+COPYENC=true
 
 function usage() {
     cat << EOF
@@ -19,6 +21,7 @@ multiple hosts.
 OPTIONS:
     -h  this help message.
     -c  number of chunks to split input file into. (default=${CHUNKS})
+    -d  do not copy encoder, assume it already exists remotely.
     -e  avconv binary to use. (default=${ENC})
     -l  comma separated list of hosts to use to encode. (default=${SERVERS})
     -m  min length of individual video chunks, in seconds. (default=${MINLEN})
@@ -37,11 +40,13 @@ function checkpaths() {
         fi
     done
     # check as absolute path first
-    if ! [ -x "$ENC" ]; then
+    if [ -x "$ENC" ]; then
+        ENCPATH="$ENC"
+    else
         # then check for something in $PATH
-        ENC="`which $ENC`"
-        if ! [ -x "$ENC" ]; then
-            echo "$ENC not executable."
+        ENCPATH="`which $ENC`"
+        if ! [ -x "$ENCPATH" ]; then
+            echo "$ENC not found."
             exit 1
         fi
     fi
@@ -71,7 +76,7 @@ function chunksize() {
 
 checkpaths
 
-while getopts “hc:e:l:m:o:s:v” OPTION; do
+while getopts “hc:de:l:m:o:s:v” OPTION; do
     case $OPTION in
     h)
     usage
@@ -80,6 +85,9 @@ while getopts “hc:e:l:m:o:s:v” OPTION; do
     c)
     CHUNKS="$OPTARG"
     ;;
+    d)
+    COPYENC=false
+    ;;
     e)
     ENC="$OPTARG"
     ;;
@@ -96,7 +104,7 @@ while getopts “hc:e:l:m:o:s:v” OPTION; do
     SUFFIX="$OPTARG"
     ;;
     v)
-    VERBOSE="info"
+    VERBOSE="warning"
     ;;
     ?)
     usage
@@ -135,13 +143,22 @@ cat "$CHUNKFILE" | parallel --gnu --eta -j 1 $ENC -y -v ${VERBOSE} -fflags +genp
 CWD=`pwd`
 cd "$OUTDIR"
 
-cp "${ENC}" ./ffmpeg
 echo "Running parallel encoding jobs"
 PAR_OPTS="--gnu -j 1 -S ${SERVERS} --eta --retries 2 --nice 10"
-PAR_OPTS="$PAR_OPTS --workdir ... --basefile ffmpeg --trc {.}.enc"
+PAR_OPTS="$PAR_OPTS --workdir ... --trc {.}.enc"
+if $COPYENC; then
+    PAR_OPTS="$PAR_OPTS --basefile ffmpeg"
+fi
 ENC_OPTS="-y -v ${VERBOSE} -i {} ${OPTS} -f matroska {.}.enc"
 
-parallel ${PAR_OPTS} ${HOME}/ffmpeg ${ENC_OPTS} ::: chunk-*.orig
+if $COPYENC; then
+    cp "${ENCPATH}" ./ffmpeg
+    REMOTEENC='${HOME}/ffmpeg'
+else
+    REMOTEENC="$ENC"
+fi
+parallel ${PAR_OPTS} ${REMOTEENC} ${ENC_OPTS} ::: chunk-*.orig
+
 echo "Combining chunks into final video file"
 echo "ffconcat version 1.0" > concat.txt
 for f in `ls chunk-*.enc | sort`; do