Opened 18 years ago

Closed 18 years ago

#2156 closed patch (fixed)

detection for DVD backups on HD for internal player (make it more usable as the overall MythVideo player)

Reported by: Pekka Jääskeläinen <pekka.jaaskelainen@…> Owned by: Anduin Withers
Priority: minor Milestone: 0.20
Component: mythtv Version: head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

This trivial patch adds support for using the Internal video player for playing DVD backups stored on HD. The patch works simply by detecting a path which is a directory containing a VIDEO_TS subdirectory or a filename with .img or .iso extension to be a DVD backup, and prepends "dvd:" in front of the pathname.

Attachments (3)

internal_dvd_player_dvdbackup_playback_improvements.diff​ (808 bytes) - added by Pekka Jääskeläinen <pekka.jaaskelainen@…> 18 years ago.
Against trunk
updated.patch​ (945 bytes) - added by skamithi 18 years ago.
updated2.patch​ (1006 bytes) - added by skamithi 18 years ago.
add .img to the list of recognized dvd images. so far only .iso ext is recognized as a dvd image.

Download all attachments as: .zip

Change History (11)

Changed 18 years ago by Pekka Jääskeläinen <pekka.jaaskelainen@…>

Attachment: internal_dvd_player_dvdbackup_playback_improvements.diff​ added

Against trunk

comment:1 Changed 18 years ago by anonymous

Owner: changed from danielk to skamithi

comment:2 Changed 18 years ago by Isaac Richards

Resolution: → wontfix
Status: new → closed

I don't think this sort of logic should go into RingBuffer? class, please move it higher up.

comment:3 Changed 18 years ago by skamithi

i agree. just wrote an updated patch. is this one okay ?

Changed 18 years ago by skamithi

Attachment: updated.patch​ added

comment:4 Changed 18 years ago by skamithi

Resolution: wontfix
Status: closed → reopened

comment:5 Changed 18 years ago by Isaac Richards

Yup, much better, thanks.

comment:6 Changed 18 years ago by skamithi

Milestone: → 0.20
Version: → head

Changed 18 years ago by skamithi

Attachment: updated2.patch​ added

add .img to the list of recognized dvd images. so far only .iso ext is recognized as a dvd image.

comment:7 Changed 18 years ago by skamithi

Owner: changed from skamithi to Anduin Withers
Status: reopened → new

comment:8 Changed 18 years ago by Anduin Withers

Resolution: → fixed
Status: new → closed

(In [10766]) Adds support for playing .img files and directories containing a VIDEO_TS directory.

Closes #2156

Note: See TracTickets for help on using tickets.