[geos-devel] [GEOS] #1015: Update geos-config tool for consistency

[GEOS]

#1015: Update geos-config tool for consistency
-----------------------------------+--------------------------
 Reporter:  robe                   |      Owner:  geos-devel@…
     Type:  defect                 |     Status:  new
 Priority:  minor                  |  Milestone:  3.9.0
Component:  Build/Install (cmake)  |    Version:  master
 Severity:  Unassigned             |   Keywords:
-----------------------------------+--------------------------
 Details repeated from https://git.osgeo.org/gitea/geos/geos/pulls/99


 ----
 **Specify bash, and use printf to escape paths (if needed)**

 This allows installation with CMake to other directories, such as `/opt/my
 prefix`, since `geos-config --prefix` would return `/opt/my\ prefix`. Also
 `printf` is a bash-only feature.

 **Restore bash variables, e.g. `${prefix}`, and use CMake's `@ONLY` option
 **
 Restore to prior syntax in b15fd1171823d16195bee17f2c7b44778603258d using
 `${prefix}` rather than substituting `@CMAKE_INSTALL_PREFIX@`. Re-use
 variables throughout script for convenience.

 Note that the current autotools setup does not fully install to prefixes
 that need escaping, but the CMake setup can.

 **Consistent indents and style between .in and .cmake versions**

 Use 2-space indents, re-order case statements to same order as usage. The
 Autotools and CMake versions are nearly the same, with the exception of
 Autotool's `libdir`, which perhaps could be something other than
 `$prefix/lib`?

 **Remove `exec_prefix`, which has never been used**

 Checking the history of this file, it does not seem to be used.

 **Disable configure and install for MSVC builds**

 The script requires bash, which is not used for MSVC. Scheme to install a
 MSVC version of `geos-config.bat` is drafted at [trac#1014](#1014).

 ----

-- 
Ticket URL: <https://trac.osgeo.org/geos/ticket/1015>
GEOS <http://trac.osgeo.org/geos>
GEOS (Geometry Engine - Open Source) is a C++ port of the Java Topology Suite (JTS).