Opened 9 years ago

Closed 7 years ago

#3290 closed patch (wontfix)

Link liblwgeom

Reported by: Bas Couwenberg Owned by: robe
Priority: medium Milestone: PostGIS 2.4.0
Component: postgis Version: master
Keywords: Cc:

Description

The attached patch is still included in the Debian package for postgis 2.2.0RC1.

Please consider including these changes in postgis itself.

Split out from #3288.

Attachments (1)

02-link-liblwgeom.patch (2.0 KB ) - added by Bas Couwenberg 9 years ago.

Download all attachments as: .zip

Change History (7)

by Bas Couwenberg, 9 years ago

Attachment: 02-link-liblwgeom.patch added

comment:1 by robe, 9 years ago

Milestone: PostGIS 2.2.0PostGIS 2.3.0

Once we have a configure option to opt in/out of this, we'll include it. We may consider having it in 2.2.1 (but for now I'll push to 2.3.0) to get out of our 2.2.0 list.

If you want to revise your patch to include a configure option (preferably defaulting to old static linking - at lease for 2.2.1) , that would be great. If not I'll try to develop one after 2.2.0 release.

comment:2 by robe, 9 years ago

Owner: changed from pramsey to robe

comment:3 by strk, 9 years ago

It would be better to add version into the liblwgeom symbols before linking dynamically, see #3328

comment:4 by robe, 8 years ago

Milestone: PostGIS 2.3.0PostGIS 2.4.0

Do we still want to do this? Anyway too late for 2.3.

comment:5 by Bas Couwenberg, 8 years ago

I think you should, although for other projects using librttopo instead of liblwgeom is a much better choice.

comment:6 by robe, 7 years ago

Resolution: wontfix
Status: newclosed

I think this is no longer necessary and not a desired thing. So dismissing as a won't fix. Feel free to reopen if you feel otherwise.

Note: See TracTickets for help on using tickets.