git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Aw: Re: Re: [PATCH] Remove pdf target from Makefiles
@ 2013-06-18 17:30 Thomas Ackermann
  2013-06-18 18:14 ` Jonathan Nieder
  0 siblings, 1 reply; 2+ messages in thread
From: Thomas Ackermann @ 2013-06-18 17:30 UTC (permalink / raw)
  To: jrnieder, th.acker; +Cc: git

> 
> If I understood the original commit message correctly, you were saying
> the XML file was not suitable for html generation and you wanted to
> tweak it, and were dropping the PDF target to avoid breaking it.  Now
> if I understand correctly you are saying the XML file actually *is*
> suitable for html generation, and that the html generation rules just
> need tweaking.  In that case, why remove the PDF target?
> 
The latter is correct ;-)
In my commit message I talked about style files meaning
/etc/asciidoc/dblatex/asciidoc-dblatex.xsl and /etc/asciidoc/dblatex/asciidoc-dblatex.sty
which are used when creating user-manual.pdf out of user-manual.xml
and ./Documentation/docbook.xsl which is used for creating user-manual.html
out of user-manual.xml.
When I want to tweak the html generation rules I also have to tweak the pdf
generation rules because html and pdf should be as similiar to each other as
possible. But the pdf rules are global rules so we have to introduce rules
local to git and also tweak both rule sets in parallel.
I asked myself "Is this really worth the effort or should we drop user-manual.pdf 
and spend our efforts better on user-manual.txt and user-manual.html?"
and so came up with this patch ...


---
Thomas

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2013-06-18 18:14 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-06-18 17:30 Aw: Re: Re: [PATCH] Remove pdf target from Makefiles Thomas Ackermann
2013-06-18 18:14 ` Jonathan Nieder

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).