Add PGTZ environment variable to initialization code.

Rename PG_DATESTYLE to PGDATESTYLE environment variable.
Move environment variable code to a different place so it now works!
Note that regression tests can now run with "setenv PGTZ PST8PDT"
 at the frontend rather than requiring the backend to have TZ set.

diff --git a/src/interfaces/libpq/fe-connect.c b/src/interfaces/libpq/fe-connect.c
index e18702282818cc5ba8485444a892bbc08fe44821..51e987a915712f8f3492ce1aa3b506fb199226aa 100644 (file)
--- a/src/interfaces/libpq/fe-connect.c
+++ b/src/interfaces/libpq/fe-connect.c
@@ -7,7 +7,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/interfaces/libpq/fe-connect.c,v 1.44 1997/11/10 05:10:45 momjian Exp $
+ *       $Header: /cvsroot/pgsql/src/interfaces/libpq/fe-connect.c,v 1.45 1997/11/10 15:41:58 thomas Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -47,6 +47,7 @@ static void closePGconn(PGconn *conn);
 static int     conninfo_parse(const char *conninfo, char *errorMessage);
 static char *conninfo_getval(char *keyword);
 static void conninfo_free(void);
+void PQsetenv(PGconn *conn);
 
 #define NOTIFYLIST_INITIAL_SIZE 10
 #define NOTIFYLIST_GROWBY 10
@@ -109,18 +110,15 @@ struct EnvironmentOptions
 }                      EnvironmentOptions[] =
 
 {
-       {
-               "PG_DATESTYLE", "datestyle"
-       },
-       {
-               NULL
-       }
+       {       "PGDATESTYLE", "datestyle" },
+       {       "PGTZ", "timezone" },
+       {       NULL }
 };
 
 /* ----------------
  *             PQconnectdb
  *
- * establishes a connectin to a postgres backend through the postmaster
+ * establishes a connection to a postgres backend through the postmaster
  * using connection information in a string.
  *
  * The conninfo string is a list of
@@ -136,7 +134,7 @@ struct EnvironmentOptions
  * then some fields may be null'ed out instead of having valid values
  * ----------------
  */
-PGconn    *
+PGconn *
 PQconnectdb(const char *conninfo)
 {
        PGconn     *conn;
@@ -255,6 +253,8 @@ PQconnectdb(const char *conninfo)
                PQclear(res);
        }
 
+       PQsetenv(conn);
+
        return conn;
 }
 
@@ -309,6 +309,11 @@ PQconndefaults(void)
  *
  *       None of the above need be defined.  There are defaults for all of them.
  *
+ * To support "delimited identifiers" for database names, only convert
+ * the database name to lower case if it is not surrounded by double quotes.
+ * Otherwise, strip the double quotes but leave the reset of the string intact.
+ * - thomas 1997-11-08
+ *
  * ----------------
  */
 PGconn    *
@@ -419,8 +424,8 @@ PQsetdb(const char *pghost, const char *pgport, const char *pgoptions, const cha
                                conn->dbName = strdup(conn->pguser);
 
                        /*
-                        * if the table name is surrounded by double-quotes, then
-                        * don't convert case
+                        * if the database name is surrounded by double-quotes,
+                        *  then don't convert case
                         */
                        if (*conn->dbName == '"')
                        {
@@ -457,6 +462,7 @@ PQsetdb(const char *pghost, const char *pgport, const char *pgoptions, const cha
                                }
                                PQclear(res);
                        }
+                       PQsetenv(conn);
                }
        }
        return conn;
@@ -625,24 +631,6 @@ connectDB(PGconn *conn)
 
        conn->port = port;
 
-       {
-               struct EnvironmentOptions *eo;
-               char            setQuery[80];           /* mjl: size okay? XXX */
-
-               for (eo = EnvironmentOptions; eo->envName; eo++)
-               {
-                       const char *val;
-
-                       if ((val = getenv(eo->envName)))
-                       {
-                               PGresult   *res;
-
-                               sprintf(setQuery, "SET %s TO '%.60s'", eo->pgName, val);
-                               res = PQexec(conn, setQuery);
-                               PQclear(res);   /* Don't care? */
-                       }
-               }
-       }
        return CONNECTION_OK;
 
 connect_errReturn:
@@ -658,6 +646,30 @@ connect_errReturn:
 
 }
 
+void
+PQsetenv(PGconn *conn)
+{
+       struct EnvironmentOptions *eo;
+       char setQuery[80];              /* mjl: size okay? XXX */
+
+       for (eo = EnvironmentOptions; eo->envName; eo++)
+       {
+               const char *val;
+
+               if ((val = getenv(eo->envName)))
+               {
+                       PGresult   *res;
+
+                       sprintf(setQuery, "SET %s TO '%.60s'", eo->pgName, val);
+#ifdef CONNECTDEBUG
+printf("Use environment variable %s to send %s\n", eo->envName, setQuery);
+#endif
+                       res = PQexec(conn, setQuery);
+                       PQclear(res);   /* Don't care? */
+               }
+       }
+} /* PQsetenv() */
+
 /*
  * freePGconn
  *      - free the PGconn data structure