Get rid of some more stuff that duplicates c.h or port.h.

Author: tglsfdc

contrib/pg_upgrade/controldata.c

@@ -7,7 +7,6 @@
 #include "pg_upgrade.h"
 
 #include <ctype.h>
-#include <stdlib.h>
 
 
 /*

contrib/pg_upgrade/file.c

@@ -6,13 +6,8 @@
 
 #include "pg_upgrade.h"
 
-#include <sys/types.h>
 #include <fcntl.h>
 
-#ifdef WIN32
-#include <windows.h>
-#endif
-
 
 static int	copy_file(const char *fromfile, const char *tofile, bool force);
 

contrib/pg_upgrade/pg_upgrade.h

@@ -5,13 +5,10 @@
 #include "postgres.h"
 
 #include <unistd.h>
-#include <string.h>
-#include <stdio.h>
 #include <assert.h>
 #include <dirent.h>
 #include <sys/stat.h>
 #include <sys/time.h>
-#include <sys/types.h>
 
 #include "libpq-fe.h"
 
@@ -38,26 +35,16 @@
 #define pg_copy_file		copy_file
 #define pg_mv_file			rename
 #define pg_link_file		link
-#define DEVNULL				"/dev/null"
-#define DEVTTY				"/dev/tty"
 #define RMDIR_CMD			"rm -rf"
 #define EXEC_EXT			"sh"
 #else
 #define pg_copy_file		CopyFile
 #define pg_mv_file			pgrename
 #define pg_link_file		win32_pghardlink
-#define EXE_EXT				".exe"
 #define sleep(x)			Sleep(x * 1000)
-#define DEVNULL				"nul"
-/* "con" does not work from the Msys 1.0.10 console (part of MinGW). */
-#define DEVTTY  "con"
-/* from pgport */
-extern int  pgrename(const char *from, const char *to);
-extern int  pgunlink(const char *path);
-#define rename(from, to)	pgrename(from, to)
-#define unlink(path)		pgunlink(path)
 #define RMDIR_CMD			"RMDIR /s/q"
 #define EXEC_EXT			"bat"
+#define EXE_EXT				".exe"
 #endif
 
 #define CLUSTERNAME(cluster)	((cluster) == CLUSTER_OLD ? "old" : "new")