[mythtv-commits] Ticket #8094: Adjust UPnP detection to search localhost

[MythTV]

#8094: Adjust UPnP detection to search localhost
---------------------------+------------------------------------------------
 Reporter:  wagnerrp       |       Owner:  dblain 
     Type:  enhancement    |      Status:  new    
 Priority:  minor          |   Milestone:  unknown
Component:  MythTV - UPnP  |     Version:  head   
 Severity:  low            |     Mlocked:  0      
---------------------------+------------------------------------------------

Comment(by superm1@…):

 Just so these aren't lost (at least for the C++ implementation):

 superm1> danielk22, could you comment whether
 http://svn.mythtv.org/trac/ticket/8094 looks too crackful at all during
 freeze - or if its at least the right direction to be headed?
 <danielk22> superm1: I think it's to late for that one. As for the patch
 itself my first concern would be that it might connect to a local master
 backend when you really want to the popup to choose which master to
 connect to.
 <superm1> danielk22, okay thanks.  that's what i was figuring, it's just
 something wagnerrp me and rhpot1991 whipped together.
 <superm1> that's a situation i certainly didn't think about
 <superm1> albeit probably uncommon for anyone but a developer
 <danielk22> I'm not terribly fond of using something as heavyweight as
 UPnP for detecting backends, but it does mostly work.
 <superm1> do you think adding a --disable-local switch to the misc
 binaries using mythcontext like that would be sufficient then to prevent
 such a type of problem?
 <danielk22> superm1: Sure, but it's always good to try to address these
 things from the start, so hacks don't get applied on top of hacks later
 on. If it isn't tractable, then a switch would be fine. Maybe just a popup
 that ask "A local master backend was found, do you want to search for
 remote ones as well?"
 <superm1> that's a good point.  i'll see about making such a popup instead
 then, sounds like a better solution to avoid hack upon hack being added

-- 
Ticket URL: <http://svn.mythtv.org/trac/ticket/8094#comment:1>
MythTV <http://www.mythtv.org/>
MythTV