Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dsvlI-0002AH-CN for pgsql-docs@arkaria.postgresql.org; Fri, 15 Sep 2017 18:54:48 +0000 Received: from localhost ([127.0.0.1] helo=postgresql.org) by malur.postgresql.org with smtp (Exim 4.84_2) (envelope-from ) id 1dsvlH-0001wo-6z for pgsql-docs@arkaria.postgresql.org; Fri, 15 Sep 2017 18:54:47 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA384:256) (Exim 4.84_2) (envelope-from ) id 1dsvlG-0001vp-CJ for pgsql-docs@postgresql.org; Fri, 15 Sep 2017 18:54:46 +0000 Received: from mout.kundenserver.de ([217.72.192.75]) by magus.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.84_2) (envelope-from ) id 1dsvlC-0004Hk-DW for pgsql-docs@postgresql.org; Fri, 15 Sep 2017 18:54:45 +0000 Received: from [192.168.0.133] ([212.96.75.72]) by mrelayeu.kundenserver.de (mreue104 [212.227.15.183]) with ESMTPSA (Nemesis) id 0Lzadu-1dOsBB0QEc-014mbx; Fri, 15 Sep 2017 20:54:39 +0200 Subject: Re: Docbook 5.x To: Peter Eisentraut References: <57179283.6080704@purtz.de> <20160503193441.GA61759@alvherre.pgsql> <572AD007.60900@gmail.com> <5752E599.2090505@gmail.com> <576d0623-a89c-b3de-e321-dc48a579ff1a@2ndquadrant.com> <4adecfc6-2f2e-2ff2-bfa3-58b7d397227b@gmail.com> <8f227b2a-5093-8d99-85da-ea00e18343f6@gmail.com> <449e34c4-9cc8-d17d-5ebe-be92b4c0a87a@gmail.com> <09b13d85-e259-2464-723a-467210264afe@2ndquadrant.com> <9e8cafe4-7cfd-59be-6856-172c15dc758f@2ndquadrant.com> Cc: pgsql-docs@postgresql.org, Alexander Law From: =?UTF-8?Q?J=c3=bcrgen_Purtz?= Message-ID: Date: Fri, 15 Sep 2017 23:54:57 +0500 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 MIME-Version: 1.0 In-Reply-To: <9e8cafe4-7cfd-59be-6856-172c15dc758f@2ndquadrant.com> Content-Type: multipart/mixed; boundary="------------E7CC9317457499C48582C7A1" Content-Language: de-DE X-Provags-ID: V03:K0:4BbjiD7Di+jV7QVaAPB8DTrlhO50i3H0NEZFRK4GqHLVpovd6w/ 67rsZqVYXB+0mLz3WgcKIP97eW3tNsoRLJtrGYXkWT3wXUD48FRg/Q3O/yQZ7ax83qDo5XB pqwbvB4ExnQ+QBs1wS8WOY8KU9Sv6OC0O+ghR7s5Vg4wY4skvCunk2BOLyKX3enBJKJqaNE 6AhLP32OTQiawtQoFQERg== X-UI-Out-Filterresults: notjunk:1;V01:K0:PkzJDAO20g0=:YaVf/o7qCJxzFJ7f8w0nto 3N4tZXBrDh5mfdmPPFtHLRIUG/JNcsfLXv0vjLtoK81ZI//XEqj7R0lL/puyMa0F/3JnnABn/ kysQoi2Ya/Yg0lo0NSVfYbSZU8YF9FIk6W99UpLn6mUcfohuFhGqIwmgSBdbAHH/7+R9gYJk0 YgsMCWtrT/4ZPRvLHJrlwiZGevhl+PEO0cxK0GJxcwJnzvFmQa7/M5YFEXE3fDNN7yKVo4qXr 1rVaWiCumwzFhtgp6Y/N5iHKLriF8oEIGNVwa2aJPcUCWBD5Bk1DptLgNIdx3ksJKw7kMfQh5 ma6zdMxVvmznc4l4ASEFndzn3p+KcS82DxfNpGdQzNt0NNBsPT2Ho9nzfUOKh5COunRLtTWW8 fiXGQk14t49UNhXjPg3PqW3R9igeICeGRSLTjVpZmVje7mjg+KD+N7edt3+dZ2gd56+D9J/CP E6uQvF+wBFnxkzfEi//MDSaDJWeT1Qda6WMvrDYALj9WIwhlEzG+U1GfBo/9NboE/zp1L1J0J R4k7FXWsj7ZI433ajq4XpuuBPO3j8AjMctd2KswvW5gOlrlvERICope+11qBfI3CTaOSy3k2q SqL4o2xhzUHkuxJkvL0TnsKFm4ooEco1LpzJkt9AYIKY97KqaOZX4RHGgTaOywATNZmFtbrPp zxY0uHZF0920Nn4DaXMX5OuuajVyJT7q/H9w8k9+3LdP+MCS4Hi9NJBGCRlpJANgb0OuhFPoz iDv0lJ42ZtAezoMwgkuprGyB7zUtHPTy8ex1ig== List-Archive: List-Help: List-ID: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: X-Mailing-List: pgsql-docs Precedence: bulk Sender: pgsql-docs-owner@postgresql.org This is a multi-part message in MIME format. --------------E7CC9317457499C48582C7A1 Content-Type: multipart/alternative; boundary="------------4F46314A3FE2CCF71903551E" --------------4F46314A3FE2CCF71903551E Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit On 15.09.2017 19:32, Peter Eisentraut wrote: > On 9/8/17 08:30, Alexander Lakhin wrote: >>> I have started working through these patches now. I have committed the >>> escaping of < and & and will work through the rest slowly, to minimize >>> disruptions to other development. >> Great! >> >> I have rebased all the remaining patches and updated scripts for the >> current master (see attachment). > So, I've been looking at this profiling stuff, to replace the marked > sections in the installation instructions. I found the overhead of that > a bit too much for building the full documentation, so I have come up > with the attached alternative solution. What do you think? > I'm not happy with the 'particular conversions'-part of 'standalone-profile.xsl'. It applies subsequent modifications, which are in not very intuitive to a reader, eg:   the documentation about client authentication and libpq This approach spreads the intended text over two very different files (in this example: 'installation.xml' and 'standalone-profile.xsl'). My suggestion is to keep the source code in one file in the same manner as with the SGML standalone-include/standalone-ignore mechanism. A *generic* xsl file shall create the extended output similar to 'standalone-profile.xsl'. installation.xml: support for authentication and connection parameter lookup (see the documentation about client authentication and libpq and for more information). On Unix, ... collectAll.xsl (similar to standalone-profile.xsl):                                                                   I'm sorry that I actually cannot deliver a patch because I'm abroad and have limited resources (but many challenges). But I hope that the idea gets clear. The attached collectAll.xsl file contains a more complex solution for the case that we have to deal with more than one include/ignore type, eg: index-generating. --------------4F46314A3FE2CCF71903551E Content-Type: text/html; charset=utf-8 Content-Transfer-Encoding: 8bit
On 15.09.2017 19:32, Peter Eisentraut wrote:
On 9/8/17 08:30, Alexander Lakhin wrote:

I have started working through these patches now. I have committed the 
escaping of < and & and will work through the rest slowly, to minimize 
disruptions to other development.

Great!

I have rebased all the remaining patches and updated scripts for the 
current master (see attachment).

So, I've been looking at this profiling stuff, to replace the marked
sections in the installation instructions.  I found the overhead of that
a bit too much for building the full documentation, so I have come up
with the attached alternative solution.  What do you think?

I'm not happy with the 'particular conversions'-part of 'standalone-profile.xsl'. It applies subsequent modifications, which are in not very intuitive to a reader, eg:

<xsl:template match="phrase[@id='install-ldap-links']">
  <xsl:text>the documentation about client authentication and libpq</xsl:text>
</xsl:template>

This approach spreads the intended text over two very different files (in this example: 'installation.xml' and 'standalone-profile.xsl').

My suggestion is to keep the source code in one file in the same manner as with the SGML standalone-include/standalone-ignore mechanism. A generic xsl file shall create the extended output similar to 'standalone-profile.xsl'.


installation.xml:

support for authentication and connection parameter lookup (see
<phrase condition="standalone">the documentation about client authentication and libpq</phrase>
<phrase condition="default"><xref linkend="libpq-ldap"/> and <xref linkend="auth-ldap"/></phrase>
for more information). On Unix,
...


collectAll.xsl (similar to standalone-profile.xsl):

  <!-- parameters and variables -->
  <xsl:param name="pg.Standalone" select="'default'"/>
  <!-- <xsl:param name="pg.Standalone" select="'standalone'"/>  -->

  <!-- Process all nodes  -->
  <xsl:template match="*|@*|text()|processing-instruction()|comment()">
    <xsl:choose>
      <xsl:when test="(not (@condition) or @condition=$pg.Standalone )">
        <!-- copy nodes without a 'condition' attribute and such nodes, where 'condition' meets  the given criteria  -->
        <xsl:copy>
          <xsl:apply-templates select="*|@*|text()|processing-instruction()|comment()"/>
        </xsl:copy>
      </xsl:when>
    </xsl:choose>
  </xsl:template>

I'm sorry that I actually cannot deliver a patch because I'm abroad and have limited resources (but many challenges). But I hope that the idea gets clear. The attached collectAll.xsl file contains a more complex solution for the case that we have to deal with more than one include/ignore type, eg: index-generating.


--------------4F46314A3FE2CCF71903551E-- --------------E7CC9317457499C48582C7A1 Content-Type: application/xslt+xml; name="collectAll.xsl" Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="collectAll.xsl" PD94bWwgdmVyc2lvbj0iMS4wIiBlbmNvZGluZz0iVVRGLTgiPz4NCg0KPCEt LSAgDQoNCiAgVGhlIHN0eWxlc2hlZXQgY29waWVzIGFueSBYTUwtZmlsZSBv bmUtYnktb25lOiBlbGVtZW50cywgYXR0cmlidXRlcywgdGV4dHMsIFBJcywN CiAgY29tbWVudHMgKERPQ1RZUEUgZGVjbGFyYXRpb25zIGdldCBsb3N0KSAt IHdpdGggb25lIGV4Y2VwdGlvbjogdGhlIERvY2Jvb2sNCiAgY29tbW9uIGF0 dHJpYnV0ZSAnY29uZGl0aW9uJyBpcyBpbnRlcnByZXRlZCBpbiB0aGUgc2Vu c2Ugb2YgYW4gJ2luY2x1ZGUvZXhjbHVkZScNCiAgY29tbWFuZC4gVGhlIHJ1 bGVzIG9mIGl0cyBldmFsdWF0aW9uIGFyZQ0KICAgIC0gaWYgdGhlIGF0dHJp YnV0ZSBpcyBub3QgcHJlc2VudCB0aGUgZWxlbWVudCBpcyBjb3BpZWQgdG8g dGhlIG91dHB1dA0KICAgIC0gaWYgdGhlIGF0dHJpYnV0ZSBpcyBwcmVzZW50 IGFuZCBpdHMgdmFsdWUgbWVldHMgb25lIG9mIHRoZSBwYXJhbWV0ZXJzLCB3 aGljaCBhcmUNCiAgICAgIGdpdmVuIHRvIHRoZSBzdHlsZXNoZWV0LCB0aGUg ZWxlbWVudCBpcyBjb3BpZWQgdG8gdGhlIG91dHB1dCBsaWtld2lzZQ0KICAg IC0gZWxzZSAocHJlc2VudCwgYnV0IG5vIG1hdGNoKTogdGhlIGVsZW1lbnQg YW5kIGl0cyBzdWItbm9kZXMgYXJlIGlnbm9yZWQgZnJvbQ0KICAgICAgZnVy dGhlciBwcm9jZXNzaW5nLg0KICBJbiBjb25zZXF1ZW5jZSwgZWxlbWVudHMg d2l0aCB0aGUgJ2NvbmRpdGlvbicgYXR0cmlidXRlIGJ1dCB3aXRoIGFuIG5v dC1tYXRjaGluZw0KICBhdHRyaWJ1dGUgdmFsdWUgYXJlIGVsaW1pbmF0ZWQg ZnJvbSB0aGUgb3V0cHV0LiAoSXQncyBhIHN1YnN0aXR1dGlvbiBmb3IgU0dN TCdzDQogIElOQ0xVREUvSUdOT1JFIG1lY2hhbmlzbS4pDQoNCiAgZS5nLjog VGhlIGVsZW1lbnQgPHNlY3QxIGNvbmRpdGlvbj0ic3RhbmRhbG9uZSI+Li4u PHNlY3QxPiBnb2VzIHRvIHRoZSANCiAgb3V0cHV0IGlmIHRoZSBzdHlsZXNo ZWV0IGlzIGNhbGxlZCB3aXRoICIkcGdTdGFuZGFsb25lIHN0YW5kYWxvbmUi IC0gYnV0IG5vdCBpZg0KICBpdCBpcyBjYWxsZWQgd2l0aCAiJHBnU3RhbmRh bG9uZSBhbnlPdGhlclZhbHVlIi4NCg0KICBQYXJhbWV0ZXJzOg0KICAgICRw Zy5EZWZhdWx0T3JTdGFuZGFsb25lDQogICAgJHBnLkluZGV4DQogICAgJHBn LlhzbHRJbmRleA0KDQogIEhpbnQ6DQogICAgVGhlIHByb2Nlc3Npbmcgb2Yg dGhlIHN0eWxlc2hlZXQgY2FuIGJlIGNvbWJpbmVkIHdpdGggeGk6aW5jbHVk ZSBwcm9jZXNzaW5nLCANCiAgICBlLmcuOiAieHNsdHByb2MgLXhpbmNsdWRl IC4uLiAiIHRvIGdlbmVyYXRlIGEgc2luZ2xlIGZpbGUgd2l0aCB0aGUgY29t cGxldGUNCiAgICBQb3N0Z3JlU1FMIHJhdyBkb2N1bWVudGF0aW9uLg0KDQot LT4NCg0KPHhzbDpzdHlsZXNoZWV0IHZlcnNpb249IjEuMCIgeG1sbnM6eHNs PSJodHRwOi8vd3d3LnczLm9yZy8xOTk5L1hTTC9UcmFuc2Zvcm0iPg0KDQog IDx4c2w6b3V0cHV0IG1ldGhvZD0ieG1sIiB2ZXJzaW9uPSIxLjAiLz4NCg0K ICA8IS0tIHBhcmFtZXRlcnMgYW5kIHZhcmlhYmxlcyAtLT4NCiAgPHhzbDpw YXJhbSBuYW1lPSJwZy5EZWZhdWx0T3JTdGFuZGFsb25lIiBzZWxlY3Q9Iidk ZWZhdWx0JyIvPiANCiAgPHhzbDpwYXJhbSBuYW1lPSJwZy5JbmRleCIgICAg ICAgICAgICAgICBzZWxlY3Q9IidObyciLz4gICA8IS0tIGNob29zZSAiTm8g SW5kZXgiIGFzIHRoZSBkZWZhdWx0IC0tPg0KICA8eHNsOnBhcmFtIG5hbWU9 InBnLlhzbHRJbmRleCIgICAgICAgICAgIHNlbGVjdD0iJ05vJyIvPg0KDQog IDwhLS0gY2hvb3NlICJOT1QgU1RBTkRBTE9ORSIgYXMgdGhlIGRlZmF1bHQg LS0+DQogIDx4c2w6dmFyaWFibGUgbmFtZT0icGcuRGVmYXVsdCI+DQogICAg PHhzbDpjaG9vc2U+DQogICAgICA8eHNsOndoZW4gdGVzdD0iJHBnLkRlZmF1 bHRPclN0YW5kYWxvbmUgPSAnZGVmYXVsdCciPmRlZmF1bHQ8L3hzbDp3aGVu Pg0KICAgICAgPHhzbDpvdGhlcndpc2U+bm88L3hzbDpvdGhlcndpc2U+DQog ICAgPC94c2w6Y2hvb3NlPg0KICA8L3hzbDp2YXJpYWJsZT4NCiAgPHhzbDp2 YXJpYWJsZSBuYW1lPSJwZy5TdGFuZGFsb25lIj4NCiAgICA8eHNsOmNob29z ZT4NCiAgICAgIDx4c2w6d2hlbiB0ZXN0PSIkcGcuRGVmYXVsdE9yU3RhbmRh bG9uZSA9ICdkZWZhdWx0JyI+bm88L3hzbDp3aGVuPg0KICAgICAgPHhzbDpv dGhlcndpc2U+c3RhbmRhbG9uZTwveHNsOm90aGVyd2lzZT4NCiAgICA8L3hz bDpjaG9vc2U+DQogIDwveHNsOnZhcmlhYmxlPg0KICA8IS0tIGl0J3MgcG9z c2libGUgdG8gY2hvb3NlIEluZGV4IEFORCBYc2x0SW5kZXguIG9rID8/Pz8/ IGRvIHdlIGhhdmUgYSBkZWZhdWx0PyAtLT4NCg0KDQogIDwhLS0gUHJvY2Vz cyBhbGwgbm9kZXMgIC0tPg0KICA8eHNsOnRlbXBsYXRlIG1hdGNoPSIqfEAq fHRleHQoKXxwcm9jZXNzaW5nLWluc3RydWN0aW9uKCl8Y29tbWVudCgpIj4N CiAgICA8eHNsOmNob29zZT4NCiAgICAgIDx4c2w6d2hlbiB0ZXN0PSIobm90 IChAY29uZGl0aW9uKSBvciBAY29uZGl0aW9uPSRwZy5EZWZhdWx0IG9yIEBj b25kaXRpb249JHBnLlN0YW5kYWxvbmUNCiAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICBvciBAY29uZGl0aW9uPSRwZy5JbmRleCAg IG9yIEBjb25kaXRpb249JHBnLlhzbHRJbmRleCkiPg0KICAgICAgICA8IS0t IGNvcHkgbm9kZXMgd2l0aG91dCBhICdjb25kaXRpb24nIGF0dHJpYnV0ZSBh bmQgc3VjaCBub2Rlcywgd2hlcmUgJ2NvbmRpdGlvbicgbWVldHMNCiAgICAg ICAgICAgICBvbmUgb2YgdGhlIGdpdmVuIGNyaXRlcmlhcyAgLS0+DQogICAg ICAgIDx4c2w6Y29weT4NCiAgICAgICAgICA8eHNsOmFwcGx5LXRlbXBsYXRl cyBzZWxlY3Q9Iip8QCp8dGV4dCgpfHByb2Nlc3NpbmctaW5zdHJ1Y3Rpb24o KXxjb21tZW50KCkiLz4NCiAgICAgICAgPC94c2w6Y29weT4NCiAgICAgIDwv eHNsOndoZW4+DQogICAgPC94c2w6Y2hvb3NlPg0KICA8L3hzbDp0ZW1wbGF0 ZT4NCg0KPC94c2w6c3R5bGVzaGVldD4NCg0K --------------E7CC9317457499C48582C7A1 Content-Type: text/plain Content-Disposition: inline Content-Transfer-Encoding: 8bit MIME-Version: 1.0 -- Sent via pgsql-docs mailing list (pgsql-docs@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-docs --------------E7CC9317457499C48582C7A1--