Package: release.debian.org
Severity: normal
Tags: bullseye
User: [email protected]
Usertags: pu
X-Debbugs-Cc: [email protected]
Control: affects -1 + src:postgis

[ Reason ]
As reported in #1031392, postgis 3.1.1 has an important issue with polar
stereographic projections which was resolved in 3.1.2.

[ Impact ]
Unusable coordinates from transformations.

[ Tests ]
Upstream test suite and manual tests have confirmed the fix.

[ Risks ]
Should be low, the patch is small and the fix has been around for quite a
while now.

[ Checklist ]
  [x] *all* changes are documented in the d/changelog
  [x] I reviewed all changes and I approve them
  [x] attach debdiff against the package in (old)stable
  [x] the issue is verified as fixed in unstable

[ Changes ]
The branch is updated to keep debcheckout and git-buildpackage working.

And the upstream patch included in 3.1.2 is cherry-picked to fix the
transformation bug,

[ Other info ]
N/A

Kind Regards,

Bas
diff -Nru postgis-3.1.1+dfsg/debian/changelog 
postgis-3.1.1+dfsg/debian/changelog
--- postgis-3.1.1+dfsg/debian/changelog 2021-01-29 06:27:33.000000000 +0100
+++ postgis-3.1.1+dfsg/debian/changelog 2023-02-16 19:00:56.000000000 +0100
@@ -1,3 +1,11 @@
+postgis (3.1.1+dfsg-1+deb11u1) bullseye; urgency=medium
+
+  * Update branch in gbp.conf & Vcs-Git URL.
+  * Add upstream patch to fix wrong Polar stereographic axis order.
+    (closes: #1031392)
+
+ -- Bas Couwenberg <[email protected]>  Thu, 16 Feb 2023 19:00:56 +0100
+
 postgis (3.1.1+dfsg-1) unstable; urgency=medium
 
   [ Christoph Berg ]
diff -Nru postgis-3.1.1+dfsg/debian/control postgis-3.1.1+dfsg/debian/control
--- postgis-3.1.1+dfsg/debian/control   2021-01-29 06:27:33.000000000 +0100
+++ postgis-3.1.1+dfsg/debian/control   2023-02-16 19:00:38.000000000 +0100
@@ -36,7 +36,7 @@
                xsltproc
 Standards-Version: 4.5.1
 Vcs-Browser: https://salsa.debian.org/debian-gis-team/postgis
-Vcs-Git: https://salsa.debian.org/debian-gis-team/postgis.git
+Vcs-Git: https://salsa.debian.org/debian-gis-team/postgis.git -b bullseye
 Homepage: http://postgis.net/
 
 Package: postgis
diff -Nru postgis-3.1.1+dfsg/debian/control.in 
postgis-3.1.1+dfsg/debian/control.in
--- postgis-3.1.1+dfsg/debian/control.in        2021-01-03 13:46:02.000000000 
+0100
+++ postgis-3.1.1+dfsg/debian/control.in        2023-02-16 19:00:49.000000000 
+0100
@@ -36,7 +36,7 @@
                xsltproc
 Standards-Version: 4.5.1
 Vcs-Browser: https://salsa.debian.org/debian-gis-team/postgis
-Vcs-Git: https://salsa.debian.org/debian-gis-team/postgis.git
+Vcs-Git: https://salsa.debian.org/debian-gis-team/postgis.git -b bullseye
 Homepage: http://postgis.net/
 
 Package: postgis
diff -Nru postgis-3.1.1+dfsg/debian/gbp.conf postgis-3.1.1+dfsg/debian/gbp.conf
--- postgis-3.1.1+dfsg/debian/gbp.conf  2020-12-19 06:46:58.000000000 +0100
+++ postgis-3.1.1+dfsg/debian/gbp.conf  2023-02-16 19:00:27.000000000 +0100
@@ -6,7 +6,7 @@
 
 # The default name for the Debian branch is "master".
 # Change it if the name is different (for instance, "debian/unstable").
-debian-branch = master
+debian-branch = bullseye
 
 # git-import-orig uses the following names for the upstream tags.
 # Change the value if you are not using git-import-orig
diff -Nru 
postgis-3.1.1+dfsg/debian/patches/0001-Only-do-axis-flips-on-CRS-that-have-a-Lat-as-the-fir.patch
 
postgis-3.1.1+dfsg/debian/patches/0001-Only-do-axis-flips-on-CRS-that-have-a-Lat-as-the-fir.patch
--- 
postgis-3.1.1+dfsg/debian/patches/0001-Only-do-axis-flips-on-CRS-that-have-a-Lat-as-the-fir.patch
   1970-01-01 01:00:00.000000000 +0100
+++ 
postgis-3.1.1+dfsg/debian/patches/0001-Only-do-axis-flips-on-CRS-that-have-a-Lat-as-the-fir.patch
   2023-02-16 19:00:56.000000000 +0100
@@ -0,0 +1,58 @@
+Description: Only do axis flips on CRS that have a "Lat" as the first column.
+ Otherwise just take the EPSG database order. References #4842, #4748
+Author: Paul Ramsey <[email protected]>
+Origin: 
https://trac.osgeo.org/postgis/changeset/4f6d81f8843914666b00683dcc93b81f6c13635d/git
+Bug: https://trac.osgeo.org/postgis/ticket/4842
+Bug-Debian: https://bugs.debian.org/1031392
+
+--- a/liblwgeom/lwgeom_transform.c
++++ b/liblwgeom/lwgeom_transform.c
+@@ -245,7 +245,7 @@ proj_crs_is_swapped(const PJ *pj_crs)
+               const char *out_name, *out_abbrev, *out_direction;
+               double out_unit_conv_factor;
+               const char *out_unit_name, *out_unit_auth_name, *out_unit_code;
+-              /* Read only first axis, see if it is degrees / north */
++              /* Read only first axis */
+               proj_cs_get_axis_info(NULL,
+                                     pj_cs,
+                                     0,
+@@ -256,13 +256,10 @@ proj_crs_is_swapped(const PJ *pj_crs)
+                                     &out_unit_name,
+                                     &out_unit_auth_name,
+                                     &out_unit_code);
+-              if (strcasecmp(out_abbrev, "E") == 0)
+-                      rv = LW_FALSE;
+-              else if (strcasecmp(out_abbrev, "Lat") == 0)
+-                      rv = LW_TRUE;
+-              else if (strcasecmp(out_direction, "north") == 0)
+-                      rv = LW_TRUE;
+-              else if (strcasecmp(out_direction, "south") == 0)
++
++              /* Only swap Lat/Lon systems */
++              /* Use whatever ordering planar systems default to */
++              if (strcasecmp(out_abbrev, "Lat") == 0)
+                       rv = LW_TRUE;
+               else
+                       rv = LW_FALSE;
+--- a/regress/core/tickets.sql
++++ b/regress/core/tickets.sql
+@@ -1298,7 +1298,9 @@ SELECT '#4670-5', ST_AsEWKT(ST_AddPoint(
+ 
+ SELECT '#4689', _ST_DistanceTree('POLYGON ((30 10, 40 40, 20 40, 30 
10))'::geography, 'POLYGON((81 6,140 35,-70 18,-51 0,-60 -46,106 -6,81 6))');
+ 
++-- Polar Stereographic Axis order issues
+ SELECT '#4748', ST_AsEWKT(ST_Transform(ST_SetSRID(ST_Point(-36.75, -54.25), 
4326), 3031),1);
++SELECT '#4842', ST_AsEWKT(ST_Transform(ST_SetSRID(ST_Point(12.572, 66.081), 
4326), 3413),1);
+ 
+ SELECT '#4718',
+       round(degrees(
+--- a/regress/core/tickets_expected
++++ b/regress/core/tickets_expected
+@@ -435,6 +435,7 @@ ERROR:  BOX2D_construct: args can not be
+ ERROR:  LWGEOM_addpoint: Invalid offset
+ #4689|0
+ #4748|SRID=3031;POINT(-2399498.7 3213318.5)
++#4842|SRID=3413;POINT(2218086.2 -1409161.3)
+ #4718|270.000|90.000
+ #4727|0
+ #4796|POLYGON((178632 397744,178738 397769,178745 397741,178749 397742,178752 
397727,178748 397726,178759 397684,178699 397669,178694 397691,178646 
397680,178632 397744))
diff -Nru postgis-3.1.1+dfsg/debian/patches/series 
postgis-3.1.1+dfsg/debian/patches/series
--- postgis-3.1.1+dfsg/debian/patches/series    2020-12-19 06:44:59.000000000 
+0100
+++ postgis-3.1.1+dfsg/debian/patches/series    2023-02-16 19:00:56.000000000 
+0100
@@ -1,3 +1,4 @@
 relax-test-timing-constraints.patch
 chaikin
 incorrect-path-for-interpreter.patch
+0001-Only-do-axis-flips-on-CRS-that-have-a-Lat-as-the-fir.patch

Reply via email to