vidi-player.py: check that the output file exists before creating the pipeline

[vidi-player.git] / vidi-player.py

diff --git a/vidi-player.py b/vidi-player.py
index dbd5a75..303d55d 100755 (executable)
--- a/vidi-player.py
+++ b/vidi-player.py
@@ -65,9 +65,9 @@ def timeline_from_midi(midi_file, video_font_path):
             print("Note name: %s start_time: %f duration: %f" %
                   (note.name, start_time, duration))
 
-            video_sample_uri = "file://%s/sample_%s.mkv" % (video_font_path, note.name)
+            video_sample_path = "%s/sample_%s.mkv" % (video_font_path, note.name)
 
-            timeline.add_clip(video_sample_uri, start_time, duration)
+            timeline.add_clip(video_sample_path, start_time, duration)
 
     return timeline
 
@@ -90,23 +90,30 @@ def main():
 
     overlapping_notes = check_overlapping_notes(midi_file)
     if overlapping_notes:
-        sys.stderr.write("Sorry, supporting only midi file with no overlapping notes")
+        sys.stderr.write("Sorry, supporting only midi file with no overlapping notes on channel 0\n")
         return 1
 
     if not os.path.isdir(sys.argv[2]):
-        sys.stderr.write("The second argument must be the path of the videofont directory")
+        sys.stderr.write("The second argument must be the path of the videofont directory\n")
         usage()
         return 1
 
+    if len(sys.argv) > 3 and os.path.exists(sys.argv[3]):
+        sys.stderr.write("File '%s' exists, exiting!\n" % sys.argv[3])
+        return 1
+
     video_font_path = os.path.realpath(sys.argv[2])
 
     timeline = timeline_from_midi(midi_file, video_font_path)
 
     if len(sys.argv) > 3:
-        destination_uri = 'file://' + os.path.realpath(sys.argv[3])
-        timeline.save(destination_uri)
+        timeline.save(sys.argv[3])
     else:
-        timeline.play()
+        try:
+            timeline.play()
+        except KeyboardInterrupt:
+            timeline.stop()
+            return 1
 
 
 if __name__ == "__main__":