[Serdev] minor postgres patches

Alexander Mayrhofer axelm at nic.at
Fri Jul 25 08:02:30 UTC 2003


Hi,

SER's postgres module does not build properly in yesterday's CVS. I
suggest two minor patches to fix that:

- adding MODULE_VERSION

---snip---

--- modules/postgres/db_mod.c.orig      Fri Jul 25 09:49:58 2003
+++ modules/postgres/db_mod.c   Fri Jul 25 09:50:30 2003
@@ -37,6 +37,7 @@
 #include "../../sr_module.h"
 #include "dbase.h"

+MODULE_VERSION

 static int mod_init(void);

---snip---

- adding "debian-friendly" includes

---snip---

--- modules/postgres/Makefile.orig      Fri Jul 25 09:49:44 2003
+++ modules/postgres/Makefile   Fri Jul 25 09:54:50 2003
@@ -7,7 +7,7 @@
 NAME=postgres.so

 # libpq-fe.h locations
-DEFS +=-I/usr/local/pgsql/include
+DEFS +=-I/usr/local/pgsql/include -I/usr/include/postgresql
 LIBS=-L$(LOCALBASE)/pgsql/lib -L$(LOCALBASE)/lib/pgsql -L/usr/pkg/lib \
                -L/usr/pkg/lib/pgsql -lpq

---snip---

There are some architecture-dependent #define's in the module's sources, e.g. 
Linux seems only to be supported on x86. I've successfully tried the module on 
Linux/powerpc (after changing __x86 to __powerpc in aug_sysdep.h), so i 
wonder if that dependency is neccessary at all? Some endianess problems i did 
miss?

thanks

Alex Mayrhofer
nic.at




More information about the Serdev mailing list