Re: Why do we have perl and sed versions of Gen_dummy_probes?
Tom Lane <tgl@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Andrew Dunstan <andrew@dunslane.net>
Cc: Dagfinn Ilmari Mannsåker <ilmari@ilmari.org>, Peter Eisentraut <peter.eisentraut@enterprisedb.com>, Andres Freund <andres@anarazel.de>, pgsql-hackers@postgresql.org, Michael Paquier <michael@paquier.xyz>, Craig Ringer <craig.ringer@enterprisedb.com>
Date: 2021-05-10T16:07:48Z
Lists: pgsql-hackers
Andrew Dunstan <andrew@dunslane.net> writes: > On 5/10/21 7:16 AM, Dagfinn Ilmari Mannsåker wrote: >> Peter Eisentraut <peter.eisentraut@enterprisedb.com> writes: >>> This recipe doesn't produce a Gen_dummy_probes.pl that matches exactly >>> the one that is there now. If this is going to be the preferred method, >>> then we should generate it once so that it matches going forward. >> Which version of perltidy do you have installed? For me it generates >> identical versions using any of 20170521 (per src/tools/pgindent/README), >> 20201207 (what I happened to have installed before), and 20210402 (the >> latest). > Yep: For me, using App-s2p-1.003 and perltidy v20170521, it works as long as I start with the previous version of Gen_dummy_probes.pl in place. I first tried to test this by "rm Gen_dummy_probes.pl; make Gen_dummy_probes.pl", and what I got was a script without all the initial commentary nor the first line of actual Perl code. I don't think this is good practice; it implies that any accidental corruption of the commentary would be carried forward. I think we should be extracting the commentary from Gen_dummy_probes.sed. regards, tom lane
Commits
-
Tweak generation of Gen_dummy_probes.pl
- 0bf62931cae0 14.0 landed
-
Add a README and Makefile recipe for Gen_dummy_probes.pl
- 8292c0675a79 14.0 landed
-
Remove dependency on psed for MSVC builds.
- 5d0320105699 9.6.0 cited