From: Alvaro Herrera <alvherre@commandprompt.com>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Josh Kupershmidt <schmiddy@gmail.com>
Cc: pgsql-docs <pgsql-docs@postgresql.org>
Subject: Re: Large SGML Cleanup
Date: Wed, 03 Nov 2010 01:12:24 -0300
Message-ID: <1288757512-sup-3380@alvh.no-ip.org> (raw)
In-Reply-To: <23257.1288754126@sss.pgh.pa.us>
References: <AANLkTi=1Sm9N3Khiued9UiMfdd_TKLimMiO9mCfHtL39@mail.gmail.com>
<23257.1288754126@sss.pgh.pa.us>
Excerpts from Tom Lane's message of mié nov 03 00:15:26 -0300 2010:
> Josh Kupershmidt <schmiddy@gmail.com> writes:
> > This HTML fails validation, as one isn't supposed to be stuffing
> > tables inside <p> nodes. The attached patch fixes all the instances of
> > this I could find, by closing out <para> nodes before beginning lists
> > and tables.
>
> I think this isn't even worth thinking about applying, unless you can
> provide a way to get future cases of that to fail during "make html".
> The chances that it'll stay fixed without such a check are not
> distinguishable from zero.
>
> An alternative that might be more workable is to fix the toolchain
> so that it generates valid HTML from what is evidently perfectly
> acceptable SGML.
Maybe we could have additional commands in the "check" rule to invoke
some HTML validator.
--
Álvaro Herrera <alvherre@commandprompt.com>
The PostgreSQL Company - Command Prompt, Inc.
PostgreSQL Replication, Consulting, Custom Development, 24x7 support
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: pgsql-docs@postgresql.org
Cc: alvherre@commandprompt.com, tgl@sss.pgh.pa.us, schmiddy@gmail.com
Subject: Re: Large SGML Cleanup
In-Reply-To: <1288757512-sup-3380@alvh.no-ip.org>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox