Opened 13 years ago

Closed 12 years ago

#1702 closed patch (invalid)

Patch that adds "Alternate Player" command to MythVideo

Reported by: eggi@… Owned by: Anduin Withers
Priority: minor Milestone: 0.21
Component: mythvideo Version:
Severity: medium Keywords:
Cc: Ticket locked: no

Description

This patch adds a "Watch in Alternate Player" command to the INFO menu in all three MythVideo? views, and an edit field to the "Player Settings" page where users can set the alternate player command. This makes it much easier to deal with videos that don't play in the Internal player. Instead of going to the Video Manager and entering a Unique Player Command for each, you can play them in Xine/MPlayer with just three keystrokes.

Attachments (1)

alternate-player.patch (11.6 KB) - added by eggi@… 13 years ago.

Download all attachments as: .zip

Change History (6)

Changed 13 years ago by eggi@…

Attachment: alternate-player.patch added

comment:1 Changed 13 years ago by Anduin Withers

Owner: changed from jdonavan to Anduin Withers

comment:2 Changed 13 years ago by danielk

Resolution: fixed
Status: newclosed

(In [10434]) Fixes #1702. Refs #1648. Implements a very basic signal monitor class for the freebox recorder.

This tries once to connect to the streaming server and reports a lock if it succeeds. It then waits for a PAT and a PMT with a program number of 1.

Ideally we would try to reconnect if the connection failed the first time, and we would make rtspcomms.cpp a little more efficient.

This also adds "-v record" debugging to the freebox recorder. When I was testing I found some problems with handling error conditions. I think these are all fixed, but the additional debugging should make any additional problems easier to track down.

BTW The freebox recorder will work with any Internet TV station broadcasting a TS MPEG2 stream using RTSP. You just need to create a m3u file as described the readme on ticket #1702. H.264 (aka MPEG-4 AVC) support could be added without much work, if there is any interest in it.

comment:3 Changed 13 years ago by danielk

Resolution: fixed
Status: closedreopened

Should have been #1704.

comment:4 Changed 13 years ago by Anduin Withers

Milestone: 0.21

comment:5 Changed 12 years ago by Anduin Withers

Resolution: invalid
Status: reopenedclosed

This is on the list for post 0.21 now. I do appreciate the patch, this functionality will be there in whatever new mechanism is actually developed.

Note: See TracTickets for help on using tickets.