On 06/25/2010 12:00 PM, Matthias Bolte wrote:
-html/%-%.html html/%-virterror.html %-api.xml %-refs.xml: \ - $(srcdir)/../include/%/*.h \ - $(srcdir)/../src/%.c \ + $(XSLTPROC) --nonet -o $(srcdir)/devhelp/ \ + $(srcdir)/devhelp/devhelp.xsl $(srcdir)/libvirt-api.xml ; \ + fi
This makes xsltproc report and "I/O error : Is a directory" for me. I think the problem is that we need to specify the output file instead of the output directory here.
I am officially stumped. I cannot, for the life of me, figure out why: xsltproc --nonet -o ./ ./newapi.xsl ./libvirt-api.xml works (outputs 4 *.html files into ./), while: xsltproc --nonet -o ./devhelp/ ./devhelp/devhelp.xsl ./libvirt-api.xml outputs 4 *.html files into ./devhelp but then tries to write to ./devhelp/ as a file (hence the I/O error) rather than writing output to the fifth file devhelp/libvirt.devhelp. Since everything about the two command lines is identical except for the .xsl argument, it is obviously some difference between those two style sheets - where newapi.xsl works gracefully for creating files into a target directory but devhelp/devhelp.xsl does not. But I do not have a clue how to read those .xsl stylesheets in order to identify the difference. DV, I see your name tied to xsltproc - can you help?
This triggers tons or warnings/errors for me:
function virDomainResume from ./../include/libvirt/libvirt.h redeclared in ../include/libvirt/libvirt.h
Adding "./../include/libvirt" instead of "../include/libvirt" as last item to the list fixes this for me.
Odd, but I can fold that in. But until I can figure out how to fix that I/O error with devhelp/devhelp.xsl, this patch has no chance of being incorporated, so maybe I should first focus on minimizing the patch down to something less invasive that fixes just the missing makefile dependency. -- Eric Blake eblake@redhat.com +1-801-349-2682 Libvirt virtualization library http://libvirt.org