Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#2593 closed task (fixed)

mythtv-iptv : merge in UDP & MPEG4 AVC support into freebox recorder

Reported by: danielk Owned by: danielk
Priority: minor Milestone: unknown
Component: mythtv Version: 0.20
Severity: medium Keywords:
Cc: scott@… Ticket locked: no

Description


Attachments (1)

2593-v1.patch (19.0 KB) - added by danielk 13 years ago.
Patch with remaining changes in branch (a patch against SVN head).

Download all attachments as: .zip

Change History (37)

comment:1 Changed 13 years ago by danielk

(In [11573]) Refs #2593. Merge patch 1 of 18. "Initial patch based on CRCIP implementation" -- Mickael Remars

comment:2 Changed 13 years ago by danielk

(In [11574]) Refs #2593. Moves FreeBoxMediaSink? and RTSPComms to iptv directory.

comment:3 Changed 13 years ago by danielk

Owner: changed from Isaac Richards to danielk

comment:4 Changed 13 years ago by danielk

(In [11575]) Refs #2593. Merge patch 2 of 18.

From Mickael Remars:

  • New FreeboxFeederUdp? implementation based on Mike Mironov's patch
  • ChannelFetcher? can read the m3u from files or ftp
  • renamed the RTSPListener to StreamDataListener?
  • Bugfix in TS find
  • add missing includes for assert()
  • merge common code between rtsp & udp

A "StreamDataListener?" class already exists in MythTV so I made it "IPTVListener" instead. I also removed all the assert() calls so I didn't add the assert includes.

comment:5 Changed 13 years ago by danielk

(In [11576]) Refs #2593. Merge patch 3 of 18. "bugfix in FreeboxRecorder::Unpause()" -- Mickael Remars

comment:6 Changed 13 years ago by danielk

(In [11582]) Refs #2593. Merge patch 4 of 18.

From Mickael Remars:

comment:7 Changed 13 years ago by danielk

(In [11583]) Refs #2593. Merge portion of patch 5 of 18. Documents FreeboxFeeder?.

comment:8 Changed 13 years ago by danielk

(In [11584]) Refs #2593. Merge portion of patch 5 of 18. Adds error check in FreeboxFeederUdp::Open().

comment:9 Changed 13 years ago by danielk

(In [11585]) Refs #2593. Merge portion of patch 5 of 18. Cleanup, removes unneeded header.

comment:10 Changed 13 years ago by danielk

(In [11586]) Refs #2593. Merge portion of patch 5 of 18. Refactors FreeboxFeederRtsp? to remove Init() and Deinit(), also removes some dead code.

comment:11 Changed 13 years ago by danielk

(In [11587]) Refs #2593. Merge portion of patch 6 of 18. Adds FreeboxFeederFile?, for debugging.

comment:12 Changed 13 years ago by danielk

(In [11589]) Refs #2593. Fixes a bug adding listeners in FreeboxFeederFile? + minor cleanup

comment:13 Changed 13 years ago by danielk

(In [11590]) Refs #2593. Undoing [11589], too many files were caught up in that commit.

comment:14 Changed 13 years ago by danielk

(In [11591]) Refs #2593. Actually, fixes a bug adding listeners in FreeboxFeederFile? + minor cleanup

comment:15 Changed 13 years ago by danielk

(In [11592]) Refs #2593. Minor cleanup

comment:16 Changed 13 years ago by danielk

(In [11593]) Refs #2593. Merge patch 14 of 18. Adds an FreeboxFeederRTP to handle RTP streams.

comment:17 Changed 13 years ago by danielk

(In [11594]) Refs #2593. Merge patch 15 of 18. Initial MPEG-4 AVC support in FreeboxRecorder?.

comment:18 Changed 13 years ago by danielk

(In [11595]) Refs #2593. Merge patch 16 of 18. Adds check for keyframe buffering for MPEG-4 AVC + adds sanity check for when to give up on placing keyframe DB record in the right place (currently at 50 TS packets, same as MPEG-2; this probably should be larger for any MPEG stream).

comment:19 Changed 13 years ago by danielk

(In [11596]) Refs #2593. Merges portions of remaining cleanup patches. No functional changes.

comment:20 Changed 13 years ago by danielk

(In [11644]) Refs #2593. Fixes bug in TimeoutedTaskScheduler? introduced in [11575] in iptv branch.

comment:21 Changed 13 years ago by danielk

(In [11645]) Refs #2593. Fixes a bug in FreeboxFeederWrapper::Open() introduced in [11582] in iptv branch.

comment:22 Changed 13 years ago by danielk

(In [11647]) Refs #2593. Merges -r11571:11645 from svn head to mythtv-iptv

comment:23 Changed 13 years ago by danielk

(In [11651]) Refs #2593. Renames the freebox*.{cpp,h} files to iptv*.{cpp,h}. The class renames will be in a seperate changeset. This recorder can handle a lot more than Freebox streams so the name change is to reflect that.

comment:24 Changed 13 years ago by danielk

(In [11652]) Refs #2593. Renames the Freebox classes to become IPTV classes.

comment:25 Changed 13 years ago by danielk

(In [11653]) Refs #2593. Fixes a TODO, allows channel scanner to update more of the channel info.

comment:26 Changed 13 years ago by danielk

(In [11654]) Refs #2593. Fixes a bug in [11563]. After the change null strings were not being dealt with properly in channel insertion.

comment:27 Changed 13 years ago by danielk

(In [11655]) Refs #2593. Fixes a TODO in URLFetcher. We were convering data from UTF8 to UTF16 without treating escape codes properly. This changes when we do the conversion so that we handle these properly.

comment:28 Changed 13 years ago by danielk

(In [11656]) Refs #2593. Various small fixes to the new IPTV code.

comment:29 Changed 13 years ago by danielk

(In [11657]) Refs #2593. A few more small fixes to the new IPTV code.

comment:30 Changed 13 years ago by danielk

(In [11659]) Refs #2593. A couple more small fixes to the new IPTV code.

comment:31 Changed 13 years ago by danielk

(In [11660]) Refs #2593. Merges in the refactor of the FreeboxRecorder?'s RTSP support.

This refactors the code so that other transport methods can be used and fixes some bugs in the M3U channel parser.

The other transport methods implemented in the IPTV branch have not been ported over since they didn't work in my testing. I've asked Mickael for his test scripts since his unmodified UDP/RTP patches didn't work for me either.

The recorder has been renamed from Freebox to IPTV to since it is now a more general purpose stream recorder. The configure option has also been changed from --disable-freebox to --disable-iptv. MPEG-4 AVC streams should also be supported now. svn commit -m Refs

Changed 13 years ago by danielk

Attachment: 2593-v1.patch added

Patch with remaining changes in branch (a patch against SVN head).

comment:32 Changed 13 years ago by danielk

(In [11665]) Refs #2593. Fixes #2647. Adds declaration to support newer gcc's.

comment:33 Changed 13 years ago by anonymous

Cc: scott@… added

comment:34 Changed 13 years ago by danielk

Resolution: fixed
Status: newclosed

(In [11720]) Fixes #2593. Adds support for UDP, RTP and file stream handlers to IPTV recorder (formerly Freebox recorder).

The format for the URLS for are: udp://ip_addr:port rtp://ip_addr:port file://pathname

This is unlike the CRC IP recorder's UDP URLs which have an "@" sign after the ":", but is internally consistent with how the other protocols are handled.

comment:35 Changed 13 years ago by danielk

(In [11721]) Refs #2593. Removes CRC's IP network recorder, the IPTV recorder does all it did and more now.

comment:36 Changed 13 years ago by danielk

(In [12620]) Refs #2593. Removes some remaining references to CRC_IP recorder, replaced with IPTV Recorder.

Note: See TracTickets for help on using tickets.