Opened 9 years ago

Closed 9 years ago

#3288 closed patch (duplicate)

Various patches from the postgis Debian package

Reported by: sebastic Owned by: pramsey
Priority: medium Milestone: PostGIS 2.2.0
Component: postgis Version: master
Keywords: Cc:

Description

The attached patches are still included in the Debian package for postgis 2.2.0RC1.

Please consider including these changes in postgis itself.

  • 01-fix-spelling.patch

Minor spelling fixes:

recieved → received
Portugese → Portuguese
Potugese → Portuguese
splitted → split

Author: Markus Wanner <markus@…>, Bas Couwenberg <sebastic@…>

  • 02-link-liblwgeom.patch

Instead of statically pulling liblwgeom into the postgis.so, dynamically link to it.

Author: Stephen Frost <sfrost@…>, Markus Wanner <markus@…>

  • 03-honor-build-flags.patch

Better honor build flags during the build process.

These changes are required to build postgis with hardening buildflags

Author: Markus Wanner <markus@…>

  • 04-fix-manpage.patch

Remove what looks like an UTF-8 marker or some such.

This used to be a problem caused by editors on Windows, I haven't seen it caused by those in a while.

Author: Markus Wanner <markus@…>

  • 05-de-translation.patch

Add partial German translation for documentation.

Author: Chris Leick <c.leick@…>

  • 06-add-lm-for-liblwgeom.patch

Add -lm for liblwgeom to support building on ppc64el

Author: Paulo Flabiano Smorigo <pfsmorigo@…>

  • 07-fix-hurd-i386-ftbfs.patch

Detect GNU/Hurd and treat it like Linux just as BSD.

Author: Bas Couwenberg <sebastic@…>

Attachments (7)

01-fix-spelling.patch (10.0 KB ) - added by sebastic 9 years ago.
02-link-liblwgeom.patch (2.0 KB ) - added by sebastic 9 years ago.
03-honor-build-flags.patch (2.5 KB ) - added by sebastic 9 years ago.
04-fix-manpage.patch (385 bytes ) - added by sebastic 9 years ago.
05-de-translation.patch (56.6 KB ) - added by sebastic 9 years ago.
06-add-lm-for-liblwgeom.patch (576 bytes ) - added by sebastic 9 years ago.
07-fix-hurd-i386-ftbfs.patch (506 bytes ) - added by sebastic 9 years ago.

Download all attachments as: .zip

Change History (11)

by sebastic, 9 years ago

Attachment: 01-fix-spelling.patch added

by sebastic, 9 years ago

Attachment: 02-link-liblwgeom.patch added

by sebastic, 9 years ago

Attachment: 03-honor-build-flags.patch added

by sebastic, 9 years ago

Attachment: 04-fix-manpage.patch added

by sebastic, 9 years ago

Attachment: 05-de-translation.patch added

by sebastic, 9 years ago

by sebastic, 9 years ago

comment:1 by strk, 9 years ago

I'd suggest to use a separate ticket for each issue as each one can have a discussion on its own. For example, for 07, I don't see LINUX macro being used anywhere, so why defining it ? Or for 01-fix-spelling: the ChangeLog is generated. I'm happy you're dynamically linking liblwgeom in debian, but I suspect there would be a discussion before accepting that to be a default for builds.

comment:2 by strk, 9 years ago

Note I've committed the spelling fixes (all but the one in shapelib) with r14113

comment:3 by sebastic, 9 years ago

This issue can be closed, please see the individual issues:

  • #3289 01-fix-spelling.patch
  • #3290 02-link-liblwgeom.patch
  • #3291 03-honor-build-flags.patch
  • #3292 04-fix-manpage.patch
  • #3293 05-de-translation.patch
  • #3294 06-add-lm-for-liblwgeom.patch
  • #3295 07-fix-hurd-i386-ftbfs.patch

comment:4 by sebastic, 9 years ago

Resolution: duplicate
Status: newclosed
Note: See TracTickets for help on using tickets.