From 1a271f0c71294c47444a15d93bc064d3a4f23674 Mon Sep 17 00:00:00 2001
From: Bruce Momjian <bruce@momjian.us>
Date: Sun, 30 Jul 2006 01:45:21 +0000
Subject: [PATCH] Fix WIN32 wait() return value macros to be accurate,
 particularly because they are used for testing the return value from
 system(). (WIN32 doesn't overlay the return code with other failure
 conditions like Unix does, so they are just simple macros.)

Fix regression checks to properly handle diff failures on Win32 using
the new macros.
---
 src/include/port/win32.h      | 17 +++++++++-----
 src/test/regress/pg_regress.c | 43 +++++++++++++++++------------------
 2 files changed, 32 insertions(+), 28 deletions(-)

diff --git a/src/include/port/win32.h b/src/include/port/win32.h
index 7a9180ea9c..fa57c25b3c 100644
--- a/src/include/port/win32.h
+++ b/src/include/port/win32.h
@@ -1,4 +1,4 @@
-/* $PostgreSQL: pgsql/src/include/port/win32.h,v 1.53 2006/07/16 20:17:04 tgl Exp $ */
+/* $PostgreSQL: pgsql/src/include/port/win32.h,v 1.54 2006/07/30 01:45:21 momjian Exp $ */
 
 /* undefine and redefine after #include */
 #undef mkdir
@@ -109,12 +109,17 @@ int			semop(int semId, struct sembuf * sops, int flag);
 
 
 /*
- * Signal stuff
+ *	Signal stuff
+ *	WIN32 doesn't have wait(), so the return value for children
+ *	is simply the return value specified by the child, without
+ *	any additional information on whether the child terminated
+ *	on its own or via a signal.  These macros are also used
+ *	to interpret the return value of system().
  */
-#define WEXITSTATUS(w)	(((w) >> 8) & 0xff)
-#define WIFEXITED(w)	(((w) & 0xff) == 0)
-#define WIFSIGNALED(w)	(((w) & 0x7f) > 0 && (((w) & 0x7f) < 0x7f))
-#define WTERMSIG(w)		((w) & 0x7f)
+#define WEXITSTATUS(w)	(w)
+#define WIFEXITED(w)	(true)
+#define WIFSIGNALED(w)	(false)
+#define WTERMSIG(w)		(0)
 
 #define sigmask(sig) ( 1 << ((sig)-1) )
 
diff --git a/src/test/regress/pg_regress.c b/src/test/regress/pg_regress.c
index abb6131a38..fb8c7d8dee 100644
--- a/src/test/regress/pg_regress.c
+++ b/src/test/regress/pg_regress.c
@@ -11,7 +11,7 @@
  * Portions Copyright (c) 1996-2006, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/test/regress/pg_regress.c,v 1.16 2006/07/27 15:37:19 tgl Exp $
+ * $PostgreSQL: pgsql/src/test/regress/pg_regress.c,v 1.17 2006/07/30 01:45:21 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -799,29 +799,32 @@ make_directory(const char *dir)
 }
 
 /*
- * Run a "diff" command and check that it didn't crash
+ * Run a "diff" command and also check that it didn't crash
  */
-static void
-run_diff(const char *cmd)
+static int
+run_diff(const char *cmd, const char *filename)
 {
 	int r;
 
 	r = system(cmd);
-	/*
-	 * XXX FIXME: it appears that include/port/win32.h's definitions of
-	 * WIFEXITED and related macros may be wrong.  They certainly don't
-	 * work for inspecting the results of system().  For the moment,
-	 * hard-wire the check on Windows.
-	 */
-#ifndef WIN32
 	if (!WIFEXITED(r) || WEXITSTATUS(r) > 1)
-#else
-	if (r != 0 && r != 1)
-#endif
 	{
 		fprintf(stderr, _("diff command failed with status %d: %s\n"), r, cmd);
 		exit_nicely(2);
 	}
+#ifdef WIN32
+	/*
+	 *	On WIN32, if the 'diff' command cannot be found, system() returns
+	 *	1, but produces nothing to stdout, so we check for that here.
+	 */
+	if (WEXITSTATUS(r) == 1 && file_size(filename) <= 0)
+	{
+		fprintf(stderr, _("diff command not found: %s\n"), cmd);
+		exit_nicely(2);
+	}
+#endif
+	
+	return WEXITSTATUS(r);
 }
 
 /*
@@ -844,7 +847,7 @@ results_differ(const char *testname)
 	int best_line_count;
 	int i;
 	int l;
-
+	
 	/* Check in resultmap if we should be looking at a different file */
 	expectname = testname;
 	for (rm = resultmap; rm != NULL; rm = rm->next)
@@ -872,12 +875,10 @@ results_differ(const char *testname)
 	snprintf(cmd, sizeof(cmd),
 			 SYSTEMQUOTE "diff %s \"%s\" \"%s\" > \"%s\"" SYSTEMQUOTE,
 			 basic_diff_opts, expectfile, resultsfile, diff);
-	run_diff(cmd);
 
 	/* Is the diff file empty? */
-	if (file_size(diff) == 0)
+	if (run_diff(cmd, diff) == 0)
 	{
-		/* No diff = no changes = good */
 		unlink(diff);
 		return false;
 	}
@@ -896,11 +897,9 @@ results_differ(const char *testname)
 		snprintf(cmd, sizeof(cmd),
 				 SYSTEMQUOTE "diff %s \"%s\" \"%s\" > \"%s\"" SYSTEMQUOTE,
 				 basic_diff_opts, expectfile, resultsfile, diff);
-		run_diff(cmd);
 
-		if (file_size(diff) == 0)
+		if (run_diff(cmd, diff) == 0)
 		{
-			/* No diff = no changes = good */
 			unlink(diff);
 			return false;
 		}
@@ -921,7 +920,7 @@ results_differ(const char *testname)
 	snprintf(cmd, sizeof(cmd),
 			 SYSTEMQUOTE "diff %s \"%s\" \"%s\" >> \"%s\"" SYSTEMQUOTE,
 			 pretty_diff_opts, best_expect_file, resultsfile, difffilename);
-	run_diff(cmd);
+	run_diff(cmd, difffilename);
 
 	/* And append a separator */
 	difffile = fopen(difffilename, "a");
-- 
2.40.0