public inbox for [email protected]  

help / color / mirror / Atom feed

From: Alexander Law <[email protected]>
To: Peter Eisentraut <[email protected]>
To: [email protected]
To: Alvaro Herrera <[email protected]>
To: Alexander Lakhin <[email protected]>
Cc: Jürgen Purtz <[email protected]>
Cc: [email protected]
Subject: Re: Docbook 5.x
Date: Sat, 4 Jun 2016 17:28:41 +0300
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
	<[email protected]>
	<CAF4Au4zUVOuWjWc+B72yvvpVo=TJad6=x9chig7Ls9DNa7Kgrg@mail.gmail.com>
	<[email protected]>
	<[email protected]>
	<[email protected]>
List-Unsubscribe: <mailto:[email protected]?body=unsub%20pgsql-docs>

Hello Peter,

Thanks for improvements!
I checked outputs (with the attached script) and found that there is 
small difference with the improved patch.
For example, look at xtypes.html:

- <link rel="home" href="index.html" title="PostgreSQL 9.6beta1 
Documentation" /><link rel="up" href="extend.html" 
title="Chapter 35. Extending SQL" /><link rel="prev" href="xaggr.html" 
title="35.10. User-defined Aggregates" /><link rel="next" 
href="xoper.html" title="35.12. User-defined Operators" /><link 
rel="copyright" href="legalnotice.html" title="Legal Notice" /></head>

+ <link rel="prev" href="xaggr.html" title="35.10. User-defined 
Aggregates" /><link rel="next" href="xoper.html" 
title="35.12. User-defined Operators" /></head>

It caused by <xsl:template name="html.head">.
Leaving aside the question whether the links "home", "up" and 
"copyright" are needed, maybe it's better to split the commit to two?
First to speed up the conversion while making sure that the output is 
the same, and the second to change the html.head output format.

Best regards,
Alexander


03.06.2016 22:31, Peter Eisentraut пишет:
> On 5/5/16 12:45 AM, Alexander Law wrote:
>> Done (previous patch cleaned).
>> This patch optimizes XSL transformations contained in docbook-xsl 
>> (1.78.1).
>
> I have looked through this patch, and it's awesome.  I have tweaked it 
> a bit more along the lines you guys have started, and now the build 
> time is pretty much the same as with DSSSL. Attached is my final 
> patch, which I plan to commit as soon as the new branch opens.
>
> (I only have Alexander Lakhin as credit right now.  Please let me know 
> if anyone else contributed.)
>



-- 
Sent via pgsql-docs mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-docs


Attachments:

  [application/x-shellscript] check_patch.sh (2.1K, 2-check_patch.sh)
  download

view thread (86+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected], [email protected], [email protected]
  Subject: Re: Docbook 5.x
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox