In 020_createdb.pl, change order of command-line arguments.
authorRobert Haas <rhaas@postgresql.org>
Tue, 29 Mar 2022 17:48:39 +0000 (13:48 -0400)
committerRobert Haas <rhaas@postgresql.org>
Tue, 29 Mar 2022 17:48:39 +0000 (13:48 -0400)
Linux thinks that something like "createdb foo -S bar" is perfectly
fine, but Windows wants the options to precede any bare arguments, so
we must write "createdb -S bar foo" for portability.

Per reports from CI, the buildfarm, and Andres.

Discussion: http://postgr.es/m/20220329173536.7d2ywdatsprxl4x6@alap3.anarazel.de

src/bin/scripts/t/020_createdb.pl

index 14d3a9563d1ffd204890f7bd007ffb84a5dfe172..b81f06a251e456eb39f582be59aa963dc4a08ee9 100644 (file)
@@ -115,12 +115,12 @@ $node->command_checks_all(
 
 # Check database creation strategy
 $node->issues_sql_like(
-   [ 'createdb', '-T', 'foobar2', 'foobar6', '-S', 'wal_log'],
+   [ 'createdb', '-T', 'foobar2', '-S', 'wal_log', 'foobar6' ],
    qr/statement: CREATE DATABASE foobar6 STRATEGY wal_log TEMPLATE foobar2/,
    'create database with WAL_LOG strategy');
 
 $node->issues_sql_like(
-   [ 'createdb', '-T', 'foobar2', 'foobar7', '-S', 'file_copy'],
+   [ 'createdb', '-T', 'foobar2', '-S', 'file_copy', 'foobar7' ],
    qr/statement: CREATE DATABASE foobar7 STRATEGY file_copy TEMPLATE foobar2/,
    'create database with FILE_COPY strategy');