FW: Timezone library
This mail apparantly didn't make it through because it was too large.
Resending it without the largest file, tzlib.tgz. I've put this file (+
the patches) on http://www.hagander.net/pgsql/.
//Magnus
-----Original Message-----
From: Magnus Hagander
Sent: Sunday, April 18, 2004 9:05 PM
To: pgsql-patches@postgresql.org
Subject: Timezone libraryHi!
Attached is a slightly modified version of Arthur David
Olson's timezone library for inclusion in postgresql, pending
review of course. This is required to give win32 working
timezone functions on dates before 1970. It has also been
indicated that it might be interesting to have such a library
on other platforms as well. Note that with this enabled, the
timezone data files are installed in <pgdir>/share/timezone.Attached are the following files:
* tzlib.tgz - the timezone implementation, including the
datafiles required to build the timezone data files.
* timezone.patch - the modifications made to existing files
to make it work. This includes setting the FRONTEND flat for ecpg.
* tzcode.diff - lists the modifications that I have made to
the files from the original timezone files, for reference.Bruce has offered to do the autoconf and Makefile integration
parts - I don't know autoconf enough to do that part... The
#defines necessary are "activeted" by setting USE_PGTZ. Of
course, the appropriate files also have to be linked in.With this patch applied, and the timezone data installed,
win32 now passes all time-related regression tests (3 now
fails, 2 "known" and one being locale-inflicted, still
investigating that one)//Magnus
<<tzcode_changes.diff>> <<timezone.patch>>
Attachments:
tzcode_changes.diffapplication/octet-stream; name=tzcode_changes.diffDownload
diff -cb /prog/tz/tzcode/asctime.c ./asctime.c
*** /prog/tz/tzcode/asctime.c Mon Dec 15 15:36:35 2003
--- ./asctime.c Mon Apr 12 21:30:10 2004
***************
*** 1,3 ****
--- 1,4 ----
+ #include "pgtz.h"
/*
** This file is in the public domain, so clarified as of
** 1996-06-05 by Arthur David Olson (arthur_david_olson@nih.gov).
diff -cb /prog/tz/tzcode/difftime.c ./difftime.c
*** /prog/tz/tzcode/difftime.c Mon Dec 15 15:36:35 2003
--- ./difftime.c Mon Apr 12 21:30:13 2004
***************
*** 1,3 ****
--- 1,4 ----
+ #include "pgtz.h"
/*
** This file is in the public domain, so clarified as of
** June 5, 1996 by Arthur David Olson (arthur_david_olson@nih.gov).
diff -cb /prog/tz/tzcode/localtime.c ./localtime.c
*** /prog/tz/tzcode/localtime.c Mon Dec 15 15:36:35 2003
--- ./localtime.c Mon Apr 12 21:29:47 2004
***************
*** 1,3 ****
--- 1,8 ----
+ #include "pgtz.h"
+ #undef open
+ #define timezone pg_timezone
+ #define USG_COMPAT
+ extern time_t pg_timezone;
/*
** This file is in the public domain, so clarified as of
** 1996-06-05 by Arthur David Olson (arthur_david_olson@nih.gov).
diff -cb /prog/tz/tzcode/zic.c ./zic.c
*** /prog/tz/tzcode/zic.c Mon Dec 15 15:36:35 2003
--- ./zic.c Mon Apr 12 21:29:34 2004
***************
*** 1,4 ****
--- 1,11 ----
static char elsieid[] = "@(#)zic.c 7.115";
+ #include "pgtz.h"
+ #undef unlink
+ #undef TZDIR
+ #define TZDIR "data"
+ #ifdef WIN32
+ #include <windows.h>
+ #endif
#include "private.h"
#include "locale.h"
***************
*** 7,17 ****
--- 14,26 ----
#if HAVE_SYS_STAT_H
#include "sys/stat.h"
#endif
+ #ifndef WIN32
#ifdef S_IRUSR
#define MKDIR_UMASK (S_IRUSR|S_IWUSR|S_IXUSR|S_IRGRP|S_IXGRP|S_IROTH|S_IXOTH)
#else
#define MKDIR_UMASK 0755
#endif
+ #endif
/*
** On some ancient hosts, predicates like `isspace(C)' are defined
***************
*** 2234,2236 ****
--- 2243,2258 ----
/*
** UNIX was a registered trademark of The Open Group in 2003.
*/
+
+
+ #ifdef WIN32
+ /*
+ * To run on win32
+ */
+ int link(const char *oldpath, const char *newpath) {
+ if (!CopyFileEx(oldpath, newpath, NULL, NULL, FALSE, 0)) {
+ return -1;
+ }
+ return 0;
+ }
+ #endif
timezone.patchapplication/octet-stream; name=timezone.patchDownload
diff -cr postgresql-snapshot-030325/src/include/c.h postgresql-newsignal/src/include/c.h
*** postgresql-snapshot-030325/src/include/c.h Wed Mar 10 22:12:46 2004
--- postgresql-newsignal/src/include/c.h Mon Apr 12 21:39:13 2004
***************
*** 310,320 ****
--- 310,324 ----
#endif
/* Global variable holding time zone information. */
+ #ifdef USE_PGTZ
+ #define TIMEZONE_GLOBAL pg_timezone
+ #else
#ifndef HAVE_UNDERSCORE_TIMEZONE
#define TIMEZONE_GLOBAL timezone
#else
#define TIMEZONE_GLOBAL _timezone
#define tzname _tzname /* should be in time.h? */
+ #endif
#endif
/* sig_atomic_t is required by ANSI C, but may be missing on old platforms */
diff -cr postgresql-snapshot-030325/src/include/port.h postgresql-newsignal/src/include/port.h
*** postgresql-snapshot-030325/src/include/port.h Wed Mar 24 04:54:16 2004
--- postgresql-newsignal/src/include/port.h Mon Apr 12 22:03:28 2004
***************
*** 157,159 ****
--- 157,193 ----
#define WIFSIGNALED(w) (((w) & 0x7f) > 0 && (((w) & 0x7f) < 0x7f))
#define WTERMSIG(w) ((w) & 0x7f)
#endif
+
+ /*
+ * Internal timezone library
+ */
+ #ifdef USE_PGTZ
+ #ifndef FRONTEND
+ #undef localtime
+ #undef gmtime
+ #undef asctime
+ #undef ctime
+ #undef difftime
+ #undef mktime
+ #undef tzset
+
+ #define localtime(timep) pg_localtime(timep)
+ #define gmtime(timep) pg_gmtime(timep)
+ #define asctime(timep) pg_asctime(timep)
+ #define ctime(timep) pg_ctime(timep)
+ #define difftime(t1,t2) pg_difftime(t1,t2)
+ #define mktime(tm) pg_mktime(tm)
+ #define tzset pg_tzset
+
+
+ extern struct tm *pg_localtime(const time_t *);
+ extern struct tm *gg_gmtime(const time_t *);
+ extern char *pg_asctime(const struct tm *);
+ extern char *pg_ctime(const time_t *);
+ extern double pg_difftime(const time_t, const time_t);
+ extern time_t pg_mktime(struct tm *);
+ extern void pg_tzset(void);
+ extern time_t pg_timezone;
+
+ #endif
+ #endif
diff -cr postgresql-snapshot-030325/src/interfaces/ecpg/pgtypeslib/Makefile postgresql-newsignal/src/interfaces/ecpg/pgtypeslib/Makefile
*** postgresql-snapshot-030325/src/interfaces/ecpg/pgtypeslib/Makefile Tue Feb 24 17:07:49 2004
--- postgresql-newsignal/src/interfaces/ecpg/pgtypeslib/Makefile Tue Apr 13 21:03:50 2004
***************
*** 16,22 ****
SO_MAJOR_VERSION= 1
SO_MINOR_VERSION= 2
! override CPPFLAGS := -I$(top_srcdir)/src/interfaces/ecpg/include -I$(top_srcdir)/src/include/utils -I$(libpq_srcdir) $(CPPFLAGS) $(THREAD_CPPFLAGS)
SHLIB_LINK += -lm
OBJS= numeric.o datetime.o common.o dt_common.o timestamp.o interval.o \
--- 16,22 ----
SO_MAJOR_VERSION= 1
SO_MINOR_VERSION= 2
! override CPPFLAGS := -I$(top_srcdir)/src/interfaces/ecpg/include -I$(top_srcdir)/src/include/utils -I$(libpq_srcdir) $(CPPFLAGS) $(THREAD_CPPFLAGS) -DFRONTEND
SHLIB_LINK += -lm
OBJS= numeric.o datetime.o common.o dt_common.o timestamp.o interval.o \
diff -cr postgresql-snapshot-030325/src/interfaces/ecpg/preproc/Makefile postgresql-newsignal/src/interfaces/ecpg/preproc/Makefile
*** postgresql-snapshot-030325/src/interfaces/ecpg/preproc/Makefile Tue Mar 2 07:45:05 2004
--- postgresql-newsignal/src/interfaces/ecpg/preproc/Makefile Tue Apr 13 21:04:47 2004
***************
*** 12,18 ****
-DMAJOR_VERSION=$(MAJOR_VERSION) \
-DMINOR_VERSION=$(MINOR_VERSION) -DPATCHLEVEL=$(PATCHLEVEL) \
-DINCLUDEDIR=\"$(includedir)\" \
! -DPKGINCLUDEDIR=\"$(pkgincludedir)\"
ifeq ($(GCC), yes)
override CFLAGS += -Wno-error
--- 12,19 ----
-DMAJOR_VERSION=$(MAJOR_VERSION) \
-DMINOR_VERSION=$(MINOR_VERSION) -DPATCHLEVEL=$(PATCHLEVEL) \
-DINCLUDEDIR=\"$(includedir)\" \
! -DPKGINCLUDEDIR=\"$(pkgincludedir)\" \
! -DFRONTEND
ifeq ($(GCC), yes)
override CFLAGS += -Wno-error
I assume we want to integrate our own timezone library into PostgreSQL.
We have talked about it for a few releases, and Magnus has done it for
us because Win32 needs it for pre-1970 dates.
The patch to PostgreSQL is 100 lines of diff, and the changes to Olson's
timezone library is 100 lines of diff. The timezone library itself is
173k tgz file, which will add to our download size, but not too much.
Your patch has been added to the PostgreSQL unapplied patches list at:
http://momjian.postgresql.org/cgi-bin/pgpatches
I will try to apply it within the next 48 hours.
---------------------------------------------------------------------------
Magnus Hagander wrote:
This mail apparantly didn't make it through because it was too large.
Resending it without the largest file, tzlib.tgz. I've put this file (+
the patches) on http://www.hagander.net/pgsql/.//Magnus
-----Original Message-----
From: Magnus Hagander
Sent: Sunday, April 18, 2004 9:05 PM
To: pgsql-patches@postgresql.org
Subject: Timezone libraryHi!
Attached is a slightly modified version of Arthur David
Olson's timezone library for inclusion in postgresql, pending
review of course. This is required to give win32 working
timezone functions on dates before 1970. It has also been
indicated that it might be interesting to have such a library
on other platforms as well. Note that with this enabled, the
timezone data files are installed in <pgdir>/share/timezone.Attached are the following files:
* tzlib.tgz - the timezone implementation, including the
datafiles required to build the timezone data files.
* timezone.patch - the modifications made to existing files
to make it work. This includes setting the FRONTEND flat for ecpg.
* tzcode.diff - lists the modifications that I have made to
the files from the original timezone files, for reference.Bruce has offered to do the autoconf and Makefile integration
parts - I don't know autoconf enough to do that part... The
#defines necessary are "activeted" by setting USE_PGTZ. Of
course, the appropriate files also have to be linked in.With this patch applied, and the timezone data installed,
win32 now passes all time-related regression tests (3 now
fails, 2 "known" and one being locale-inflicted, still
investigating that one)//Magnus
<<tzcode_changes.diff>> <<timezone.patch>>
Content-Description: tzcode_changes.diff
[ Attachment, skipping... ]
Content-Description: timezone.patch
[ Attachment, skipping... ]
---------------------------(end of broadcast)---------------------------
TIP 4: Don't 'kill -9' the postmaster
--
Bruce Momjian | http://candle.pha.pa.us
pgman@candle.pha.pa.us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073
Am Tuesday 27 April 2004 05:06 schrieb Bruce Momjian:
I assume we want to integrate our own timezone library into PostgreSQL.
We have talked about it for a few releases, and Magnus has done it for
us because Win32 needs it for pre-1970 dates.The patch to PostgreSQL is 100 lines of diff, and the changes to Olson's
timezone library is 100 lines of diff. The timezone library itself is
173k tgz file, which will add to our download size, but not too much.
The patch says: Note! Not integrated with makefile/autoconf. Bruce has
promised to fix this :-)
Where is the patch for that? ISTM that this should be available first or the
build will be broken?
Peter Eisentraut wrote:
Am Tuesday 27 April 2004 05:06 schrieb Bruce Momjian:
I assume we want to integrate our own timezone library into PostgreSQL.
We have talked about it for a few releases, and Magnus has done it for
us because Win32 needs it for pre-1970 dates.The patch to PostgreSQL is 100 lines of diff, and the changes to Olson's
timezone library is 100 lines of diff. The timezone library itself is
173k tgz file, which will add to our download size, but not too much.The patch says: Note! Not integrated with makefile/autoconf. Bruce has
promised to fix this :-)Where is the patch for that? ISTM that this should be available first or the
build will be broken?
I will do the integration when I apply the patch. It is a simple enable
for win32 port only at this point. If we want to use it for Unix, we
can discuss that.
--
Bruce Momjian | http://candle.pha.pa.us
pgman@candle.pha.pa.us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073I have added this to CVS under src/timezone and have integrated this into
configure.
However, I am getting a compile error, probably because my OS has a
timezone function defined in time.h:
gcc -O2 -fno-strict-aliasing -Wall -Wmissing-prototypes
-Wmissing-declarations -O1 -Wall -Wmissing-prototypes
-Wmissing-declarations -Wpointer-arith -Wcast-align -I../../src/include
-I/usr/local/include/readline -I/usr/contrib/include -c -o localtime.o
localtime.c
In file included from private.h:91,
from localtime.c:25:
/usr/include/time.h:104: `pg_timezone' redeclared as different kind of
symbol
localtime.c:5: previous declaration of `pg_timezone'
localtime.c:203: `pg_timezone' redeclared as different kind of symbol
/usr/include/time.h:104: previous declaration of `pg_timezone'
localtime.c:933: warning: static declaration for `tzsetwall' follows
non-static
gmake: *** [localtime.o] Error 1
Fortunately only Win32 compiles this code right now.
---------------------------------------------------------------------------
Magnus Hagander wrote:
This mail apparantly didn't make it through because it was too large.
Resending it without the largest file, tzlib.tgz. I've put this file (+
the patches) on http://www.hagander.net/pgsql/.//Magnus
-----Original Message-----
From: Magnus Hagander
Sent: Sunday, April 18, 2004 9:05 PM
To: pgsql-patches@postgresql.org
Subject: Timezone libraryHi!
Attached is a slightly modified version of Arthur David
Olson's timezone library for inclusion in postgresql, pending
review of course. This is required to give win32 working
timezone functions on dates before 1970. It has also been
indicated that it might be interesting to have such a library
on other platforms as well. Note that with this enabled, the
timezone data files are installed in <pgdir>/share/timezone.Attached are the following files:
* tzlib.tgz - the timezone implementation, including the
datafiles required to build the timezone data files.
* timezone.patch - the modifications made to existing files
to make it work. This includes setting the FRONTEND flat for ecpg.
* tzcode.diff - lists the modifications that I have made to
the files from the original timezone files, for reference.Bruce has offered to do the autoconf and Makefile integration
parts - I don't know autoconf enough to do that part... The
#defines necessary are "activeted" by setting USE_PGTZ. Of
course, the appropriate files also have to be linked in.With this patch applied, and the timezone data installed,
win32 now passes all time-related regression tests (3 now
fails, 2 "known" and one being locale-inflicted, still
investigating that one)//Magnus
<<tzcode_changes.diff>> <<timezone.patch>>
Content-Description: tzcode_changes.diff
[ Attachment, skipping... ]
Content-Description: timezone.patch
[ Attachment, skipping... ]
---------------------------(end of broadcast)---------------------------
TIP 4: Don't 'kill -9' the postmaster
--
Bruce Momjian | http://candle.pha.pa.us
pgman@candle.pha.pa.us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073
Bruce Momjian <pgman@candle.pha.pa.us> writes:
/usr/include/time.h:104: `pg_timezone' redeclared as different kind of
symbol
Your <time.h> really defines "pg_timezone"?? I'm wondering if this is
an indirect effect of a macro naming collision.
regards, tom lane
Bruce Momjian wrote:
I have added this to CVS under src/timezone and have integrated this
into configure.
I think it would be better to put it under one of the generic
subdirectories, such as src/utils/timezone. We may have to add
additional libraries of this kind in the future, and it would be better
to have them grouped somehow.
Peter Eisentraut wrote:
Bruce Momjian wrote:
I have added this to CVS under src/timezone and have integrated this
into configure.I think it would be better to put it under one of the generic
subdirectories, such as src/utils/timezone. We may have to add
additional libraries of this kind in the future, and it would be better
to have them grouped somehow.
I put it under /src because it is a separate module of code that we may
need to update someday with a new release. In that way it isn't really
part of our backend binary but a module we call.
--
Bruce Momjian | http://candle.pha.pa.us
pgman@candle.pha.pa.us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073Am Freitag, 30. April 2004 15:27 schrieb Bruce Momjian:
I put it under /src because it is a separate module of code that we may
need to update someday with a new release. In that way it isn't really
part of our backend binary but a module we call.
The same could be said of regex, but that has been very happy in its place.
Peter Eisentraut wrote:
Am Freitag, 30. April 2004 15:27 schrieb Bruce Momjian:
I put it under /src because it is a separate module of code that we may
need to update someday with a new release. In that way it isn't really
part of our backend binary but a module we call.The same could be said of regex, but that has been very happy in its place.
True, but timezone has to install files in pgsql/timezone, and it isn't
_in_ the binary like the regex is in the binary.
Do we want a subdirectory of backend installing its own files in a
separate directory? Seems strange.
--
Bruce Momjian | http://candle.pha.pa.us
pgman@candle.pha.pa.us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073Bruce Momjian wrote:
Peter Eisentraut wrote:
Am Freitag, 30. April 2004 15:27 schrieb Bruce Momjian:
I put it under /src because it is a separate module of code that we may
need to update someday with a new release. In that way it isn't really
part of our backend binary but a module we call.The same could be said of regex, but that has been very happy in its place.
True, but timezone has to install files in pgsql/timezone, and it isn't
_in_ the binary like the regex is in the binary.Do we want a subdirectory of backend installing its own files in a
separate directory? Seems strange.
I realize backend does create stuff in share and lib and stuff. If
folks want it under backend, I can move it. You want it where?
--
Bruce Momjian | http://candle.pha.pa.us
pgman@candle.pha.pa.us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073Tom Lane wrote:
Bruce Momjian <pgman@candle.pha.pa.us> writes:
/usr/include/time.h:104: `pg_timezone' redeclared as different kind of
symbolYour <time.h> really defines "pg_timezone"?? I'm wondering if this is
an indirect effect of a macro naming collision.
I applied the following patch to allow it to compile on Unix. I had to
mask the 'timezone' define before including "time.h".
I also pass pkglibdir into the timezone makefile like we do for fmgr so
it will know where to find the timezone database. This is probably only
an interim solution. We need a more general fix for lib/ anyway, and
this is basically the same problem (that the lib directory is compiled
into the binary and can't be moved).
With this change it compiles but I get this error:
zic -d /usr/local/pgsql/share/timezone data/africa data/antarctica
data/asia data/australasia data/europe data/northamerica
data/southamerica data/pacificnew data/etcetera data/factory
data/backward data/systemv data/solar87 data/solar88 data/solar89
"data/solar87", line 385: too many local time types (rule from
"data/solar87", line 45)
Magnus doesn't see it on Win32.
To test, define PGTZ in pg_config.h and set PGTZ=yes in Makefile.global.
--
Bruce Momjian | http://candle.pha.pa.us
pgman@candle.pha.pa.us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073Attachments:
/bjm/difftext/plainDownload
Index: src/timezone/Makefile
===================================================================
RCS file: /cvsroot/pgsql-server/src/timezone/Makefile,v
retrieving revision 1.2
diff -c -c -r1.2 Makefile
*** src/timezone/Makefile 30 Apr 2004 04:31:52 -0000 1.2
--- src/timezone/Makefile 30 Apr 2004 14:21:23 -0000
***************
*** 8,17 ****
#
#-------------------------------------------------------------------------
! subdir = src/tz
top_builddir = ../..
include $(top_builddir)/src/Makefile.global
OBJS= asctime.o difftime.o localtime.o pgtz.o
ZICOBJS= zic.o ialloc.o scheck.o localtime.o asctime.o pgtz.o
--- 8,19 ----
#
#-------------------------------------------------------------------------
! subdir = src/timezone
top_builddir = ../..
include $(top_builddir)/src/Makefile.global
+ override CPPFLAGS += -DPKGLIBDIR=\"$(pkglibdir)\"
+
OBJS= asctime.o difftime.o localtime.o pgtz.o
ZICOBJS= zic.o ialloc.o scheck.o localtime.o asctime.o pgtz.o
***************
*** 25,30 ****
--- 27,33 ----
$(LD) $(LDREL) $(LDOUT) SUBSYS.o $(OBJS)
zic: $(ZICOBJS)
+ $(CC) $(CFLAGS) $(ZICOBJS) $(LDFLAGS) $(LIBS) -o $@$(X)
install: zic
zic -d $(datadir)/timezone $(TZDATAFILES)
Index: src/timezone/pgtz.c
===================================================================
RCS file: /cvsroot/pgsql-server/src/timezone/pgtz.c,v
retrieving revision 1.1
diff -c -c -r1.1 pgtz.c
*** src/timezone/pgtz.c 30 Apr 2004 04:09:23 -0000 1.1
--- src/timezone/pgtz.c 30 Apr 2004 14:21:23 -0000
***************
*** 15,21 ****
#include "tzfile.h"
- #ifdef WIN32
static char tzdir[MAXPGPATH];
static int done_tzdir = 0;
char *pgwin32_TZDIR(void) {
--- 15,20 ----
***************
*** 23,30 ****
--- 22,33 ----
if (done_tzdir)
return tzdir;
+ #ifndef WIN32
+ StrNCpy(tzdir,PKGLIBDIR, MAXPGPATH);
+ #else
if (GetModuleFileName(NULL,tzdir,MAXPGPATH) == 0)
return NULL;
+ #endif
canonicalize_path(tzdir);
if ((p = last_path_separator(tzdir)) == NULL)
***************
*** 32,42 ****
else
*p = '\0';
! strcat(tzdir,"/../share/timezone");
done_tzdir=1;
return tzdir;
}
- #else
- #error pgwin32_TZDIR not implemented on non win32 yet!
- #endif
--- 35,42 ----
else
*p = '\0';
! strcat(tzdir,"/../timezone");
done_tzdir=1;
return tzdir;
}
Index: src/timezone/private.h
===================================================================
RCS file: /cvsroot/pgsql-server/src/timezone/private.h,v
retrieving revision 1.2
diff -c -c -r1.2 private.h
*** src/timezone/private.h 30 Apr 2004 04:44:06 -0000 1.2
--- src/timezone/private.h 30 Apr 2004 14:21:23 -0000
***************
*** 88,94 ****
--- 88,98 ----
#include "errno.h"
#include "string.h"
#include "limits.h" /* for CHAR_BIT */
+ #define _timezone timezone
+ #undef timezone
#include "time.h"
+ #define timezone _timezone
+ #undef _timezone
#include "stdlib.h"
#if HAVE_GETTEXT - 0
Peter Eisentraut <peter_e@gmx.net> writes:
I think it would be better to put it under one of the generic
subdirectories, such as src/utils/timezone. We may have to add
additional libraries of this kind in the future, and it would be better
to have them grouped somehow.
The closest precedent we have at the moment is src/backend/regex, which
is a fairly large library that we imported lock-stock-and-barrel.
However, this precedent wouldn't hold if the code needs to be usable
from non-backend code. ecpg seems to want to use the tz code ...
Right at the moment I am thinking we may end up keeping only about one
file out of the existing src/timezone import, so maybe it doesn't need
a whole subdirectory anyway?
regards, tom lane
Tom Lane wrote:
Peter Eisentraut <peter_e@gmx.net> writes:
I think it would be better to put it under one of the generic
subdirectories, such as src/utils/timezone. We may have to add
additional libraries of this kind in the future, and it would be better
to have them grouped somehow.The closest precedent we have at the moment is src/backend/regex, which
is a fairly large library that we imported lock-stock-and-barrel.
However, this precedent wouldn't hold if the code needs to be usable
from non-backend code. ecpg seems to want to use the tz code ...
I asked if ecpg should use the same timezone database as the server and
someone said it could just use the local one.
I was concerned ecpg would be reading structure and had to assume the
timezones split up the same way as the server.
Right at the moment I am thinking we may end up keeping only about one
file out of the existing src/timezone import, so maybe it doesn't need
a whole subdirectory anyway?
Not sure, but it is easy to move.
--
Bruce Momjian | http://candle.pha.pa.us
pgman@candle.pha.pa.us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073I am getting farther with the timezone library on Unix.
First, I realized that the share/timezone library doesn't have a
localtime file by default, so the PostgreSQL server doesn't know the
local timezone. I added that and got:
test=> select current_timestamp;
timestamptz
-------------------------------
2004-04-30 18:52:14.930852+00
(1 row)
test=> select timeofday();
timeofday
-------------------------------------
Fri Apr 30 22:52:17.686324 2004 EDT
(1 row)
(How are we going to deal with the missing localtime file?)
It is obviously seeing my local timezone, but it doesn't know that it is
+04. It is at least reading the right directory.
To test, define USE_PGTZ in pg_config.h and set USE_PGTZ=yes in Makefile.global.
--
Bruce Momjian | http://candle.pha.pa.us
pgman@candle.pha.pa.us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073Bruce Momjian <pgman@candle.pha.pa.us> writes:
Tom Lane wrote:
Right at the moment I am thinking we may end up keeping only about one
file out of the existing src/timezone import, so maybe it doesn't need
a whole subdirectory anyway?
We will still need the timezone database files.
Good point, I had forgotten about the data/ subdirectory.
I think src/timezone is an okay place for it ...
regards, tom lane
Import Notes
Reply to msg id not found: 200405012010.i41KAwj00952@candle.pha.pa.usReference msg id not found: 200405012010.i41KAwj00952@candle.pha.pa.us | Resolved by subject fallback
Tom Lane wrote:
Bruce Momjian <pgman@candle.pha.pa.us> writes:
Tom Lane wrote:
Right at the moment I am thinking we may end up keeping only about one
file out of the existing src/timezone import, so maybe it doesn't need
a whole subdirectory anyway?We will still need the timezone database files.
Good point, I had forgotten about the data/ subdirectory.
I think src/timezone is an okay place for it ...
Digging into getting this working on unix, it seems the configure tests
that look at /usr/include are then setting values related to timezone
that don't apply to src/timezone code. Not sure how I am going to fix
that. I tried adding -Isrc/timezone but that didn't help. I am trying
to find the exact setting that it fault first.
--
Bruce Momjian | http://candle.pha.pa.us
pgman@candle.pha.pa.us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073Bruce Momjian <pgman@candle.pha.pa.us> writes:
Digging into getting this working on unix, it seems the configure tests
that look at /usr/include are then setting values related to timezone
that don't apply to src/timezone code. Not sure how I am going to fix
that.
You rip 'em all out is how. If we are using our own tz library then
there is no need for configure-time determination of its capabilities.
regards, tom lane
Tom Lane wrote:
Bruce Momjian <pgman@candle.pha.pa.us> writes:
Digging into getting this working on unix, it seems the configure tests
that look at /usr/include are then setting values related to timezone
that don't apply to src/timezone code. Not sure how I am going to fix
that.You rip 'em all out is how. If we are using our own tz library then
there is no need for configure-time determination of its capabilities.
Yea, but I have to find the right values for the timezone database, and
right now I am having problems.
--
Bruce Momjian | http://candle.pha.pa.us
pgman@candle.pha.pa.us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073Bruce Momjian wrote:
Tom Lane wrote:
Bruce Momjian <pgman@candle.pha.pa.us> writes:
Digging into getting this working on unix, it seems the configure tests
that look at /usr/include are then setting values related to timezone
that don't apply to src/timezone code. Not sure how I am going to fix
that.You rip 'em all out is how. If we are using our own tz library then
there is no need for configure-time determination of its capabilities.Yea, but I have to find the right values for the timezone database, and
right now I am having problems.
I ran configure under mingw and compared it to the Unix pg_config.h. I
made the timezone-related changes to Unix so it had the same values as
mingw, but it still fails.
I am giving up for now.
--
Bruce Momjian | http://candle.pha.pa.us
pgman@candle.pha.pa.us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073