Opened 12 years ago

Closed 11 years ago

#4811 closed enhancement (invalid)

Patch to perl bindings to allow overriding master_host and master_port

Reported by: eharris@… Owned by: Isaac Richards
Priority: minor Milestone: 0.22
Component: mythtv Version: 0.20-fixes
Severity: medium Keywords: perl bindings
Cc: Ticket locked: no

Description

The current perl bindings do not appear to provide a method for overriding the master_host and master_port settings from the ones in the db. I needed this to be able to communicate with more than one master myth server, so added it to the optional param hash (similar to the existing DB overrides).

While it wasn't strictly necessary for my purposes, since there was already a dependency in the perl bindings for reading other overrides out of the mysql.txt file, I decided to go ahead and add the same ability for these settings, even though they aren't really related to mysql.

Attachments (2)

MythTV.pm.diff (2.1 KB) - added by eharris@… 12 years ago.
Unified diff to add support to override master settings
MythTV.pm.2.diff (2.5 KB) - added by eharris@… 12 years ago.
Unified diff for same patch except using current svn 0.21 tree as base

Download all attachments as: .zip

Change History (4)

Changed 12 years ago by eharris@…

Attachment: MythTV.pm.diff added

Unified diff to add support to override master settings

Changed 12 years ago by eharris@…

Attachment: MythTV.pm.2.diff added

Unified diff for same patch except using current svn 0.21 tree as base

comment:1 Changed 12 years ago by Isaac Richards

Milestone: 0.210.22

comment:2 Changed 11 years ago by cpinkham

Resolution: invalid
Status: newclosed

We don't use mysql.txt anymore, the perl bindings use the new config.xml file, so this patch doesn't apply anymore. If you'd like to submit a version that handles config.xml, you can reopen the ticket.

Note: See TracTickets for help on using tickets.