From 4cb6e1ecfa4ad3a875616551599a1d29c9acccf0 Mon Sep 17 00:00:00 2001
From: Carl Hetherington <cth@carlh.net>
Date: Tue, 14 May 2019 22:00:31 +0100
Subject: [PATCH] swaroop: allow output container specification.

---
 src/tools/dcpomatic_ecinema.cc | 19 +++++++++++++++++--
 1 file changed, 17 insertions(+), 2 deletions(-)

diff --git a/src/tools/dcpomatic_ecinema.cc b/src/tools/dcpomatic_ecinema.cc
index a0324f281..a5ba115d1 100644
--- a/src/tools/dcpomatic_ecinema.cc
+++ b/src/tools/dcpomatic_ecinema.cc
@@ -48,6 +48,7 @@ help (string n)
 	     << "  -v, --version        show DCP-o-matic version\n"
 	     << "  -h, --help           show this help\n"
 	     << "  -o, --output         output directory\n"
+	     << "  -f, --format         output format (mov or mp4; defaults to mov)\n"
 	     << "\n"
 	     << "<FILE> is the unencrypted .mp4 file.\n";
 }
@@ -56,6 +57,7 @@ int
 main (int argc, char* argv[])
 {
 	optional<boost::filesystem::path> output;
+	optional<boost::filesystem::path> format;
 
 	int option_index = 0;
 	while (true) {
@@ -63,9 +65,10 @@ main (int argc, char* argv[])
 			{ "version", no_argument, 0, 'v'},
 			{ "help", no_argument, 0, 'h'},
 			{ "output", required_argument, 0, 'o'},
+			{ "format", required_argument, 0, 'f'},
 		};
 
-		int c = getopt_long (argc, argv, "vho:", long_options, &option_index);
+		int c = getopt_long (argc, argv, "vho:f:", long_options, &option_index);
 
 		if (c == -1) {
 			break;
@@ -81,6 +84,9 @@ main (int argc, char* argv[])
 		case 'o':
 			output = optarg;
 			break;
+		case 'f':
+			format = optarg;
+			break;
 		}
 	}
 
@@ -94,6 +100,15 @@ main (int argc, char* argv[])
 		exit (EXIT_FAILURE);
 	}
 
+	if (!format) {
+		format = "mov";
+	}
+
+	if (*format != "mov" && *format != "mp4") {
+		cerr << "Invalid format specified: must be mov or mp4\n";
+		exit (EXIT_FAILURE);
+	}
+
 	boost::filesystem::path input = argv[optind];
 	boost::filesystem::path output_mp4 = *output / (input.filename().string() + ".ecinema");
 
@@ -115,7 +130,7 @@ main (int argc, char* argv[])
 	}
 
 	AVFormatContext* output_fc;
-	avformat_alloc_output_context2 (&output_fc, av_guess_format("mp4", 0, 0), 0, 0);
+	avformat_alloc_output_context2 (&output_fc, av_guess_format(format->c_str(), 0, 0), 0, 0);
 
 	for (uint32_t i = 0; i < input_fc->nb_streams; ++i) {
 		AVStream* is = input_fc->streams[i];
-- 
2.30.2