Re: File based Incremental backup v8

On Sat, January 31, 2015 15:14, Marco Nenciarini wrote:

> 0001-public-parse_filename_for_nontemp_relation.patch
> 0002-copydir-LSN-v2.patch
> 0003-File-based-incremental-backup-v8.patch

Hi,

It looks like it only compiles with assert enabled.

This is perhaps not yet really a problem at this stage but I thought I'd mention it:

make --quiet -j 8
In file included from gram.y:14403:0:
scan.c: In function ‘yy_try_NUL_trans’:
scan.c:10174:23: warning: unused variable ‘yyg’ [-Wunused-variable]    struct yyguts_t * yyg = (struct
yyguts_t*)yyscanner;/* This var may be unused depending upon options. */                      ^
 
basebackup.c: In function ‘writeBackupProfileLine’:
basebackup.c:1545:8: warning: format ‘%lld’ expects argument of type ‘long long int’, but argument 8 has type
‘__off_t’
[-Wformat=]       filename);       ^
basebackup.c:1545:8: warning: format ‘%lld’ expects argument of type ‘long long int’, but argument 8 has type
‘__off_t’
[-Wformat=]
pg_basebackup.c: In function ‘ReceiveTarFile’:
pg_basebackup.c:858:2: warning: implicit declaration of function ‘assert’ [-Wimplicit-function-declaration] assert(res
||(strcmp(basedir, "-") == 0)); ^
 
pg_basebackup.c:865:2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement] gzFile
ztarfile= NULL; ^
 
pg_basebackup.o: In function `ReceiveAndUnpackTarFile':
pg_basebackup.c:(.text+0x690): undefined reference to `assert'
pg_basebackup.o: In function `ReceiveTarFile':
pg_basebackup.c:(.text+0xeb0): undefined reference to `assert'
pg_basebackup.c:(.text+0x10ad): undefined reference to `assert'
collect2: error: ld returned 1 exit status
make[3]: *** [pg_basebackup] Error 1
make[3]: *** Waiting for unfinished jobs....
make[2]: *** [all-pg_basebackup-recurse] Error 2
make[2]: *** Waiting for unfinished jobs....
make[1]: *** [all-bin-recurse] Error 2
make: *** [all-src-recurse] Error 2

The configure used was:
./configure \--prefix=/home/aardvark/pg_stuff/pg_installations/pgsql.incremental_backup
\--bindir=/home/aardvark/pg_stuff/pg_installations/pgsql.incremental_backup/bin.fast
\--libdir=/home/aardvark/pg_stuff/pg_installations/pgsql.incremental_backup/lib.fast\--with-pgport=6973 --quiet
--enable-depend\--with-extra-version=_incremental_backup_20150131_1521_08bd0c581158 \--with-openssl --with-perl
--with-libxml--with-libxslt --with-zlib
 


A build with --enable-cassert and --enable-debug builds fine:

./configure \--prefix=/home/aardvark/pg_stuff/pg_installations/pgsql.incremental_backup
\--bindir=/home/aardvark/pg_stuff/pg_installations/pgsql.incremental_backup/bin
\--libdir=/home/aardvark/pg_stuff/pg_installations/pgsql.incremental_backup/lib\--with-pgport=6973 --quiet
--enable-depend\--with-extra-version=_incremental_backup_20150131_1628_08bd0c581158 \--enable-cassert --enable-debug
\--with-openssl--with-perl --with-libxml --with-libxslt --with-zlib
 

I will further test with that.


thanks,

Erik Rijkers