Fix pg_regress to print the correct postmaster address on Windows.

tglsfdc · tglsfdc · commit 59392746cb3f · 2022-03-04T13:23:58.000-05:00
pg_regress reported "Unix socket" as the default location whenever HAVE_UNIX_SOCKETS is defined. However, that's not been accurate on Windows since 8f3ec75. Update this logic to match what libpq actually does now. This is just cosmetic, but still it's potentially misleading. Back-patch to v13 where 8f3ec75 came in. Discussion: https://postgr.es/m/3894060.1646415641@sss.pgh.pa.us
diff --git a/src/interfaces/libpq/fe-connect.c b/src/interfaces/libpq/fe-connect.c
@@ -1100,6 +1100,11 @@ connectOptions2(PGconn *conn)
 		{
 			if (ch->host)
 				free(ch->host);
+
+			/*
+			 * This bit selects the default host location.  If you change
+			 * this, see also pg_regress.
+			 */
 #ifdef HAVE_UNIX_SOCKETS
 			if (DEFAULT_PGSOCKET_DIR[0])
 			{
diff --git a/src/test/regress/pg_regress.c b/src/test/regress/pg_regress.c
@@ -895,10 +895,16 @@ initialize_environment(void)
 		 */
 		pghost = getenv("PGHOST");
 		pgport = getenv("PGPORT");
-#ifndef HAVE_UNIX_SOCKETS
 		if (!pghost)
-			pghost = "localhost";
+		{
+			/* Keep this bit in sync with libpq's default host location: */
+#ifdef HAVE_UNIX_SOCKETS
+			if (DEFAULT_PGSOCKET_DIR[0])
+				 /* do nothing, we'll print "Unix socket" below */ ;
+			else
 #endif
+				pghost = "localhost";	/* DefaultHost in fe-connect.c */
+		}
 
 		if (pghost && pgport)
 			printf(_("(using postmaster on %s, port %s)\n"), pghost, pgport);

Original file line number	Diff line number	Diff line change
`@@ -1100,6 +1100,11 @@ connectOptions2(PGconn *conn)`
`1100`	`1100`	`{`
`1101`	`1101`	`if (ch->host)`
`1102`	`1102`	`free(ch->host);`
	`1103`	`+`
	`1104`	`+ /*`
	`1105`	`+ * This bit selects the default host location. If you change`
	`1106`	`+ * this, see also pg_regress.`
	`1107`	`+ */`
`1103`	`1108`	`#ifdef HAVE_UNIX_SOCKETS`
`1104`	`1109`	`if (DEFAULT_PGSOCKET_DIR[0])`
`1105`	`1110`	`{`