Ticket #1191 (closed enhancement: fixed)

Opened 21 months ago

Last modified 20 months ago

move PREPARED_GEOM define in postgis_config.h

Reported by: strk Owned by: pramsey
Priority: medium Milestone: PostGIS 2.0.0
Component: postgis Version: trunk
Keywords: Cc:

Description

The PREPARED_GEOM define is a configuration directive, so it'd be best placed in postgis_config.h. Currently it is in lwgeom_geos_prepared.h and failing to include it from an implementation file results in disabling PREPARED geometries w/out further build errors. See ticket #1165

Change History

Changed 20 months ago by strk

  • status changed from new to closed
  • resolution set to fixed

The only reason for PREPARED_GEOM to be undefined is for builds againts geos 3.1 Since we _require_ 3.2 now, there's no point in making it a configuration. I've moved the define in the implementation file, where it is used, with r7879.

@robe: worth checking it didn't slowdown again (you never know!)

Changed 20 months ago by strk

r7880 makes it even build (thanks Hudson!)

Changed 20 months ago by robe

  • status changed from closed to reopened
  • resolution fixed deleted

Do you have debug code stuck somewhere? my regress is failing

Run tests: 74
Failed: 70

with stuff like this using r7880

--- loader/PointM-w.select.expected	Fri Sep  9 06:49:41 2011
+++ /tmp/pgis_reg_4740/test_2_out	Wed Sep 21 22:02:12 2011
@@ -1,3 +1,6 @@
+NOTICE:  [lwgeom_functions_basic.c:LWGEOM_asEWKT:2296] LWGEOM_asEWKT called.
+NOTICE:  [lwgeom_functions_basic.c:LWGEOM_asEWKT:2296] LWGEOM_asEWKT called.
+NOTICE:  [lwgeom_functions_basic.c:LWGEOM_asEWKT:2296] LWGEOM_asEWKT called.
 POINTM(0 1 3)
 POINTM(9 -1 -123)
 POINTM(9 -1 -3)

Changed 20 months ago by robe

  • status changed from reopened to closed
  • resolution set to fixed

disregard. For some reason I had enable-debug on my 9.1 configure. My 9.0 was fine. I suspect gremlins are changing my scripts since I don't recall changing my build script in past couple of days and certainly not adding in an enable-debug.

Though I thought we stripped notices from our regress tests results.

Changed 20 months ago by strk

It's most likely gremlins, see #1205

Note: See TracTickets for help on using tickets.