pgsql: Change qr/foo$/m to qr/foo\n/m, for Perl 5.8.8.

From: Noah Misch <noah(at)leadboat(dot)com>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Change qr/foo$/m to qr/foo\n/m, for Perl 5.8.8.
Date: 2016-11-08 01:31:35
Message-ID: E1c3vGB-0007tZ-NK@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Change qr/foo$/m to qr/foo\n/m, for Perl 5.8.8.

In each case, absence of a trailing newline would itself constitute a
PostgreSQL bug. Therefore, this slightly enhances the changed tests.
This works around a bug that last appeared in Perl 5.8.8, fixing
src/test/modules/test_pg_dump when run against that version. Commit
e7293e3271bf618eeb2d4779a15fc516a69fe463 worked around the bug, but the
subsequent addition of test_pg_dump introduced affected code. As that
commit had shown, slight increases in pattern complexity can suppress
the bug. This commit edits qr/foo$/m patterns too complex to encounter
the bug today, for style consistency and robustness against unrelated
pattern changes. Back-patch to 9.6, where test_pg_dump was introduced.

As of this writing, a fresh MSYS installation includes an affected Perl
5.8.8. The Perl 5.8.8 in Red Hat Enterprise Linux 5.11 carries a patch
that renders it unaffected, but the Perl 5.8.5 of Red Hat Enterprise
Linux 4.4 is affected.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/650b96707672599e290c982dd63e3a896dbbaba6

Modified Files
--------------
src/bin/pg_basebackup/t/010_pg_basebackup.pl | 8 ++---
src/test/modules/test_pg_dump/t/001_base.pl | 48 ++++++++++++++--------------
src/test/perl/README | 4 +++
src/tools/msvc/Project.pm | 2 +-
4 files changed, 32 insertions(+), 30 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Amit Kapila 2016-11-08 04:55:11 Re: Re: [COMMITTERS] pgsql: Change the way that LWLocks for extensions are allocated.
Previous Message Tom Lane 2016-11-07 17:08:29 pgsql: Band-aid fix for incorrect use of view options as StdRdOptions.