cgi/query-pr.cgi does not show things correctly

[Shaun Amott]

On Wed, Nov 15, 2006 at 01:28:38PM -0600, Mark Linimon wrote:
> 
> On Wed, Nov 15, 2006 at 06:38:36PM +0000, Bjoern A. Zeeb wrote:
> > Just in case someone wants to look at that and fix it;)
> 
> I don't understand exactly what the failure mode was, but after beating
> on it for a while it looks "better" but still not right.
> 

I've fixed the PR. It looks reasonable now.

Unfortunately, due to the free-form nature of GNATS, PRs are very
difficult to parse reliably, and adding bits in manually aggravates the
situation somewhat.

For future reference:

- The script expects a single space before each line of a reply (message
  body only).

- Adding "--- patchname.diff (begins|ends) here ---" before/after
  patches allows them to be easily found.

- The PR's URL (http://www.freebsd.org/cgi/query-pr.cgi?pr=xxxxx) is
  expected after Responsible-Changed-Why on a line of its own.

- There should be a single empty line between each reply/state-change
  block.

If you find any more badly-handled PRs (that can't be fixed with the
above), let me know.

Shaun

-- 
Shaun Amott // PGP: 0x6B387A9A
"A foolish consistency is the hobgoblin
of little minds." - Ralph Waldo Emerson
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/freebsd-www/attachments/20061127/5a05d5d4/attachment.pgp