[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [RESEND PATCHv3 2/3] build: fix VPATH build of docs



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

Attachment: signature.asc
Description: OpenPGP digital signature


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]