pgsql: Fix Perl coding error in msvc build system - Mailing list pgsql-committers

From Peter Eisentraut
Subject pgsql: Fix Perl coding error in msvc build system
Date
Msg-id E1YOFsy-0002pD-Ax@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix Perl coding error in msvc build system

Code like

    open(P, "cl /? 2>&1 |") || die "cl command not found";

does not actually catch any errors, because the exit status of the
command before the pipe is ignored.  The fix is to look at $?.

This also gave the opportunity to clean up the logic of this code a bit.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/d30292b8c45a1d909ff7d84bd6787c8827134fc3

Modified Files
--------------
src/tools/msvc/Solution.pm        |   14 +++-----------
src/tools/msvc/VSObjectFactory.pm |   28 +++++++---------------------
2 files changed, 10 insertions(+), 32 deletions(-)


pgsql-committers by date:

Previous
From: Alvaro Herrera
Date:
Subject: pgsql: Fix opclass/opfamily identity strings
Next
From: Tom Lane
Date:
Subject: pgsql: Split array_push into separate array_append and array_prepend fu