[Python-projects] xmldiff: build problems

[Sylvain Thénault]

Le Thursday 11 December 2008 20:00:19 Greg Ward, vous avez écrit :
> Hi --

Hi,

> I'm trying to use xmldiff 0.6.8 and encountering some simple build
> problems.  First, Python 2.5 complains about the incorrect placement of
> "from __future__ import" in the setup.py script:
>
> $ python2.5 setup.py
>   File "setup.py", line 23
>     from __future__ import nested_scopes
> SyntaxError: from __future__ imports must occur at the beginning of the
> file

this has been fixed in the repository a while ago, though not officially 
released yet :(

> The next one is a little trickier.  Building an RPM from the source tree
> fails:
>
>   $ python2.5 setup.py bdist_rpm
>   [...]
>   running install_data
>   creating /var/tmp/xmldiff-0.6.8-1-buildroot/usr/share
>   creating /var/tmp/xmldiff-0.6.8-1-buildroot/usr/share/sgml
>   creating /var/tmp/xmldiff-0.6.8-1-buildroot/usr/share/sgml/stylesheet
>   creating
> /var/tmp/xmldiff-0.6.8-1-buildroot/usr/share/sgml/stylesheet/xmldiff error:
> can't copy 'xsl/docbook_rev.xsl': doesn't exist or not a regular file
> error: Bad exit status from /var/tmp/rpm-tmp.16214 (%install)
>
>
>   RPM build errors:
>       Bad exit status from /var/tmp/rpm-tmp.16214 (%install)
>   error: command 'rpmbuild' failed with exit status 1
>
> Something is wrong with the MANIFEST file generated by your setup.py.
> I was able to fix this by running
>
>   ls -1 xsl/* >> MANIFEST
>
> after the above failure.  But I can't provide a patch, because I do not
> understand how that MANIFEST file is generated.  ;-(

that one is strange. The MANIFEST file is generated according to rules in the 
MANIFEST.in file (though the latter may not be included in release tarball). 
At least in the repository the MANIFEST.in had correct input regarding the 
xsl subdirectory.

I think you should actually used the code from the repository... (more below)

> P.S. it would be nice if the project home page had a usable link to the
> Mercurial repository.  I would be happy to clone your repo and provide
> patches relative to it.  But this:
>  
> http://www.logilab.org/embed?url=http%3A%2F%2Fwww.logilab.org%2Fsrc%2Fxmldi
>ff&rql=Any%20X%20WHERE%20X%20eid%20859&custom_css=mercurial.css&vtitle=brows
>e%20source
>
> is not a friendly or usable link.  ;-(

right, though this is only the url used to see mercurial changes through our 
web site. For use with hg, simply use the following url:
 http://www.logilab.org/src/xmldiff

much more friendly isn't it ;)

> P.P.S. thank you for writing xmldiff and making it available.  I hope I
> can get it working!

Thanks! And I hope you'll get it working as well. We haven't developed it for 
a while but it should work.

-- 
Sylvain Thénault                               LOGILAB, Paris (France)
Formations Python, Zope, Plone, Debian:  http://www.logilab.fr/formations
Développement logiciel sur mesure:       http://www.logilab.fr/services
Python et calcul scientifique:           http://www.logilab.fr/science