[GRASS-dev] [GRASS GIS] #3421: GEOS C API usage

#3421: GEOS C API usage
----------------------------+-------------------------
Reporter: Bas Couwenberg | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 7.2.3
Component: Default | Version: 7.2.2
Keywords: | CPU: Unspecified
Platform: Linux |
----------------------------+-------------------------
GRASS is claimed to be using the GEOS C API instead of the unstable C++
API according to:

  https://trac.osgeo.org/geos/wiki/Applications

`configure.in` checks for `geos_c.h`, but uses `geos-config --libs`
instead of `geos-config --clibs` causing linking to `libgeos-<VERSION>.so`
in addition to `libgeos_c.so`.

The attached patch updates the `geos-config` command to use `--clibs`.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3421&gt;
GRASS GIS <https://grass.osgeo.org>

#3421: GEOS C API usage
-----------------------------+-------------------------
  Reporter: Bas Couwenberg | Owner: grass-dev@…
      Type: defect | Status: new
  Priority: normal | Milestone: 7.2.3
Component: Default | Version: 7.2.2
Resolution: | Keywords:
       CPU: Unspecified | Platform: Linux
-----------------------------+-------------------------
Changes (by Bas Couwenberg):

* Attachment "geos-c.patch" added.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3421&gt;
GRASS GIS <https://grass.osgeo.org>

#3421: GEOS C API usage
-----------------------------+-------------------------
  Reporter: Bas Couwenberg | Owner: grass-dev@…
      Type: defect | Status: new
  Priority: normal | Milestone: 7.2.3
Component: Default | Version: 7.2.2
Resolution: | Keywords:
       CPU: Unspecified | Platform: Linux
-----------------------------+-------------------------

Comment (by mmetz):

Replying to [ticket:3421 Bas Couwenberg]:
> GRASS is claimed to be using the GEOS C API instead of the unstable C++
API according to:
>
> https://trac.osgeo.org/geos/wiki/Applications
>
> `configure.in` checks for `geos_c.h`, but uses `geos-config --libs`
instead of `geos-config --clibs` causing linking to `libgeos-<VERSION>.so`
in addition to `libgeos_c.so`.
>
> The attached patch updates the `geos-config` command to use `--clibs`.

Thanks for the patch! Applied to trunk and relbr72 in r71544,5.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3421#comment:1&gt;
GRASS GIS <https://grass.osgeo.org>

#3421: GEOS C API usage
-----------------------------+-------------------------
  Reporter: Bas Couwenberg | Owner: grass-dev@…
      Type: defect | Status: new
  Priority: normal | Milestone: 7.2.3
Component: Default | Version: 7.2.2
Resolution: | Keywords:
       CPU: Unspecified | Platform: Linux
-----------------------------+-------------------------

Comment (by mmetz):

Replying to [comment:1 mmetz]:
> Replying to [ticket:3421 Bas Couwenberg]:
> > GRASS is claimed to be using the GEOS C API instead of the unstable
C++ API according to:
> >
> > https://trac.osgeo.org/geos/wiki/Applications
> >
> > `configure.in` checks for `geos_c.h`, but uses `geos-config --libs`
instead of `geos-config --clibs` causing linking to `libgeos-<VERSION>.so`
in addition to `libgeos_c.so`.
> >
> > The attached patch updates the `geos-config` command to use `--clibs`.
>
> Thanks for the patch! Applied to trunk and relbr72 in r71544,5.

I guess the patch provided by you is working for you and it has been
applied to trunk and relbr72. Closing ticket.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3421#comment:2&gt;
GRASS GIS <https://grass.osgeo.org>

#3421: GEOS C API usage
-----------------------------+-------------------------
  Reporter: Bas Couwenberg | Owner: grass-dev@…
      Type: defect | Status: closed
  Priority: normal | Milestone: 7.2.3
Component: Default | Version: 7.2.2
Resolution: fixed | Keywords:
       CPU: Unspecified | Platform: Linux
-----------------------------+-------------------------
Changes (by martinl):

* status: new => closed
* resolution: => fixed

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3421#comment:3&gt;
GRASS GIS <https://grass.osgeo.org>