[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