[Xen-devel] [PATCH] README: Update references to PyXML to lxml (Was: Re: [PATCH] Re: remove dependency on PyXML from xen?)

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Mon, 2012-08-20 at 20:44 +0100, M A Young wrote:
> On Fri, 17 Aug 2012, Ian Campbell wrote:
> 
> > ...
> > and pushed. Is the following section of README still accurate? At least
> > the mention of "PyXML" seems wrong to me.
> >
> >        Python Runtime Libraries
> >        ========================
> >
> >        Xend (the Xen daemon) has the following runtime dependencies:
> >
> >            * Python 2.3 or later.
> >              In some distros, the XML-aspects to the standard library
> >              (xml.dom.minidom etc) are broken out into a separate 
> > python-xml package.
> >              This is also required.
> >              In more recent versions of Debian and Ubuntu the XML-aspects 
> > are included
> >              in the base python package however (python-xml has been removed
> >              from Debian in squeeze and from Ubuntu in intrepid).
> >
> >                  URL:    http://www.python.org/
> >                  Debian: python
> >
> >            * For optional SSL support, pyOpenSSL:
> >                  URL:    http://pyopenssl.sourceforge.net/
> >                  Debian: python-pyopenssl
> >
> >            * For optional PAM support, PyPAM:
> >                  URL:    http://www.pangalactic.org/PyPAM/
> >                  Debian: python-pam
> >
> >            * For optional XenAPI support in XM, PyXML:
> >                  URL:    http://codespeak.net/lxml/
> >                  Debian: python-lxml
> >                  YUM:    python-lxml
> 
> Yes, it should be lxml not PyXML. The link could be upgraded as well as 
> http://codespeak.net/lxml/ redirects to http://lxml.de/ .

Thanks, patch below.

> I also found mention of PyXML in tools/python/logging/logging-0.4.9.2/ in 
> the files README.txt, python_logging.html, and test/logrecv.py (in an 
> error message) as a dependency for ZSI though I don't think xen ever uses 
> the code.

Great, yet another imported source base!

It seems like the functionality is now part of mainline python. From
looking at Debian it seems that python 2.5+ contains this module
directly.

According to tools/python/logging/setup.py it is at least only installed
if the system version of python does not support it.

AFAICT this is only used by xend and the bit which uses pyxml is not
used.

8<--------------------------

# HG changeset patch
# User Ian Campbell <ian.campbell@xxxxxxxxxx>
# Date 1345537817 -3600
# Node ID 7ba9eaf898afafac6cf3956d01c86b861ec5d853
# Parent  b51f4c5a08480297d13439e66793a9002d9b49e9
README: Update references to PyXML to lxml

Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx>

diff -r b51f4c5a0848 -r 7ba9eaf898af README
--- a/README    Tue Aug 21 09:20:41 2012 +0100
+++ b/README    Tue Aug 21 09:30:17 2012 +0100
@@ -148,9 +148,9 @@ Xend (the Xen daemon) has the following 
           URL:    http://www.pangalactic.org/PyPAM/
           Debian: python-pam
 
-    * For optional XenAPI support in XM, PyXML:
-          URL:    http://codespeak.net/lxml/
-         Debian: python-lxml
+    * For optional XenAPI support in XM, lxml:
+          URL:    http://lxml.de/
+          Debian: python-lxml
           YUM:    python-lxml
 
 



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel