Opened 7 months ago

Closed 3 months ago

#2136 closed feature_patch (fixed)

Set mythtv version in DT

Reported by: esperegu Owned by:
Priority: normal Milestone: 1204
Component: unknown Version: 1204
Severity: normal Keywords:
Cc:

Description

To make it easier for users to test other version or upgrade be able to set the myth protocol version and token on the DT

sqlcsv: 620, 621, 4321, -358, 4322

Attachments (7)

mythtv-plugin-setversioninDT.dff (2.5 KB) - added by esperegu 7 months ago.
changes of mythtv plugin and socket code.
mythtv-plugin-setversioninDT.2.dff (2.9 KB) - added by esperegu 7 months ago.
now with the version set in the code and overridden by the DT if required
mythtv-plugin-setversioninDT.diff (2.9 KB) - added by esperegu 7 months ago.
with check of both values
mythtv-plugin-setversioninDT_2.diff (2.7 KB) - added by esperegu 7 months ago.
with new device data
MythTV_PlugIn.diff (2.6 KB) - added by esperegu 3 months ago.
updated patch. stuff in svn changed?
Gen_Devices_MythTV_PlugInBase.diff (1.6 KB) - added by esperegu 3 months ago.
Gen_Devices diff
MythTV_PlugIn.2.diff (2.8 KB) - added by esperegu 3 months ago.
with the catalog remove line intact

Download all attachments as: .zip

Change History (17)

Changed 7 months ago by esperegu

changes of mythtv plugin and socket code.

comment:1 Changed 7 months ago by phenigma

  • Resolution set to wontfix
  • Status changed from new to closed

This will not be implemented. The DT affects ALL versions of LMCE whereas the source does not. This change will only be implemented in source code for the version of lmce that we currently support. Anyone wishing to test upgrading will need the knowledge of building the plugin with the correct values.

Changed 7 months ago by esperegu

now with the version set in the code and overridden by the DT if required

comment:2 Changed 7 months ago by esperegu

I adapted the code as requested so it defaults to the version set in the code. The DT now only needs to be set if one wants to override the parameters. I also changed the naming to be more inline with other naming however running sqlcvs diff does not show the changes (only changed the parameter names, description and removed the is required condition) so here they are:


#320 Myth Protocol(int)
The mythtv protocol version (this should be empty when using the version that is installed by default. Only use this when upgrading mythtv manually)
allowed to modify


#321 Myth Token(string)
The mythtv token (this should be empty when using the version that is installed by default. Only use this when upgrading mythtv manually)
allowed to modify


comment:3 Changed 7 months ago by esperegu

  • Resolution wontfix deleted
  • Status changed from closed to reopened

comment:4 Changed 7 months ago by esperegu

Maybe convenient to add this to the descriptions: http://www.mythtv.org/wiki/Category:Myth_Protocol

Changed 7 months ago by esperegu

with check of both values

comment:5 Changed 7 months ago by esperegu

#320 Myth Protocol(int)

The mythtv protocol version (this should be empty when using the version that is installed by default. Only use this when upgrading mythtv manually. This value is only taken into account if both the version and the token are set in the template. See http://www.mythtv.org/wiki/Category:Myth_Protocol for valid values)
allowed to modify


#321 Myth Token(string)
The mythtv token (this should be empty when using the version that is installed by default. Only use this when upgrading mythtv manually. This value is only taken into account if both the version and the token are set in the template. See http://www.mythtv.org/wiki/Category:Myth_Protocol for valid values)
allowed to modify

comment:6 Changed 7 months ago by phenigma

Repository: dce, Batch ID: 4326 - Refs #2136 - add a Token DeviceData? field for use with MythTV

Please sqlCVS update, run sql2cpp to generate the new DATA access function. Update the diff to use the existing Protocol DD and the new Token DD and to include the added DATA function from the sql2cpp run. Thanks!

comment:7 Changed 7 months ago by esperegu

sqlCVS 4336, -363

Changed 7 months ago by esperegu

with new device data

Changed 3 months ago by esperegu

updated patch. stuff in svn changed?

comment:9 Changed 3 months ago by phenigma

Repository: dce, Batch ID: -363 - MythTV_Plugin : set version in DT 2 - approved in Repository: dce, Batch ID: 4400

Changed 3 months ago by esperegu

Gen_Devices diff

Changed 3 months ago by esperegu

with the catalog remove line intact

comment:10 Changed 3 months ago by phenigma

  • Resolution set to fixed
  • Status changed from reopened to closed

In [29348]:

Refs #2136 - makes mythtv protocol and token user changeable for easier mythtv version testing

Note: See TracTickets for help on using tickets.