Heikki Linnakangas wrote:
> Addressed most of your comments, and Michael's. Another version attached.
Looking at the set of TAP tests, I think that those lines open again
the door of CVE-2014-0067 (vulnerability with make check) on Windows: # Initialize master, data checksums are
mandatory remove_tree($test_master_datadir); system_or_bail("initdb -N -A trust -D $test_master_datadir
>>$log_path");
IMO we should use standard_initdb in TestLib.pm instead as pg_regress
--config-auth would be used for SSPI. standard_initdb should be
extended a bit as well to be able to pass a path to logs with
/dev/null as default. TAP tests do not run on Windows, still I think
that it would be better to cover any eventuality in this area before
we forget. Already mentioned by Peter, but I think as well that the
new additions to TAP should be a separate patch.
Random thought (not related to this patch), have a new option in
initdb doing this legwork:
+ # Accept replication connections on master
+ append_to_file("$test_master_datadir/pg_hba.conf", qq(
+local replication all trust
+host replication all 127.0.0.1/32 trust
+host replication all ::1/128 trust
+));
I am still getting a warning when building with MSVC: xlogreader.obj : warning LNK4049: locally defined symbol
pg_crc32c_table imported
[C:\Users\ioltas\git\postgres\pg_rewind.vcxproj] 1 Warning(s) 0 Error(s)
Nitpicking: number of spaces here is incorrect:
+ that when <productname>PostgreSQL</> is started, it will start replay
+ from that checkpoint and apply all the required WAL.)
+ </para>
The header of src/bin/pg_rewind/Makefile mentions pg_basebackup:
+#-------------------------------------------------------------------------
+#
+# Makefile for src/bin/pg_basebackup
In this Makefile as well, I think that EXTRA_CLEAN can be removed:+EXTRA_CLEAN = $(RMGRDESCSOURCES) xlogreader.c
Regards,
--
Michael