# [PATCH] Documentation/Makefile: Allow custom XMLTO binary

4 messages from 2012-09-19 to 2012-09-19. Participants: dborowitz@google.com, Junio C Hamano, Dave Borowitz.
Thread: https://gitlist.dev/t/31607

## dborowitz@google.com, 2012-09-19 17:06

Subject: [PATCH] Documentation/Makefile: Allow custom XMLTO binary
Message-ID: <1348074397-29978-1-git-send-email-dborowitz@google.com>
URL: https://gitlist.dev/e/1348074397-29978-1-git-send-email-dborowitz%40google.com

```
From: Dave Borowitz <dborowitz@google.com>

Signed-off-by: Dave Borowitz <dborowitz@google.com>
---
 Documentation/Makefile | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/Documentation/Makefile b/Documentation/Makefile
index cf5916f..b045628 100644
--- a/Documentation/Makefile
+++ b/Documentation/Makefile
@@ -47,6 +47,7 @@ man7dir=$(mandir)/man7
 ASCIIDOC=asciidoc
 ASCIIDOC_EXTRA =
 MANPAGE_XSL = manpage-normal.xsl
+XMLTO=xmlto
 XMLTO_EXTRA =
 INSTALL?=install
 RM ?= rm -f
@@ -245,7 +246,7 @@ manpage-base-url.xsl: manpage-base-url.xsl.in
 
 %.1 %.5 %.7 : %.xml manpage-base-url.xsl
 	$(QUIET_XMLTO)$(RM) $@ && \
-	xmlto -m $(MANPAGE_XSL) $(XMLTO_EXTRA) man $<
+	$(XMLTO) -m $(MANPAGE_XSL) $(XMLTO_EXTRA) man $<
 
 %.xml : %.txt
 	$(QUIET_ASCIIDOC)$(RM) $@+ $@ && \
-- 
1.7.12.1

```

## Junio C Hamano, 2012-09-19 17:59

Subject: Re: [PATCH] Documentation/Makefile: Allow custom XMLTO binary
Message-ID: <7vehlxhp0v.fsf@alter.siamese.dyndns.org>
URL: https://gitlist.dev/e/7vehlxhp0v.fsf%40alter.siamese.dyndns.org
In-Reply-To: <1348074397-29978-1-git-send-email-dborowitz@google.com>

```
dborowitz@google.com writes:

> From: Dave Borowitz <dborowitz@google.com>
>
> Signed-off-by: Dave Borowitz <dborowitz@google.com>
> ---

Thanks; the patch sort-of makes sense but makes me wonder what your
use case is.  Do you have xmlto2 program you want to use in place of
xmlto or you have xmlto but not on your $PATH?

>  Documentation/Makefile | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/Makefile b/Documentation/Makefile
> index cf5916f..b045628 100644
> --- a/Documentation/Makefile
> +++ b/Documentation/Makefile
> @@ -47,6 +47,7 @@ man7dir=$(mandir)/man7
>  ASCIIDOC=asciidoc
>  ASCIIDOC_EXTRA =
>  MANPAGE_XSL = manpage-normal.xsl
> +XMLTO=xmlto
>  XMLTO_EXTRA =
>  INSTALL?=install
>  RM ?= rm -f
> @@ -245,7 +246,7 @@ manpage-base-url.xsl: manpage-base-url.xsl.in
>  
>  %.1 %.5 %.7 : %.xml manpage-base-url.xsl
>  	$(QUIET_XMLTO)$(RM) $@ && \
> -	xmlto -m $(MANPAGE_XSL) $(XMLTO_EXTRA) man $<
> +	$(XMLTO) -m $(MANPAGE_XSL) $(XMLTO_EXTRA) man $<
>  
>  %.xml : %.txt
>  	$(QUIET_ASCIIDOC)$(RM) $@+ $@ && \

```

## Dave Borowitz, 2012-09-19 20:49

Subject: Re: [PATCH] Documentation/Makefile: Allow custom XMLTO binary
Message-ID: <CAD0k6qTMm7MKLYOOMaB72cWc=Z8nJfe8qseWfyM6x_NzCorgZg@mail.gmail.com>
URL: https://gitlist.dev/e/CAD0k6qTMm7MKLYOOMaB72cWc%3DZ8nJfe8qseWfyM6x_NzCorgZg%40mail.gmail.com
In-Reply-To: <7vehlxhp0v.fsf@alter.siamese.dyndns.org>

```
On Wed, Sep 19, 2012 at 10:59 AM, Junio C Hamano <gitster@pobox.com> wrote:
> dborowitz@google.com writes:
>
>> From: Dave Borowitz <dborowitz@google.com>
>>
>> Signed-off-by: Dave Borowitz <dborowitz@google.com>
>> ---
>
> Thanks; the patch sort-of makes sense but makes me wonder what your
> use case is.  Do you have xmlto2 program you want to use in place of
> xmlto or you have xmlto but not on your $PATH?

The latter; I can easily symlink it into my $PATH of course. But since
I was already specifying ASCIIDOC on the make command line for the
same reason, I was surprised to find I couldn't also specify XMLTO,
hence the patch.

>>  Documentation/Makefile | 3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/Documentation/Makefile b/Documentation/Makefile
>> index cf5916f..b045628 100644
>> --- a/Documentation/Makefile
>> +++ b/Documentation/Makefile
>> @@ -47,6 +47,7 @@ man7dir=$(mandir)/man7
>>  ASCIIDOC=asciidoc
>>  ASCIIDOC_EXTRA =
>>  MANPAGE_XSL = manpage-normal.xsl
>> +XMLTO=xmlto
>>  XMLTO_EXTRA =
>>  INSTALL?=install
>>  RM ?= rm -f
>> @@ -245,7 +246,7 @@ manpage-base-url.xsl: manpage-base-url.xsl.in
>>
>>  %.1 %.5 %.7 : %.xml manpage-base-url.xsl
>>       $(QUIET_XMLTO)$(RM) $@ && \
>> -     xmlto -m $(MANPAGE_XSL) $(XMLTO_EXTRA) man $<
>> +     $(XMLTO) -m $(MANPAGE_XSL) $(XMLTO_EXTRA) man $<
>>
>>  %.xml : %.txt
>>       $(QUIET_ASCIIDOC)$(RM) $@+ $@ && \

```

## Junio C Hamano, 2012-09-19 23:27

Subject: Re: [PATCH] Documentation/Makefile: Allow custom XMLTO binary
Message-ID: <7vvcf9egqf.fsf@alter.siamese.dyndns.org>
URL: https://gitlist.dev/e/7vvcf9egqf.fsf%40alter.siamese.dyndns.org
In-Reply-To: <CAD0k6qTMm7MKLYOOMaB72cWc=Z8nJfe8qseWfyM6x_NzCorgZg@mail.gmail.com>

```
Dave Borowitz <dborowitz@google.com> writes:

> On Wed, Sep 19, 2012 at 10:59 AM, Junio C Hamano <gitster@pobox.com> wrote:
>> dborowitz@google.com writes:
>>
>>> From: Dave Borowitz <dborowitz@google.com>
>>>
>>> Signed-off-by: Dave Borowitz <dborowitz@google.com>
>>> ---
>>
>> Thanks; the patch sort-of makes sense but makes me wonder what your
>> use case is.  Do you have xmlto2 program you want to use in place of
>> xmlto or you have xmlto but not on your $PATH?
>
> The latter; I can easily symlink it into my $PATH of course. But since
> I was already specifying ASCIIDOC on the make command line for the
> same reason, I was surprised to find I couldn't also specify XMLTO,
> hence the patch.

OK.  Was just curious.

```
