]> granicus.if.org Git - postgresql/commitdiff
Rename find_my_binary/find_other_binary to
authorBruce Momjian <bruce@momjian.us>
Wed, 12 May 2004 13:38:49 +0000 (13:38 +0000)
committerBruce Momjian <bruce@momjian.us>
Wed, 12 May 2004 13:38:49 +0000 (13:38 +0000)
find_my_exec/find_other_exec().  Remove passing of progname to these
functions as they can find that out from argv[0], which they already
have.

Make get_progname return const char *, and update all progname variables
to be const char *.

20 files changed:
src/backend/postmaster/postmaster.c
src/backend/tcop/postgres.c
src/bin/initdb/initdb.c
src/bin/pg_controldata/pg_controldata.c
src/bin/pg_dump/pg_dumpall.c
src/bin/pg_resetxlog/pg_resetxlog.c
src/bin/psql/settings.h
src/bin/psql/tab-complete.c
src/bin/scripts/clusterdb.c
src/bin/scripts/createdb.c
src/bin/scripts/createlang.c
src/bin/scripts/createuser.c
src/bin/scripts/dropdb.c
src/bin/scripts/droplang.c
src/bin/scripts/dropuser.c
src/bin/scripts/vacuumdb.c
src/include/port.h
src/interfaces/ecpg/preproc/ecpg.c
src/port/exec.c
src/port/path.c

index d2c71041744fa9acae4d9f4de4682cfb64b06844..bd183739204c862dd9f1b07d5897d332112a4eb5 100644 (file)
@@ -37,7 +37,7 @@
  *
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/backend/postmaster/postmaster.c,v 1.384 2004/05/12 03:48:42 momjian Exp $
+ *       $PostgreSQL: pgsql/src/backend/postmaster/postmaster.c,v 1.385 2004/05/12 13:38:39 momjian Exp $
  *
  * NOTES
  *
@@ -172,7 +172,7 @@ int                 MaxBackends;
 int                    ReservedBackends;
 
 
-static char *progname = NULL;
+static const char *progname = NULL;
 
 /* The socket(s) we're listening to. */
 #define MAXLISTEN      10
@@ -412,7 +412,7 @@ PostmasterMain(int argc, char *argv[])
 
        *original_extraoptions = '\0';
 
-       progname = argv[0];
+       progname = get_progname(argv[0]);
 
        IsPostmasterEnvironment = true;
 
@@ -692,7 +692,7 @@ PostmasterMain(int argc, char *argv[])
        /*
         * On some systems our dynloader code needs the executable's pathname.
         */
-       if (find_my_binary(pg_pathname, argv[0], "postgres") < 0)
+       if (find_my_exec(pg_pathname, argv[0]) < 0)
                ereport(FATAL,
                                (errmsg("%s: could not locate postgres executable",
                                                progname)));
@@ -3222,7 +3222,7 @@ CreateOptsFile(int argc, char *argv[])
        FILE       *fp;
        int                     i;
 
-       if (find_my_binary(fullprogname, argv[0], "postmaster") < 0)
+       if (find_my_exec(fullprogname, argv[0]) < 0)
        {
                elog(LOG, "could not locate postmaster");
                return false;
index caeef3cd5708dec8ccd66661bde25e46edf3285e..4c006f00b8b00e4b7979ca037b78ee46131d3818 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/backend/tcop/postgres.c,v 1.403 2004/05/11 21:57:14 momjian Exp $
+ *       $PostgreSQL: pgsql/src/backend/tcop/postgres.c,v 1.404 2004/05/12 13:38:40 momjian Exp $
  *
  * NOTES
  *       this is the "main" module of the postgres backend and
@@ -2083,7 +2083,7 @@ assign_max_stack_depth(int newval, bool doit, GucSource source)
 
 
 static void
-usage(char *progname)
+usage(const char *progname)
 {
        printf(gettext("%s is the PostgreSQL stand-alone backend.  It is not\nintended to be used by normal users.\n\n"), progname);
 
@@ -2649,7 +2649,7 @@ PostgresMain(int argc, char *argv[], const char *username)
                 * On some systems our dynloader code needs the executable's
                 * pathname.  (If under postmaster, this was done already.)
                 */
-               if (find_my_binary(pg_pathname, argv[0], "postgres") < 0)
+               if (find_my_exec(pg_pathname, argv[0]) < 0)
                        ereport(FATAL,
                                        (errmsg("%s: could not locate postgres executable",
                                                        argv[0])));
index fa8bf28da865750fecba578d8767cbdbf66bb14a..c7ebb60f03dc5340cc1b41241ed66f03d9867c17 100644 (file)
@@ -39,7 +39,7 @@
  * Portions Copyright (c) 1994, Regents of the University of California
  * Portions taken from FreeBSD.
  *
- * $PostgreSQL: pgsql/src/bin/initdb/initdb.c,v 1.27 2004/05/11 21:57:14 momjian Exp $
+ * $PostgreSQL: pgsql/src/bin/initdb/initdb.c,v 1.28 2004/05/12 13:38:42 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -93,7 +93,7 @@ bool          show_setting = false;
 
 
 /* internal vars */
-char      *progname;
+const char *progname;
 char      *postgres;
 char      *encodingid = "0";
 char      *bki_file;
@@ -1932,7 +1932,7 @@ main(int argc, char *argv[])
        sprintf(pgdenv, "PGDATA=%s", pg_data);
        putenv(pgdenv);
 
-       if ((ret = find_other_binary(backendbin, argv[0], progname, "postgres",
+       if ((ret = find_other_exec(backendbin, argv[0], "postgres",
                                                   PG_VERSIONSTR)) < 0)
        {
                if (ret == -1)
index 4342d1e10afa3143f2cbdefa81fa86d5db938a1a..86b4d557d9c6e9577b5aea66a0e00b1e9af3ef12 100644 (file)
@@ -6,7 +6,7 @@
  * copyright (c) Oliver Elphick <olly@lfix.co.uk>, 2001;
  * licence: BSD
  *
- * $PostgreSQL: pgsql/src/bin/pg_controldata/pg_controldata.c,v 1.14 2004/03/22 15:34:22 tgl Exp $
+ * $PostgreSQL: pgsql/src/bin/pg_controldata/pg_controldata.c,v 1.15 2004/05/12 13:38:43 momjian Exp $
  */
 #include "postgres.h"
 
@@ -75,7 +75,7 @@ main(int argc, char *argv[])
        char            ckpttime_str[128];
        char            sysident_str[32];
        char       *strftime_fmt = "%c";
-       char       *progname;
+       const char *progname;
 
        setlocale(LC_ALL, "");
 #ifdef ENABLE_NLS
index 0817cabfb95df3531e64291866c78a701053f078..cac837865e327b8298097c4130fbb231a6d75896 100644 (file)
@@ -6,7 +6,7 @@
  * Portions Copyright (c) 1994, Regents of the University of California
  *
  *
- * $PostgreSQL: pgsql/src/bin/pg_dump/pg_dumpall.c,v 1.31 2004/05/11 21:57:14 momjian Exp $
+ * $PostgreSQL: pgsql/src/bin/pg_dump/pg_dumpall.c,v 1.32 2004/05/12 13:38:44 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -39,7 +39,7 @@ int                   optreset;
 #define PG_VERSIONSTR "pg_dump (PostgreSQL) " PG_VERSION "\n"
 
 
-static char *progname;
+static const char *progname;
 
 static void help(void);
 
@@ -123,7 +123,7 @@ main(int argc, char *argv[])
                }
        }
 
-       if ((ret = find_other_binary(pg_dump_bin, argv[0], progname, "pg_dump",
+       if ((ret = find_other_exec(pg_dump_bin, argv[0], "pg_dump",
                                                   PG_VERSIONSTR)) < 0)
        {
                if (ret == -1)
index 80281e8ef7a413dfa2b16426184012ddca9ce089..8c57f7101159724e50a3d115087c7e5fe1194ec9 100644 (file)
@@ -23,7 +23,7 @@
  * Portions Copyright (c) 1996-2003, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/bin/pg_resetxlog/pg_resetxlog.c,v 1.17 2004/03/22 16:46:28 tgl Exp $
+ * $PostgreSQL: pgsql/src/bin/pg_resetxlog/pg_resetxlog.c,v 1.18 2004/05/12 13:38:44 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -75,7 +75,7 @@ static ControlFileData ControlFile;           /* pg_control values */
 static uint32 newXlogId,
                        newXlogSeg;                     /* ID/Segment of new XLOG segment */
 static bool guessed = false;   /* T if we had to guess at any values */
-static char *progname;
+static const char *progname;
 
 static bool ReadControlFile(void);
 static void GuessControlValues(void);
index e28383b16be7a57e14f52c8f59bd5efd258ad517..d8766b045a88336d8ed7bee452f3c463128e50f3 100644 (file)
@@ -3,7 +3,7 @@
  *
  * Copyright (c) 2000-2003, PostgreSQL Global Development Group
  *
- * $PostgreSQL: pgsql/src/bin/psql/settings.h,v 1.17 2004/03/21 22:29:11 tgl Exp $
+ * $PostgreSQL: pgsql/src/bin/psql/settings.h,v 1.18 2004/05/12 13:38:45 momjian Exp $
  */
 #ifndef SETTINGS_H
 #define SETTINGS_H
@@ -42,7 +42,7 @@ typedef struct _psqlSettings
                                                                 * loop */
        bool            cur_cmd_interactive;
 
-       char       *progname;           /* in case you renamed psql */
+       const char *progname;           /* in case you renamed psql */
        char       *inputfile;          /* for error reporting */
        unsigned        lineno;                 /* also for error reporting */
 
index f2b2d0902b65ca50d0782bb98a9ab0395ad7303f..2eb2f19f09f84c0ebba38d5913dcba49743e043d 100644 (file)
@@ -3,7 +3,7 @@
  *
  * Copyright (c) 2000-2003, PostgreSQL Global Development Group
  *
- * $PostgreSQL: pgsql/src/bin/psql/tab-complete.c,v 1.105 2004/05/07 00:24:58 tgl Exp $
+ * $PostgreSQL: pgsql/src/bin/psql/tab-complete.c,v 1.106 2004/05/12 13:38:46 momjian Exp $
  */
 
 /*----------------------------------------------------------------------
@@ -428,7 +428,7 @@ static char *dequote_file_name(char *text, char quote_char);
 void
 initialize_readline(void)
 {
-       rl_readline_name = pset.progname;
+       rl_readline_name = (char *)pset.progname;
        rl_attempted_completion_function = (void *) psql_completion;
 
        rl_basic_word_break_characters = "\t\n@$><=;|&{( ";
index 6518e010b7d6576624c725e9e4aac124d12b327b..0fcab8ec6daca6ebe34d3b581610f9b22c5d103d 100644 (file)
@@ -4,7 +4,7 @@
  *
  * Portions Copyright (c) 2002-2003, PostgreSQL Global Development Group
  *
- * $PostgreSQL: pgsql/src/bin/scripts/clusterdb.c,v 1.5 2004/01/01 19:27:15 tgl Exp $
+ * $PostgreSQL: pgsql/src/bin/scripts/clusterdb.c,v 1.6 2004/05/12 13:38:46 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -43,7 +43,7 @@ main(int argc, char *argv[])
                {NULL, 0, NULL, 0}
        };
 
-       char       *progname;
+       const char *progname;
        int                     optindex;
        int                     c;
 
index 005c942cca3a1b6f89458338cb0af4eee735bbf6..8ee5fdca6516105d7856b2138905807cc7c39970 100644 (file)
@@ -5,7 +5,7 @@
  * Portions Copyright (c) 1996-2003, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/bin/scripts/createdb.c,v 1.7 2004/01/01 19:27:15 tgl Exp $
+ * $PostgreSQL: pgsql/src/bin/scripts/createdb.c,v 1.8 2004/05/12 13:38:47 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -37,7 +37,7 @@ main(int argc, char *argv[])
                {NULL, 0, NULL, 0}
        };
 
-       char       *progname;
+       const char *progname;
        int                     optindex;
        int                     c;
 
index 64e257bc2f05d66aeff9b71c42e7c4568e16c07a..f75e9e07cdd5be67930ccc3ef0ac2d4ca550e744 100644 (file)
@@ -5,7 +5,7 @@
  * Portions Copyright (c) 1996-2003, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/bin/scripts/createlang.c,v 1.8 2004/03/19 18:58:07 tgl Exp $
+ * $PostgreSQL: pgsql/src/bin/scripts/createlang.c,v 1.9 2004/05/12 13:38:47 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -33,7 +33,7 @@ main(int argc, char *argv[])
                {NULL, 0, NULL, 0}
        };
 
-       char       *progname;
+       const char *progname;
        int                     optindex;
        int                     c;
 
index 12c1aabb5d75c7e62f4cc59905d039522770b0cc..2a262ecb39bac4792467a7da233778f022a96874 100644 (file)
@@ -5,7 +5,7 @@
  * Portions Copyright (c) 1996-2003, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/bin/scripts/createuser.c,v 1.9 2004/01/09 00:15:11 tgl Exp $
+ * $PostgreSQL: pgsql/src/bin/scripts/createuser.c,v 1.10 2004/05/12 13:38:48 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -39,7 +39,7 @@ main(int argc, char *argv[])
                {NULL, 0, NULL, 0}
        };
 
-       char       *progname;
+       const char *progname;
        int                     optindex;
        int                     c;
 
index 91ad090f42aae0374194901af38b3574d22d08f2..5b8540e4c38f745ca242d3299ff637af8cf7621f 100644 (file)
@@ -5,7 +5,7 @@
  * Portions Copyright (c) 1996-2003, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/bin/scripts/dropdb.c,v 1.8 2004/01/01 19:27:15 tgl Exp $
+ * $PostgreSQL: pgsql/src/bin/scripts/dropdb.c,v 1.9 2004/05/12 13:38:48 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -32,7 +32,7 @@ main(int argc, char *argv[])
                {NULL, 0, NULL, 0}
        };
 
-       char       *progname;
+       const char *progname;
        int                     optindex;
        int                     c;
 
index 43f57115f53d47c781f759bcfc35f10f99679510..63fe0f7c32c2568442dd5703afd6e50c6dd7a53b 100644 (file)
@@ -5,7 +5,7 @@
  * Portions Copyright (c) 1996-2003, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/bin/scripts/droplang.c,v 1.7 2004/03/19 18:58:07 tgl Exp $
+ * $PostgreSQL: pgsql/src/bin/scripts/droplang.c,v 1.8 2004/05/12 13:38:48 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -34,7 +34,7 @@ main(int argc, char *argv[])
                {NULL, 0, NULL, 0}
        };
 
-       char       *progname;
+       const char *progname;
        int                     optindex;
        int                     c;
 
index b0dc5a7d7c3472bc014633addfba9587eddec5a5..352f913a06c93b473603deda6d0bcf3e1e693fa2 100644 (file)
@@ -5,7 +5,7 @@
  * Portions Copyright (c) 1996-2003, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/bin/scripts/dropuser.c,v 1.7 2004/01/01 19:27:15 tgl Exp $
+ * $PostgreSQL: pgsql/src/bin/scripts/dropuser.c,v 1.8 2004/05/12 13:38:48 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -32,7 +32,7 @@ main(int argc, char *argv[])
                {NULL, 0, NULL, 0}
        };
 
-       char       *progname;
+       const char *progname;
        int                     optindex;
        int                     c;
 
index cc9d8be1396ed34140cf6b70a120cf3ba89b1ae7..ae64f34ab92c193b71f130352929062bcaae5df5 100644 (file)
@@ -5,7 +5,7 @@
  * Portions Copyright (c) 1996-2003, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/bin/scripts/vacuumdb.c,v 1.5 2004/01/01 19:27:15 tgl Exp $
+ * $PostgreSQL: pgsql/src/bin/scripts/vacuumdb.c,v 1.6 2004/05/12 13:38:48 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -48,7 +48,7 @@ main(int argc, char *argv[])
                {NULL, 0, NULL, 0}
        };
 
-       char       *progname;
+       const char *progname;
        int                     optindex;
        int                     c;
 
index 177c82f8c4e7b8f64a6056db64991f3ad4f5ab2b..a037ac1f5f4099186a9711409c3a387be88eead0 100644 (file)
@@ -6,7 +6,7 @@
  * Portions Copyright (c) 1996-2003, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/include/port.h,v 1.29 2004/05/11 21:57:15 momjian Exp $
+ * $PostgreSQL: pgsql/src/include/port.h,v 1.30 2004/05/12 13:38:48 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -25,12 +25,11 @@ extern bool is_absolute_path(const char *filename);
 extern char *first_path_separator(const char *filename);
 extern char *last_path_separator(const char *filename);
 extern void canonicalize_path(char *path);
-extern char *get_progname(char *argv0);
+extern const char *get_progname(const char *argv0);
 
 /* Portable way to find binaries */
-extern int find_my_binary(char *full_path, const char *argv0,
-                                        const char *binary_name);
-extern int find_other_binary(char *retpath, const char *argv0, const char *progname,
+extern int find_my_exec(char *full_path, const char *argv0);
+extern int find_other_exec(char *retpath, const char *argv0,
                                           char const *target, const char *versionstr);
 
 #if defined(__CYGWIN__) || defined(WIN32)
index 5629f2b944b36aecf6a83e53905ca124d8e55f39..40edf0ecc434534ced783ff612fcbb76c0a1e95c 100644 (file)
@@ -1,4 +1,4 @@
-/* $PostgreSQL: pgsql/src/interfaces/ecpg/preproc/ecpg.c,v 1.85 2004/01/28 20:43:03 tgl Exp $ */
+/* $PostgreSQL: pgsql/src/interfaces/ecpg/preproc/ecpg.c,v 1.86 2004/05/12 13:38:48 momjian Exp $ */
 
 /* New main for ecpg, the PostgreSQL embedded SQL precompiler. */
 /* (C) Michael Meskes <meskes@postgresql.org> Feb 5th, 1998 */
@@ -119,7 +119,7 @@ main(int argc, char *const argv[])
                                verbose = false,
                                out_option = 0;
        struct _include_path *ip;
-       char       *progname;
+       const char *progname;
 
        progname = get_progname(argv[0]);
 
index 27e66cc0c8f74d99809839b1ed4baf11a48b6c5e..2ea6531c8eb9242eb62012988830713c7416dc29 100644 (file)
@@ -7,7 +7,7 @@
  *
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/port/exec.c,v 1.1 2004/05/11 21:57:15 momjian Exp $
+ *       $PostgreSQL: pgsql/src/port/exec.c,v 1.2 2004/05/12 13:38:49 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -165,7 +165,7 @@ validate_exec(char *path)
 }
 
 /*
- * find_my_binary -- find an absolute path to a valid executable
+ * find_my_exec -- find an absolute path to a valid executable
  *
  * The reason we have to work so hard to find an absolute path is that
  * on some platforms we can't do dynamic loading unless we know the
@@ -177,13 +177,14 @@ validate_exec(char *path)
  * non-threaded binaries, not in library routines.
  */
 int
-find_my_binary(char *full_path, const char *argv0, const char *binary_name)
+find_my_exec(char *full_path, const char *argv0)
 {
        char            buf[MAXPGPATH + 2];
        char       *p;
        char       *path,
                           *startp,
                           *endp;
+       const char *binary_name = get_progname(argv0);
 
        /*
         * First try: use the binary that's located in the
@@ -273,14 +274,14 @@ find_my_binary(char *full_path, const char *argv0, const char *binary_name)
  * Find our binary directory, then make sure the "target" executable
  * is the proper version.
  */
-int find_other_binary(char *retpath, const char *argv0, const char *progname,
+int find_other_exec(char *retpath, const char *argv0,
                            char const *target, const char *versionstr)
 {
        char            cmd[MAXPGPATH];
        char            line[100];
        FILE       *pgver;
 
-       if (find_my_binary(retpath, argv0, progname) < 0)
+       if (find_my_exec(retpath, argv0) < 0)
                return -1;
 
        /* Trim off program name and keep just directory */     
index 965d63ea330f5e32be8a510b1947dfde84e393bb..5d102a90c2489fbca24e84c5b5abac7b504a85d4 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/port/path.c,v 1.6 2004/05/11 21:57:15 momjian Exp $
+ *       $PostgreSQL: pgsql/src/port/path.c,v 1.7 2004/05/12 13:38:49 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -113,8 +113,8 @@ canonicalize_path(char *path)
 /*
  * Extracts the actual name of the program as called.
  */
-char *
-get_progname(char *argv0)
+const char *
+get_progname(const char *argv0)
 {
        if (!last_path_separator(argv0))
                return argv0;