Re: [Xen-devel] [PATCH] docs: make 'gmake dist-docs' work

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Mon, 2013-04-22 at 12:55 +0100, Egger Christoph wrote:
> doc: Makefile fixes
>     
> - use correct pathes (make gmake dist-docs from toplevel directory work)
> - use tools as found by configure (perl)

Did you test this? Because it causes the hypercall docs to not be built
or installed for me.
        w /local/scratch/ianc/devel/committer.git/docs/xen-headers -O 
html/hypercall \
                        -T 'arch-x86_64 - Xen public headers' \
                        -X arch-ia64 -X arch-x86_32 -X xen-x86_32 -X arch-arm \
                        /local/scratch/ianc/devel/committer.git/docs/../xen 
include/public include/xen/errno.h
        w: invalid option -- 'O'
        
        Usage:
         w [options]
        
        Options:
         -h, --no-header     do not print header
         -u, --no-current    ignore current process username
         -s, --short         short format
         -f, --from          show remote hostname field
         -o, --old-style     old style output
        
             --help     display this help and exit
         -V, --version  output version information and exit
        
        For more details see w(1).
        make[1]: [html/hypercall/index.html] Error 1 (ignored)

$(PERL) is defined in config/Tools.mk but docs/Makefile by design only
includes config/Docs.mk. I think you need to check for perl in
docs/configure.ac as well and propagate the result into config/Docs.mk
        AC_ARG_VAR([PERL], [Path to Perl parser])
        AX_PATH_PROG_OR_FAIL([PERL], [perl])
        
The reason for checking twice is to support people who only want to
build the docs and not the tools and therefore only run configure in the
subdirectory.

Ian.



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