Opened 11 years ago

Closed 11 years ago

#2368 closed task (fixed)

Test failure for regress/topogeo_addlinestring (test_29)

Reported by: Mike Taves Owned by: strk
Priority: medium Milestone: PostGIS 2.1.0
Component: topology Version: 2.1.x
Keywords: Cc:

Description

Testing PostGIS 2.1.0beta3 on a 32-bit Ubuntu 13.04 (raring) yields one test failure on regress/topogeo_addlinestring (test_29).

The build of PostGIS uses Ubuntu repository packages for all components, specifically GEOS (3.3.3) and GDAL (1.9.0), i.e.:

sudo apt-get install build-essential postgresql-9.1 postgresql-server-dev-9.1 libxml2-dev libgeos-dev libgdal-dev libproj-dev libjson0-dev xsltproc docbook-xsl docbook-mathml

Attachments (4)

test_29_diff (716 bytes ) - added by Mike Taves 11 years ago.
test_29_out (2.9 KB ) - added by Mike Taves 11 years ago.
test_29_diff.2 (300 bytes ) - added by Mike Taves 11 years ago.
from r11583
test_29_out.2 (2.9 KB ) - added by Mike Taves 11 years ago.
from r11583

Download all attachments as: .zip

Change History (12)

by Mike Taves, 11 years ago

Attachment: test_29_diff added

by Mike Taves, 11 years ago

Attachment: test_29_out added

comment:1 by robe, 11 years ago

Type: defecttask

Yah this is because strk decided to apply a regression test that he patched up in not yet releases geos 3.3.9.

strk — givien that the ubuntu folks are many micro versions behind in their patching), I think we've got to disable this test if you are running < geos 3.3.9

comment:2 by robe, 11 years ago

Component: postgistopology
Owner: changed from pramsey to strk

comment:3 by strk, 11 years ago

Indeed the change in postgis should probably just be reverted. Once we change, we should change to succeed in both cases.

comment:4 by strk, 11 years ago

Please try with r11583 — that revision uses a different expected output depending on GEOS version. It needs a new run of ./autogen.sh. If you're happy with this I'll backport to 2.1

comment:5 by Mike Taves, 11 years ago

I have a different fail on test_29 with r11583 (also disabling LIBGDAL_DEPLIBS_LDFLAGS, which is a different un-ticketed issue).

by Mike Taves, 11 years ago

Attachment: test_29_diff.2 added

from r11583

by Mike Taves, 11 years ago

Attachment: test_29_out.2 added

from r11583

comment:6 by strk, 11 years ago

Please try r11584, should be better

comment:7 by Mike Taves, 11 years ago

All good in r11584 with this setup.

comment:8 by strk, 11 years ago

Resolution: fixed
Status: newclosed

backported to 2.1 branch with r11585

Note: See TracTickets for help on using tickets.