Ticket #1221 (closed task: fixed)

Opened 8 months ago

Last modified 5 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 5 months ago.

Change History

Changed 8 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 5 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 5 months ago by strk

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

Changed 5 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 5 months ago by strk

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

Changed 5 months ago by delawen

Changed 5 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 5 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 5 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.