Horton

Re: Missing [NO] INDENT flag in XMLSerialize backward parsing

Michael Paquier <michael@paquier.xyz>

View thread View on postgr.es
From: Michael Paquier <michael@paquier.xyz>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Jim Jones <jim.jones@uni-muenster.de>, Andrew Dunstan <andrew@dunslane.net>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>, Pavel Stehule <pavel.stehule@gmail.com>
Date: 2025-02-24T00:19:09Z
Lists: pgsql-hackers 
On Fri, Feb 21, 2025 at 09:53:06AM -0500, Tom Lane wrote:
> If I have to actually debug or test behavior touching this, I use a
> non-reporting buildfarm instance that I have lying around, which is
> configured to run the cross-version tests and not very much else.
> I can switch that between pulling source from the community repo
> and pulling from a local checkout that has the mods I want to test.

Another simpler method that I have used here: grab the diff report
from the buildfarm and use a small perl script to build and checj the
regex that would be needed to make the test pass.  The version number
to apply when adjusting the dumps is easy enough to guess from the
reports.
--
Michael

Commits

  1. Fix cross-version upgrades with XMLSERIALIZE(NO INDENT)

  2. Add missing deparsing of [NO] IDENT to XMLSERIALIZE()