On Mon, Apr 07, 2025 at 04:45:52PM -0500, Nathan Bossart wrote:
> The other pg_upgrade tests chdir to tmp_check prior to running pg_upgrade
> to avoid leaving behind delete_old_cluster.{sh,bat}. 006_transfer_modes.pl
> should, too. However, this test is a little different because it loops
> over all of the available transfer modes and runs pg_upgrade for each one
> supported by the platform. From my testing and code analysis, it seems
> sufficient to change the directory once at the beginning of the test, but
> we could alternatively save the current directory and change back to it in
> each iteration to be safe.
>
> Thoughts?Hmm. Doing one chdir at the beginning of the test should be OK, because we don't do a move to a different directory within the test for each transfer mode tested. So your patch looks like the simplest thing to do to avoid the generation of these files. Note that delete_old_cluster.sh would be left around even if not using a VPATH build with ./configure (your commit message does not mention that). Even if .gitignore discards it, the file is here. -- Michael
signature.asc
Description: PGP signature
