| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | Thomas Hallgren <thhal(at)mailblocks(dot)com> |
| Cc: | pgsql-hackers(at)postgresql(dot)org, Joe Conway <mail(at)joeconway(dot)com> |
| Subject: | Re: Module dependency on PostgeSQL version |
| Date: | 2004-05-11 21:00:08 |
| Message-ID: | 15112.1084309208@sss.pgh.pa.us |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Thomas Hallgren <thhal(at)mailblocks(dot)com> writes:
> The PGSQL_MAJOR_VER and PGSQL_MINOR_VER does not exist today. Ideally,
> I'd like to find them in src/Makefile.global. Only thing present seems
> to be the VERSION. I'd like to see something like:
> PGSQL_MAJOR_VER := 7
> PGSQL_MINOR_VER := 5
> PGSQL_PATCH_VER := devel
> VERSION := $(PGSQL_MAJOR_VER).$(PGSQL_MINOR_VER).$(PGSQL_PATCH_VER)
> To be used in CPPFLAGS as: -DPGSQL_MAJOR_VER=$(PGSQL_MAJOR_VER) etc.
Wouldn't it be better to just put those #defines in to begin with,
rather than requiring people to hack on their CPPFLAGS? I don't offhand
see much need for knowing the PG version at the Makefile level, but I
do see the usefulness at the C-code level.
I think Joe Conway is already doing something like this for pl/r ...
leastwise he's shown bits of #ifdef'd code in past email. It would
be interesting to see the details of his solution.
regards, tom lane
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bruce Momjian | 2004-05-11 21:01:45 | Re: XLog: how to log? |
| Previous Message | Bruce Momjian | 2004-05-11 20:55:26 | Re: PITR Signalling the Archiver |