Ticket #420 (closed defect: fixed)

Opened 16 months ago

Last modified 16 months ago

Non-portable test in configure script

Reported by: anon Owned by: somebody
Priority: normal Milestone:
Component: component1 Version: 2.9.22
Severity: normal Keywords:
Cc:

Description

The configure(.in) script contains the following test:

if test ${a} == "gpgsql"; then

However the "==" notation is a bash'ism, it is not a standard (POSIX) requirement for the "test" or "[" command to implement it. Worse, in many implementations this expression will unconditionally evaluate to false (without error), leading to unexpected behaviour.

Please change this to "=" for portable behaviour across POSIX Bourne shells. Patch attached.

Attachments

configure.patch Download (294 bytes) - added by anon 16 months ago.

Change History

Changed 16 months ago by anon

Changed 16 months ago by peter

  • status changed from new to closed
  • resolution set to fixed

Applied in revision 2373, thanks!

Note: See TracTickets for help on using tickets.