[mythtv-commits] Re: Ticket #222: DVB: Use TS format by default

MythTV mythtv at cvs.mythtv.org
Tue Aug 16 16:38:58 UTC 2005


#222: DVB: Use TS format by default
-----------------------------------+----------------------------------------
 Reporter:  jesper at datapartner.se  |        Owner:  ijr 
     Type:  patch                  |       Status:  new 
 Priority:  minor                  |    Milestone:  0.19
Component:  mythtv                 |      Version:      
 Severity:  low                    |   Resolution:      
       Cc:                         |  
-----------------------------------+----------------------------------------
Comment (by kirwin at ncsu.edu):

 We already changed it to be the default for new users a while back.   The
 earlier change was also supposed to change it for existing users upon the
 next database upgrade, but I botched that one.  Sorry.

 This patch goes a lot further, even changing the setting for existing
 users, who, presumably like their settings how they are.  This I think is
 a bad idea.  Users can change their own settings if they don't like the
 results.  Changing their settings without telling them is bad.  In
 general, I would argue, user settings should only be changed when code
 changes make it so that those settings are broken or inapplicable.

 It also updates the database version in order to forcibly push through the
 default change in the table.  This I had chosen not to do with my patch,
 but is probably not unreasonable.  If it were my decision, I would add the
 code for the next update, but not update the db version for just a change
 in the default setting.  That way the change would go through whenever
 someone made a significant enough change to the database to require an
 upgrade.  If I had put the code in the right place in my patch, this would
 have already happened.  Sorry.  But if you want to bump the database
 version, I certainly won't complain.

 Keith

-- 
Ticket URL: <http://cvs.mythtv.org/trac/ticket/222>
MythTV <http://www.mythtv.org/>
MythTV


More information about the mythtv-commits mailing list