[mythtv-commits] mythtv commit: r22972 by mdean

[mythtv at cvs.mythtv.org]

      Author: mdean
        Date: 2009-12-10 05:44:39 +0000 (Thu, 10 Dec 2009)
New Revision: 22972
   Changeset: http://svn.mythtv.org/trac/changeset/22972

Modified:

   trunk/mythtv/contrib/config_files/config.xml

Log:

After [22970], remove the commented <LocalHostName> element since the Perl bindings do not use a real XML parser to read config.xml, so they were picking up the commented value.

I hope to revert this change once I get approval to modify the bindings to use XML::Simple or XML::Parser for reading config.xml.  While I think the user of a real parser was avoided so as not to add a dependency, I can't imagine a system having Perl, but lacking XML::Simple and/or XML::Parser (or at the very least, expat), these days.  I believe the added robustness for parsing config.xml is worth the requirement (since config.xml is often edited by users, who may not realize that commented elements will still be used by the Perl bindings).