Re: documentation pdf build fail (HEAD)

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>
Cc: Erikjan Rijkers <er(at)xs4all(dot)nl>, pgsql-hackers(at)postgresql(dot)org, pgsql-docs(at)lists(dot)postgresql(dot)org
Subject: Re: documentation pdf build fail (HEAD)
Date: 2020-03-24 15:09:46
Message-ID: 14757.1585062586@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-docs pgsql-hackers

Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> writes:
> On 2020-03-24 15:31, Tom Lane wrote:
>> The problem seems to be that cedffbdb8
>> introduced some broken table markup. I wonder why xmllint
>> failed to catch it?

> It's not a validity issue in the DocBook markup. The error comes from
> FOP, which complains because it requires the column count, but other
> processors don't necessarily require it.

Maybe not, but if the count is there, shouldn't it be checked?

In this particular case, the table was obviously broken if you looked
at the rendered HTML, but I'd kind of expect the toolchain to provide
basic sanity checks without having to do that.

regards, tom lane

In response to

Browse pgsql-docs by date

  From Date Subject
Next Message PG Doc comments form 2020-03-24 15:57:25 terminology in example
Previous Message Peter Eisentraut 2020-03-24 14:46:59 Re: documentation pdf build fail (HEAD)

Browse pgsql-hackers by date

  From Date Subject
Next Message Sergei Kornilov 2020-03-24 15:17:58 Re: replay pause vs. standby promotion
Previous Message David Steele 2020-03-24 15:08:01 Re: extension patch of CREATE OR REPLACE TRIGGER