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

Greg Troxel gdt at lexort.com
Wed Feb 19 06:09:01 PST 2020


Sandro Santilli <strk at kbt.io> writes:

> Escaping can also be done in POSIX shell, not a big deal.
> Let's just understand what we really need. For instance,
> how do we want the output of `geos-config --libs` look like,
> when there's a space in the path ?
>
>       0. -L/usr/local silly/lib -lgeos-3.9.0
>       1. -L/usr/local\ silly/lib -lgeos-3.9.0
>       2. -L"/usr/local silly/lib" -lgeos-3.9.0
>       3. -L'/usr/local silly/lib' -lgeos-3.9.0
>
> We got 0 before the change...
>
> With c263a0f7 I went with option 1, and restored proper output in
> presence of an exec_prefix.

I think 1 is reasonable.   I am inclined to say: let's call it a day and
wait for an actual bug report from an actual user (not a contrived test
case) to address any further nonstandard characters in prefixes.   This
really feels like a non-problem to me, at least once spaces are handled.


More information about the geos-devel mailing list