Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#6791 closed defect (fixed)

configure has a bashism

Reported by: Even Rouault Owned by: Even Rouault
Priority: normal Milestone: 2.1.4
Component: ConfigBuild Version: unspecified
Severity: normal Keywords:
Cc:

Description

Reported as https://lists.osgeo.org/pipermail/gdal-dev/2017-January/045875.html

"""Not a big deal but it will cause configure to fail with a POSIX-conforming shell without bash extensions, and pkgsrc's portability check code throws an error."""

$NetBSD$

Remediate bashism.
Not yet reported upstream.

--- configure.orig      2016-10-24 11:00:31.000000000 +0000
+++ configure
@@ -16861,7 +16861,7 @@ ac_link='$CC -o conftest$ac_exeext $CFLA
 ac_compiler_gnu=$ac_cv_c_compiler_gnu
 
 
-if test "x$CXX" == x ; then
+if test "x$CXX" = x ; then
   as_fn_error $? "\"You don't have a working C++ compiler.\"" "$LINENO" 5
 fi

Change History (3)

comment:1 by Even Rouault, 7 years ago

Fixed in trunk r37198

comment:2 by Even Rouault, 7 years ago

Resolution: fixed
Status: newclosed

In 37199:

configure: remove bashism (reported by Greg Troxel) (fixes #6791)

comment:3 by Even Rouault, 7 years ago

Milestone: 2.1.4
Note: See TracTickets for help on using tickets.