Re: [PATCH] Add pretty-printed XML output option
Jim Jones <jim.jones@uni-muenster.de>
From: Jim Jones <jim.jones@uni-muenster.de>
To: Alvaro Herrera <alvherre@alvh.no-ip.org>,
Peter Smith <smithpb2250@gmail.com>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, pgsql-hackers@lists.postgresql.org,
Peter Eisentraut <peter.eisentraut@enterprisedb.com>
Date: 2023-02-15T10:33:50Z
Lists: pgsql-hackers
On 15.02.23 11:11, Alvaro Herrera wrote: > Note that there's another file, xml_2.out, which does not contain the > additional part of the DETAIL message. I suspect in Peter's case it's > xml_2.out that was originally being used as a comparison basis (not > xml.out), but that one is not getting patched, and ultimately the diff > is reported for him against xml.out because none of them matches. > > An easy way forward might be to manually apply the patch from xml.out to > xml_2.out, and edit it by hand to remove the additional lines. > > See commit 085423e3e326 for a bit of background. Hi Álvaro, As my test cases were not specifically about how the error message looks like, I thought that suppressing part of the error messages by setting VERBOSITY terse would suffice.[1] Is this approach not recommended? Thanks! Best, Jim 1 - v9 patch: https://www.postgresql.org/message-id/CAHut%2BPtoH8zkBHxv44XyO%2Bo4kW_nZdGhNdVaJ_cpEjrckKDqtw%40mail.gmail.com
Commits
-
Fix memory leak in XMLSERIALIZE(... INDENT).
- f24605e2dc16 18.0 landed
- ee58de10084b 16.10 landed
- 20bae0690322 17.6 landed
-
doc: Move documentation of md5_password_warnings to a better place
- f3622b64762b 18.0 cited
-
Support [NO] INDENT option in XMLSERIALIZE().
- 483bdb2afec9 16.0 landed
-
Add an expected-file to match behavior of latest libxml2.
- 085423e3e326 9.6.0 cited