[mythtv-commits] mythtv commit: r22972 by mdean
mythtv at cvs.mythtv.org
mythtv at cvs.mythtv.org
Thu Dec 10 05:44:40 UTC 2009
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).
More information about the mythtv-commits
mailing list