From: | "Erik Rijkers" <er(at)xs4all(dot)nl> |
---|---|
To: | "Marco Nenciarini" <marco(dot)nenciarini(at)2ndquadrant(dot)it> |
Cc: | "PostgreSQL-development" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: File based Incremental backup v8 |
Date: | 2015-01-31 16:22:58 |
Message-ID: | 8fec6739f5e3b1d62cd0a824d3bf97f5.squirrel@webmail.xs4all.nl |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
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
From | Date | Subject | |
---|---|---|---|
Next Message | Petr Jelinek | 2015-01-31 19:08:19 | Re: TABLESAMPLE patch |
Previous Message | Marco Nenciarini | 2015-01-31 14:14:02 | Re: File based Incremental backup v8 |