Skip to content

Fix Spatial demo dependency resolution for current Maven hosts - #186

Open
happyherp wants to merge 2 commits into
hibernate:mainfrom
happyherp:fix/spatial-maven-repos
Open

Fix Spatial demo dependency resolution for current Maven hosts#186
happyherp wants to merge 2 commits into
hibernate:mainfrom
happyherp:fix/spatial-maven-repos

Conversation

@happyherp

Copy link
Copy Markdown

Summary

  • Replace dead Maven repositories (Boundless, old OSGeo webdav, hibernatespatial) with https://repo.osgeo.org/repository/release/ and Maven Central
  • Bump org.opengeo:geodb from 0.8 (no longer published) to 0.9, and hatbox to 1.0.b10 so the Spatial module resolves and compiles with a normal ./gradlew build

Test plan

  • With JDK 8: cd hibernate-orm/core && ./gradlew :Spatial:clean :Spatial:compileJava --refresh-dependencies
  • ./gradlew test (all modules) succeeds
  • CI / reviewer verify Spatial module still builds without local Maven installs

The Boundless and old OSGeo webdav repositories are gone, and geodb 0.8
is no longer published. Point Spatial at OSGeo's release repo and use
geodb 0.9 with hatbox 1.0.b10 so Gradle can download deps cleanly.

@yrodiere yrodiere left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, but judging from this comment in the Hibernate ORM code, the geodb is neither desirable nor necessary in recent versions of Hibernate ORM:

https://github.com/hibernate/hibernate-orm/blob/24d4c956f7d10883c182700dfd75e562f6e7b67e/hibernate-spatial/src/main/java/org/hibernate/spatial/dialect/h2gis/H2GISWkb.java#L37-L38

I also can't vouch for GeoDB, as I have never used it, so I can't encourage anyone to run this demo.

If we're going to change anything here, I'd rather upgrade to a recent version of Hibernate ORM, making GeoDB unnecessary?

@beikov

beikov commented Jul 15, 2026

Copy link
Copy Markdown
Member

I agree. Either remove the demo or upgrade Hibernate ORM version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants