Ticket #1221 (closed task: fixed)

Opened 20 months ago

Last modified 17 months ago

Update Java components to support SRID<=0 as unknown SRID

Reported by: pramsey Owned by: delawen
Priority: high Milestone: PostGIS 2.0.0
Component: java Version: trunk
Keywords: Cc:

Description

After implementation of #286, the java subcomponents remain full of unsubstituted references to -1.

Attachments

jdbc.patch Download (41.6 KB) - added by delawen 17 months ago.

Change History

Changed 20 months ago by strk

  • summary changed from Update Java components to support SRID=0 as unknown SRID to Update Java components to support SRID<=0 as unknown SRID

As per  http://postgis.refractions.net/pipermail/postgis-devel/2011-October/015426.html UNKNOWN is now anything <= 0. Such check might be worth doing with a macro.

Changed 17 months ago by strk

  • owner changed from pramsey to strk
  • status changed from new to assigned

first windows, now java... it's a weird day for me today.

Changed 17 months ago by strk

Actually, are there any users of the java components ? There's no regression test I can see.

Changed 17 months ago by strk

So, r8481 attempts at a blind fix. I didn't even succeed in building anything though, in that "make" said it didn't know what to do.

If anyone knows how to build anything under that dir please provide a Makefile for the rest of the world ?

Changed 17 months ago by strk

  • owner changed from strk to delawen
  • status changed from assigned to new
  • component changed from postgis to Java bindings

Changed 17 months ago by delawen

Changed 17 months ago by delawen

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

This patch should be used on the jdbc folder:

$ cd jdbc $ patch -i .../jdbc.patch -p0

I also changed the way to build the jdbc from makefile to maven, which is more appropiate for java buildings (and cleaner and easier to use, although probably slower).

Changed 17 months ago by strk

  • status changed from closed to reopened
  • resolution fixed deleted

@delawen please don't close tickets till the patch is actually submitted. Will be closed with a comment containing a pointer to the actual revision adding the patch.

Changed 17 months ago by strk

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

@delawen: I committed your patch as r8710 but please next time keep logically separated changes in separate patches (this one contained: code fixes for SRID, indenting changes, new build system)

Thanks a lot, looking forward for more cleanups in the java comonents!

Note: See TracTickets for help on using tickets.