Patch: Suggest that mythtranscode shouldn't always remove the original file

[Bill Meek <keemllib@…>]

changeset:3d64f8a now calls CompleteJob?() for all types of transcodes.

When using --hlsstreamid, the output goes to the Streaming Storage Group and the original file(s) shouldn't be removed.

Currently, the following happens (there is no .tmp file):

mythtranscode: about to rename '/srv/mythtv-0/recordings/2609_20150217165800.ts' to '/srv/mythtv-0/recordings/2609_20150217165800.ts.old'
mythtranscode: About to unlink/delete file: /srv/mythtv-0/recordings/2609_20150217165800.ts.old

Patch tested on v0.28-pre-2670-g3d05b39-dirty. I'm not sure if the test should be for --hls or --hlsstreamid. Either works for the Content/AddLiveStream? endpoint and I believe cares for the original intent to handle command line xcodes.

diff --git a/mythtv/programs/mythtranscode/main.cpp b/mythtv/programs/mythtranscode/main.cpp
index 8c84813..a01acde 100644
--- a/mythtv/programs/mythtranscode/main.cpp
+++ b/mythtv/programs/mythtranscode/main.cpp
@@ -748,7 +748,8 @@ int main(int argc, char *argv[])
         exitcode = result;
     }
 
-    CompleteJob(jobID, pginfo, useCutlist, &deleteMap, exitcode, result);
+    if (!cmdline.toBool("hlsstreamid"))
+        CompleteJob(jobID, pginfo, useCutlist, &deleteMap, exitcode, result);
 
     transcode->deleteLater();

[Stuart Morgan <smorgan@…>]

In 0f52e5c146625aef83ca880974ce94efcd17c5fb/mythtv:

Don't remove the original file for HLS transcodes. Fixes #12385

[stuartm]

Thanks Bill, apologies for missing that case, I hope the damage wasn't too bad.

[steve_g@…]

Should also not delete is FIFO was used. THe files may not have been transcode to the same place they came from.

}

if (!cmdline.toBool("hls")

(cmdline.toBool("fifodir")))

CompleteJob?(jobID, pginfo, useCutlist, &deleteMap, exitcode, result);

or do O have to file another bug report?

[Stuart Auchterlonie]

Moving unresolved tickets to next point release

[Bradley Baetz <bbaetz@…>]

patch

[Bradley Baetz <bbaetz@…>]

I've attached a patch that also avoids deleting the source file when a fifodir is being used (e.g. by nuvexport)

[Christophe Boyanique <tof-mythtvtrac@…>]

I can confirm that patch provided by Bradley Baetz corrects the unwanted deletion of original file in fifo mode. Please consider including it in 0.28.1 release.

[perkins1724@…]

Can the use case of the -o output file override please be considered / checked as well? It seems the original recording file is getting deleted even though there is no xxxxx.ts.tmp file to move back into its place - this is resulting in orphaned records.

​http://www.gossamer-threads.com/lists/mythtv/commits/601913 refers.

[JYA]

duplicate of #12845