[mythtv] Mini patch: DVB Pids

Edward Wildgoose edward.wildgoose at frmhedge.com
Mon Aug 25 20:12:03 EDT 2003


What to do with the pids field in the database has been a hot issue, so I'm
hopefully putting an acceptable mini patch on the table that I would like
everyone to bludgeon into an acceptable state and then, with luck, we can
move on and do something with it.

If everyone agrees (Ben, Ramon, Kenneth), or at least doesn't disagree, then
can we please have this committed

In a nutshell I have ammended the DB pid table to have 6 pid columns now:

:: pids_audio    :: pids_video :: pids_teletext ::
:: pids_subtitles :: pids_pcr   :: pids_other    ::

I believe these are the acceptable names for things, and the only point to
watch seems to be whether "pids_subtitles" should be plural, ie with an "s"
on the end.  In seperate emails I have seen it both ways, so I have plumped
for the plural here.

The change to dvb_channel is trivial and just hooks out these six columns
and concatenates them together.  Although this could be done in an
alternative way, I am hoping that we can put the patch in as-is to keep the
modifications small and I will then have another patch coming up soon after
which will neaten things up.  The point here is more to get agreement on pid
column names

Oh, please also notice that I am including all the other tables that Ramon
has proposed.  After looking at them they all seem quite reasonable to me.
There *are* two extra tables on the end for the channel groupings which I
have also left in - these are not going to be used until mythchannels
develops, so I would be happy to remove them for the time being if
requested?

I would be grateful for feedback and a checkin if nothing arises from the
review.

Thanks all

Ed W
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mc.sql
Type: application/octet-stream
Size: 19828 bytes
Desc: not available
Url : http://mythtv.org/pipermail/mythtv-dev/attachments/20030825/c599308e/mc-0001.obj
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cvs.sql
Type: application/octet-stream
Size: 20496 bytes
Desc: not available
Url : http://mythtv.org/pipermail/mythtv-dev/attachments/20030825/c599308e/cvs-0001.obj
-------------- next part --------------
A non-text attachment was scrubbed...
Name: dvbchannel-with_new_pid_names.diff
Type: application/octet-stream
Size: 1034 bytes
Desc: not available
Url : http://mythtv.org/pipermail/mythtv-dev/attachments/20030825/c599308e/dvbchannel-with_new_pid_names-0001.obj


More information about the mythtv-dev mailing list