# Why is git-subtree docs XML and not HTML?
5 messages from 2013-10-29 to 2013-10-30. Participants: Sebastian Schuberth, Jeff King, Junio C Hamano.
Thread: https://gitlist.dev/t/35237
## Sebastian Schuberth, 2013-10-29 14:03
Subject: Why is git-subtree docs XML and not HTML?
Message-ID:
URL: https://gitlist.dev/e/CAHGBnuO1xPrAFG1tvS5zaVD9yefa2VUNYM93YLt1pQ1yJwHkrw%40mail.gmail.com
```
Hi,
I'm currently looking at subtree's Makefile and wondering why the
generated markup docs is git-subtree.xml, and not git-subtree.html
like for the rest of the git commands. I have not looked at
git-subtree.xml because I'm lacking asciidoc right now, but is it just
a matter of changing the file extension from .xml to .html, or does
generating a HTML file involve more work?
--
Sebastian Schuberth
```
## Jeff King, 2013-10-29 15:50
Subject: Re: Why is git-subtree docs XML and not HTML?
Message-ID: <20131029155037.GA3628@sigill.intra.peff.net>
URL: https://gitlist.dev/e/20131029155037.GA3628%40sigill.intra.peff.net
In-Reply-To:
```
On Tue, Oct 29, 2013 at 03:03:40PM +0100, Sebastian Schuberth wrote:
> I'm currently looking at subtree's Makefile and wondering why the
> generated markup docs is git-subtree.xml, and not git-subtree.html
> like for the rest of the git commands. I have not looked at
> git-subtree.xml because I'm lacking asciidoc right now, but is it just
> a matter of changing the file extension from .xml to .html, or does
> generating a HTML file involve more work?
The XML is an intermediate format for the manpage; the original source
is .txt, from which asciidoc generate docbook .xml, from which xmlto
generate the roff git-subtree.1.
If you want HTML, you need to have asciidoc generate .html from the .txt
directly (you probably _can_ generate it from docbook, but that is more
complicated, and not how regular git commands work).
I think this would do what you want:
diff --git a/contrib/subtree/Makefile b/contrib/subtree/Makefile
index 435b2de..e9026da 100644
--- a/contrib/subtree/Makefile
+++ b/contrib/subtree/Makefile
@@ -21,6 +21,7 @@ GIT_SUBTREE := git-subtree
GIT_SUBTREE_DOC := git-subtree.1
GIT_SUBTREE_XML := git-subtree.xml
GIT_SUBTREE_TXT := git-subtree.txt
+GIT_SUBTREE_HTML := git-subtree.html
all: $(GIT_SUBTREE)
@@ -46,6 +47,10 @@ $(GIT_SUBTREE_XML): $(GIT_SUBTREE_TXT)
asciidoc -b docbook -d manpage -f $(ASCIIDOC_CONF) \
-agit_version=$(gitver) $^
+$(GIT_SUBTREE_HTML): $(GIT_SUBTREE_TXT)
+ asciidoc -b xhtml11 -d manpage -f $(ASCIIDOC_CONF) \
+ -agit_version=$(gitver) $^
+
test:
$(MAKE) -C t/ test
```
## Sebastian Schuberth, 2013-10-29 16:03
Subject: Re: Why is git-subtree docs XML and not HTML?
Message-ID:
URL: https://gitlist.dev/e/CAHGBnuNDBH-V%2BEbi27BOE8ebNQdhn%2BMCvabh97_zL3nTctz4yA%40mail.gmail.com
In-Reply-To: <20131029155037.GA3628@sigill.intra.peff.net>
```
On Tue, Oct 29, 2013 at 4:50 PM, Jeff King wrote:
> I think this would do what you want:
>
> diff --git a/contrib/subtree/Makefile b/contrib/subtree/Makefile
> index 435b2de..e9026da 100644
> --- a/contrib/subtree/Makefile
> +++ b/contrib/subtree/Makefile
> @@ -21,6 +21,7 @@ GIT_SUBTREE := git-subtree
> GIT_SUBTREE_DOC := git-subtree.1
> GIT_SUBTREE_XML := git-subtree.xml
> GIT_SUBTREE_TXT := git-subtree.txt
> +GIT_SUBTREE_HTML := git-subtree.html
>
> all: $(GIT_SUBTREE)
>
> @@ -46,6 +47,10 @@ $(GIT_SUBTREE_XML): $(GIT_SUBTREE_TXT)
> asciidoc -b docbook -d manpage -f $(ASCIIDOC_CONF) \
> -agit_version=$(gitver) $^
>
> +$(GIT_SUBTREE_HTML): $(GIT_SUBTREE_TXT)
> + asciidoc -b xhtml11 -d manpage -f $(ASCIIDOC_CONF) \
> + -agit_version=$(gitver) $^
> +
> test:
> $(MAKE) -C t/ test
>
I just gave it a try and it works nicely:
Tested-by: Sebastian Schuberth
Also, the clean target already contains "*.html", so nothing else to
do there. Any chance to get this queued in pu?
Thanks a lot!
--
Sebastian Schuberth
```
## Jeff King, 2013-10-29 16:30
Subject: [PATCH] subtree: add makefile target for html docs
Message-ID: <20131029163037.GB3628@sigill.intra.peff.net>
URL: https://gitlist.dev/e/20131029163037.GB3628%40sigill.intra.peff.net
In-Reply-To:
```
On Tue, Oct 29, 2013 at 05:03:53PM +0100, Sebastian Schuberth wrote:
> I just gave it a try and it works nicely:
>
> Tested-by: Sebastian Schuberth
>
> Also, the clean target already contains "*.html", so nothing else to
> do there. Any chance to get this queued in pu?
Here it is with a commit message, and the additional wiring into "make
doc".
-- >8 --
Subject: subtree: add makefile target for html docs
The Makefile currently builds the roff manpage, but not the
html form. As some people may prefer the latter, let's make
it an option to build that, too. We also wire it into "make
doc" so that it is built by default.
This patch does not build or install it as part of
"install-doc"; that would require extra infrastructure to
handle installing the html as we do in git's regular
Documentation/ tree. That can come later if somebody is
interested.
Tested-by: Sebastian Schuberth
Signed-off-by: Jeff King
---
contrib/subtree/Makefile | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/contrib/subtree/Makefile b/contrib/subtree/Makefile
index 435b2de..4030a16 100644
--- a/contrib/subtree/Makefile
+++ b/contrib/subtree/Makefile
@@ -21,13 +21,14 @@ GIT_SUBTREE := git-subtree
GIT_SUBTREE_DOC := git-subtree.1
GIT_SUBTREE_XML := git-subtree.xml
GIT_SUBTREE_TXT := git-subtree.txt
+GIT_SUBTREE_HTML := git-subtree.html
all: $(GIT_SUBTREE)
$(GIT_SUBTREE): $(GIT_SUBTREE_SH)
cp $< $@ && chmod +x $@
-doc: $(GIT_SUBTREE_DOC)
+doc: $(GIT_SUBTREE_DOC) $(GIT_SUBTREE_HTML)
install: $(GIT_SUBTREE)
$(INSTALL) -d -m 755 $(DESTDIR)$(libexecdir)
@@ -46,6 +47,10 @@ $(GIT_SUBTREE_XML): $(GIT_SUBTREE_TXT)
asciidoc -b docbook -d manpage -f $(ASCIIDOC_CONF) \
-agit_version=$(gitver) $^
+$(GIT_SUBTREE_HTML): $(GIT_SUBTREE_TXT)
+ asciidoc -b xhtml11 -d manpage -f $(ASCIIDOC_CONF) \
+ -agit_version=$(gitver) $^
+
test:
$(MAKE) -C t/ test
--
1.8.4.1.898.g8bf8a41.dirty
```
## Junio C Hamano, 2013-10-30 17:48
Subject: Re: [PATCH] subtree: add makefile target for html docs
Message-ID:
URL: https://gitlist.dev/e/xmqqwqkubqff.fsf%40gitster.dls.corp.google.com
In-Reply-To: <20131029163037.GB3628@sigill.intra.peff.net>
```
Jeff King writes:
> On Tue, Oct 29, 2013 at 05:03:53PM +0100, Sebastian Schuberth wrote:
>
>> I just gave it a try and it works nicely:
>>
>> Tested-by: Sebastian Schuberth
>>
>> Also, the clean target already contains "*.html", so nothing else to
>> do there. Any chance to get this queued in pu?
>
> Here it is with a commit message, and the additional wiring into "make
> doc".
Thanks, both.
```