On Saturday 01 December 2007 13:24, Marc Cousin wrote: > Yes it does. OK, thanks. I've now added it to the trunk and will commit soon. That is probably the simplest solution to handle the various PostgreSQL versions.
Best regards, Kern > > On Saturday 01 December 2007 13:12:17 Kern Sibbald wrote: > > On Saturday 01 December 2007 12:52, Marc Cousin wrote: > > > I've been working on postgresql 8.3 for a few days, and I just tried to > > > recompile bacula with it. > > > > > > I had the following error : > > > > > > Compiling postgresql.c > > > postgresql.c: In function 'int my_postgresql_currval(B_DB*, char*)': > > > postgresql.c:566: error: 'NAMEDATALEN' was not declared in this scope > > > postgresql.c:572: error: 'sequence' was not declared in this scope > > > postgresql.c:574: error: 'sequence' was not declared in this scope > > > postgresql.c:580: error: 'sequence' was not declared in this scope > > > postgresql.c:581: error: 'query' was not declared in this scope > > > postgresql.c: In function 'int my_postgresql_batch_start(JCR*, B_DB*)': > > > postgresql.c:617: warning: deprecated conversion from string constant > > > to 'char*' postgresql.c: In function 'int > > > my_postgresql_batch_insert(JCR*, B_DB*, ATTR_DBR*)': postgresql.c:721: > > > warning: deprecated conversion from string constant to 'char*' make[1]: > > > *** [postgresql.o] Error 1 make[1]: Leaving directory > > > `/usr/src/bacula-2.2.6/src/cats' > > > > > > The reason can be found in the postgresql 8.3 changelog : > > > Move NAMEDATALEN definition from postgres_ext.h to pg_config_manual.h > > > (Peter) > > > > > > I replaced the include in src/cats/postgresql.c to : > > > > > > #include "pg_config_manual.h" /* needed for NAMEDATALEN */ > > > > > > And it compiled ... > > > > Does it still compile on your system if you *add* the new include and > > keep the old one at the same time? > > > > Regards, > > > > Kern > > > > > So I guess there will be something to be done about it > > > (cleaner than what I've done, I mean :) ) > > > > > > ----------------------------------------------------------------------- > > >-- SF.Net email is sponsored by: The Future of Linux Business White > > > Paper from Novell. From the desktop to the data center, Linux is going > > > mainstream. Let it simplify your IT future. > > > http://altfarm.mediaplex.com/ad/ck/8857-50307-18918-4 > > > _______________________________________________ > > > Bacula-devel mailing list > > > [email protected] > > > https://lists.sourceforge.net/lists/listinfo/bacula-devel > > ------------------------------------------------------------------------- > SF.Net email is sponsored by: The Future of Linux Business White Paper > from Novell. From the desktop to the data center, Linux is going > mainstream. Let it simplify your IT future. > http://altfarm.mediaplex.com/ad/ck/8857-50307-18918-4 > _______________________________________________ > Bacula-devel mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/bacula-devel ------------------------------------------------------------------------- SF.Net email is sponsored by: The Future of Linux Business White Paper from Novell. From the desktop to the data center, Linux is going mainstream. Let it simplify your IT future. http://altfarm.mediaplex.com/ad/ck/8857-50307-18918-4 _______________________________________________ Bacula-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/bacula-devel
