Ticket #9385 (closed Patch - Feature: fixed)
Opened 2 years ago
Last modified 2 years ago
mythburn.py fails on audio coded labeled EAC3
| Reported by: | hobbes1069@… | Owned by: | paulh |
|---|---|---|---|
| Priority: | minor | Milestone: | 0.24.1 |
| Component: | Plugin - MythArchive | Version: | 0.24-fixes |
| Severity: | medium | Keywords: | mytharchive, mythburn |
| Cc: | Ticket locked: | no |
Description (last modified by paulh) (diff)
One recording had the audio codec labeled as EAC3 in the streaminfo.xml. ffmpeg didn't seem to care (I'm not sure if it uses streaminfo.xml) but mythreplex choked on it.
progress.log: 2010-12-16 14:52:12 Preferred audio languages eng and eng 2010-12-16 14:52:12 Video id: 0x1e0, Audio1: [1] 0x80 (EAC3, N/A), Audio2: [-1] - 0x-1 (N/A, N/A) 2010-12-16 14:52:12 Splitting MPEG stream into audio and video parts 2010-12-16 14:52:12 Running: mythreplex --demux --fix_sync -o /home/mythuser/temp/work/2/stream -v 224 "/home/mythuser/temp/work/2/newfile2.mpg" 2010-12-16 14:54:09 ************************************************************ 2010-12-16 14:54:09 ERROR: Failed while running mythreplex. Command was mythreplex --demux --fix_sync -o /home/mythuser/temp/work/2/stream -v 224 "/home/mythuser/temp/work/2/newfile2.mpg" 2010-12-16 14:54:09 ************************************************************ 2010-12-16 14:54:09 2010-12-16 14:54:09 Terminated ---
Running the failed command manually produces the following after processing part of the file: Running the failed command manually runs for a while before I get this:
Can't find all required streams Please check if audio and video have standard IDs (0xc0 or 0xe0) ---
Added EAC3 lines to mythburn.py (patch attached) and it worked.
Attachments
Change History
Changed 2 years ago by hobbes1069@…
- Attachment mythburn.py.diff added
comment:1 Changed 2 years ago by kenni
- Status changed from new to assigned
- Type changed from Bug Report to Patch - Feature
- Milestone changed from 0.24.1 to unknown
Read the TicketHowTo before creating tickets, it's not your decision when the responsible developer should look at your issue.
comment:2 Changed 2 years ago by Richard Shaw <hobbes1069@…>
Sorry, severity and priority are obvious... Milestone not so much.
comment:4 Changed 2 years ago by paulh
Fixed in master https://github.com/MythTV/mythtv/commit/206bde055
Fixed in fixes/0.24 https://github.com/MythTV/mythtv/commit/bac67dc27
comment:5 Changed 2 years ago by paulh
- Status changed from assigned to closed
- Resolution set to Fixed
- Milestone changed from unknown to 0.24.1
comment:6 Changed 2 years ago by hobbes1069
- Resolution changed from Fixed to fixed
MythArchive?: Add EAC3 as a valid audio format in the script.
Fixes ticket 9385.
Signed-off-by: Paul Harrison <pharrison@…>
Changeset: 206bde055d2d440a730270bf15c34dcf42992d8f
quick fix patch for mythburn.py