Re: [PATCH xorg-docs 2/2] Replace XMLTO with XSLTPROC, XMLLINT and W3M tools

2011-03-26 Thread Dan Nicholson
On Fri, Mar 25, 2011 at 10:21 AM, Gaetan Nadon mems...@videotron.ca wrote: On Fri, 2011-03-25 at 09:28 -0600, Matt Dew wrote: Dan,  which justifications do you feel are incorrect?  Agreed that accurate commit msgs are important so we'll try to clear them up. I think the issue has been

Re: [PATCH xorg-docs 2/2] Replace XMLTO with XSLTPROC, XMLLINT and W3M tools

2011-03-25 Thread Matt Dew
On 03/22/2011 06:14 PM, Dan Nicholson wrote: On Tue, Mar 22, 2011 at 4:28 PM, Matt Dewmar...@osource.org wrote: On 03/22/2011 03:25 PM, Dan Nicholson wrote: On Tue, Mar 22, 2011 at 7:19 AM, Gaetan Nadonmems...@videotron.ca wrote: Xmlto is a script that selects the appropriate back-end

Re: [PATCH xorg-docs 2/2] Replace XMLTO with XSLTPROC, XMLLINT and W3M tools

2011-03-25 Thread Gaetan Nadon
On Fri, 2011-03-25 at 09:28 -0600, Matt Dew wrote: Dan, which justifications do you feel are incorrect? Agreed that accurate commit msgs are important so we'll try to clear them up. I think the issue has been resolved with the new series I have posted and Dan has acknowledged the

Re: [PATCH xorg-docs 2/2] Replace XMLTO with XSLTPROC, XMLLINT and W3M tools

2011-03-25 Thread Matt Dew
On 03/25/2011 11:21 AM, Gaetan Nadon wrote: On Fri, 2011-03-25 at 09:28 -0600, Matt Dew wrote: Dan, which justifications do you feel are incorrect? Agreed that accurate commit msgs are important so we'll try to clear them up. I think the issue has been resolved with the new series I have

Re: [PATCH xorg-docs 2/2] Replace XMLTO with XSLTPROC, XMLLINT and W3M tools

2011-03-25 Thread Matt Dew
On 03/23/2011 01:58 PM, Gaetan Nadon wrote: On Tue, 2011-03-22 at 14:25 -0700, Dan Nicholson wrote: If you pass -x to xmlto instead of -m, you can use a full stylesheet instead of a fragment. Moving to xsltproc means you have to duplicate the internal smarts of xmlto. I have verified -x works

Re: [PATCH xorg-docs 2/2] Replace XMLTO with XSLTPROC, XMLLINT and W3M tools

2011-03-23 Thread Gaetan Nadon
On Tue, 2011-03-22 at 14:25 -0700, Dan Nicholson wrote: If you pass -x to xmlto instead of -m, you can use a full stylesheet instead of a fragment. Moving to xsltproc means you have to duplicate the internal smarts of xmlto. I have verified -x works as described. This removes one issue with

Re: [PATCH xorg-docs 2/2] Replace XMLTO with XSLTPROC, XMLLINT and W3M tools

2011-03-23 Thread Gaetan Nadon
On Tue, 2011-03-22 at 17:14 -0700, Dan Nicholson wrote: More importantly, xmlto always uses --nonet for xsltproc and xmllint. Do you really want builds to hang trying to fetch sheets over the network? You guys are the ones doing the doc work, so I won't try to slow you down anymore. Please

[PATCH xorg-docs 2/2] Replace XMLTO with XSLTPROC, XMLLINT and W3M tools

2011-03-22 Thread Gaetan Nadon
Xmlto is a script that selects the appropriate back-end based on options and tools availability. Xmlto uses an xsl fragment which is not compatible with the standard use of xsl stylesheets. The customization for xhtml and fo cannot be used with xsltproc in that context. It makes adoption of

Re: [PATCH xorg-docs 2/2] Replace XMLTO with XSLTPROC, XMLLINT and W3M tools

2011-03-22 Thread Dan Nicholson
On Tue, Mar 22, 2011 at 7:19 AM, Gaetan Nadon mems...@videotron.ca wrote: Xmlto is a script that selects the appropriate back-end based on options and tools availability. Xmlto uses an xsl fragment which is not compatible with the standard use of xsl stylesheets. The customization for xhtml

Re: [PATCH xorg-docs 2/2] Replace XMLTO with XSLTPROC, XMLLINT and W3M tools

2011-03-22 Thread Matt Dew
On 03/22/2011 03:25 PM, Dan Nicholson wrote: On Tue, Mar 22, 2011 at 7:19 AM, Gaetan Nadonmems...@videotron.ca wrote: Xmlto is a script that selects the appropriate back-end based on options and tools availability. Xmlto uses an xsl fragment which is not compatible with the standard use of

Re: [PATCH xorg-docs 2/2] Replace XMLTO with XSLTPROC, XMLLINT and W3M tools

2011-03-22 Thread Dan Nicholson
On Tue, Mar 22, 2011 at 4:28 PM, Matt Dew mar...@osource.org wrote: On 03/22/2011 03:25 PM, Dan Nicholson wrote: On Tue, Mar 22, 2011 at 7:19 AM, Gaetan Nadonmems...@videotron.ca  wrote: Xmlto is a script that selects the appropriate back-end based on options and tools availability. Xmlto