[mythtv] Autodiscovery Patch for MythFrontend.

Jochen Kühner jochen.kuehner at gmx.de
Mon Apr 9 15:07:16 UTC 2007

David schrieb:
> Attached is a patch against Revision 13184 that adds Auto-Discovery to the
> Frontend.
> NOTE: The Screens to pick a backend and enter a password are downright UGLY!
> The code in the patch is completely functional, and only needs cosmetic
> changes.
> A few things to note:
> - Themes don't work without a database available for settings.  This
> includes the existing dialog that should be presented to the user if
> mysql.txt is missing and the defaults don't work.  I was able to trick it
> into working by calling OverrideSettingForSession with 31 settings, however
> as soon as the MythMainWindow is created, it tries to insert all of the
> global key bindings into the database which fails.  Although I could have
> continued down this route, it seemed like too much of a hack so I scrapped
> it for the current implementation.
> - I tried to make as few changes to mythcontext as possible.  If the user
> chooses "No master Backend" it will default to the existing program flow.
> - A new Command Line option -p or --prompt will force the selection screen
> to be displayed even if the config.xml file has a valid USN in it.
> - The autodiscovery code stores the users default selection in the
> .mythtv/config.xml file.  It's also the file that can be used to override
> default UPnP settings.  (See prior commit for more details on file format).
> - Only the USN (Unique Service Name) of the backend is stored, not it's IP
> address.  This allows the backend to change IPs at will, and the frontend
> will always be able to connect to it with no user intervention.
> - All of the poorly designed dialog boxes are in the masterselection.cpp/.h
> files in case anyone wants to improve upon them.
> - With this patch, the frontend starts to participate in the UPnP protocol
> as a AVMediaRenderer.  A full implementation at this time would make the
> patch too complex.  If this, or a modified version of this patch gets
> committed, I will add the remaining classes to stub out all services needed
> for a fully compliant UPnP Renderer.  (However, this patch does make the
> frontend a discoverable device).
> Let me know if anyone has any questions.
> David Blain.
> ------------------------------------------------------------------------
> _______________________________________________
> mythtv-dev mailing list
> mythtv-dev at mythtv.org
> http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-dev
will you create a ticket for this???

More information about the mythtv-dev mailing list