From 3402d968a79a39a1b09acf5b8239127497c507fd Mon Sep 17 00:00:00 2001
From: kfrn <knfrances@gmail.com>
Date: Wed, 31 Jul 2019 23:32:05 +1200
Subject: [PATCH 1/2] =?UTF-8?q?Remove=20MKV=20=E2=86=92=20MP4=20command?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Addressing user feedback received by Andrew Weaver (@privatezero).

When the source file is ffv1 in MKV, the command will fail on the `-c:v copy` flag, since MP4s can't house ffv1.
Therefore, the command is misleading, and has limited value from an archival perspective.

We already have a basic rewrap command (which includes a note addressing the possibility of stream/wrapper incompability), so I think our bases are covered with that.

See https://github.com/amiaopensource/ffmprovisr/issues/392
---
 index.html | 22 ----------------------
 1 file changed, 22 deletions(-)

diff --git a/index.html b/index.html
index 08bb434..fb6c486 100644
--- a/index.html
+++ b/index.html
@@ -213,28 +213,6 @@
     </div>
     <!-- End Basic rewrap command -->
 
-    <!-- MKV to MP4 -->
-    <label class="recipe" for="mkv_to_mp4">Convert Matroska (MKV) to MP4</label>
-    <input type="checkbox" id="mkv_to_mp4">
-    <div class="hiding">
-      <h3>MKV to MP4</h3>
-      <p><code>ffmpeg -i <em>input_file</em>.mkv -c:v copy -c:a aac <em>output_file</em>.mp4</code></p>
-      <p>This will convert your Matroska (MKV) files to MP4 files.</p>
-      <dl>
-        <dt>ffmpeg</dt><dd>starts the command</dd>
-        <dt>-i <em>input_file</em></dt><dd>path and name of the input file<br>
-        The extension for the Matroska container is <code>.mkv</code>.</dd>
-        <dt>-c:v copy</dt><dd>copies the video stream without re-encoding it</dd>
-        <dt>-c:a aac</dt><dd>re-encodes the audio stream using the AAC audio codec<br>
-        Note that sadly MP4 cannot contain sound encoded by a PCM (Pulse-Code Modulation) audio codec.<br>
-        For silent videos you can replace <code>-c:a aac</code> by <code>-an</code>, which means that there will be no audio track in the output file.</dd>
-        <dt><em>output_file</em></dt><dd>path and name of the output file<br>
-        The extension for the MP4 container is <code>.mp4</code>.</dd>
-      </dl>
-      <p class="link"></p>
-    </div>
-    <!-- ends MKV to MP4 -->
-
     <!-- Rewrap DV -->
     <label class="recipe" for="rewrap-dv">Rewrap DV video to .dv file</label>
     <input type="checkbox" id="rewrap-dv">

From a6dd9c203c5ef75a06da44b959cd673b761e9056 Mon Sep 17 00:00:00 2001
From: Reto Kromer <retokromer@users.noreply.github.com>
Date: Wed, 31 Jul 2019 16:50:39 +0200
Subject: [PATCH 2/2] update recipe list

---
 recipes.txt | 2 --
 1 file changed, 2 deletions(-)

diff --git a/recipes.txt b/recipes.txt
index ec099df..52c11b6 100644
--- a/recipes.txt
+++ b/recipes.txt
@@ -1,7 +1,5 @@
 # Basic rewrap command
 ffmpeg -i input_file.ext -c copy -map 0 output_file.ext
-# Convert Matroska (MKV) to MP4
-ffmpeg -i input_file.mkv -c:v copy -c:a aac output_file.mp4
 # Rewrap DV video to .dv file
 ffmpeg -i input_file -f rawvideo -c:v copy output_file.dv
 # Transcode to deinterlaced Apple ProRes LT