Skip to content

Conversation

@groldan
Copy link
Member

@groldan groldan commented Jun 9, 2025

Integrate the GeoTools Bill of Materials (gt-bom) into GeoWebCache, centralizing management of GeoTools dependency versions.

  • Import org.geotools:gt-bom in the root pom.xml's <dependencyManagement> section. This makes GeoWebCache a direct consumer of GeoTools' curated dependency set.
  • Remove explicit <version>${gt.version}</version> declarations for individual GeoTools artifacts from all module POMs. Versions are now inherited from the imported gt-bom.
  • Remove unused dependency org.freemarker:freemarker
  • Delegate the following dependency versions to the GeoTools BOM:
    • apache commons (io, logging, collections, text, dbcp)
    • guava
    • h2
    • hsqldb
    • httpclient
    • imageio-ext
    • jackson-annotations
    • jackson-core
    • jackson-databind
    • jai_core/codec
    • jai_imageio
    • jaiext
    • jts
    • log4j-1.2-api
    • log4j-api
    • log4j-core
    • log4j-jcl
    • log4j-jul
    • log4j-slf4j2-impl
    • ojdbc14
    • postgresql
    • slf4j-api

@aaime
Copy link
Member

aaime commented Jun 9, 2025

Thinking out loud here... what about the cloud connection libraries? azure/s3/google
They are also spread out across different projects (imageio-ext, gwc, possibly geoserver) and could use a BOM.

@groldan
Copy link
Member Author

groldan commented Jun 10, 2025

Good point but I'd like to close this one as scoped for the geotools bom and see about that separately if you don't mind, it's taken too long already. The cog libraries are actually more related to sharing them across imageio-ext and geowebcache and geotools itself, so not sure where to tackle it

@groldan
Copy link
Member Author

groldan commented Jun 10, 2025

Aside, it looks like we'll be adding PMTiles support as a datastore, like in the mbtiles store community module, and that'll require file/http/s3/azure range readers, so I'd like to leverage some synergies there too

…ency versions

This commit integrates the GeoTools Bill of Materials (gt-bom) into
GeoWebCache, centralizing management of GeoTools dependency versions.

* Import `org.geotools:gt-bom` in the root `pom.xml`'s
  `<dependencyManagement>` section. This makes GeoWebCache a direct
  consumer of GeoTools' curated dependency set.
* Remove explicit `<version>${gt.version}</version>` declarations
  for individual GeoTools artifacts from all module POMs. Versions
  are now inherited from the imported `gt-bom`.
* Remove unused dependency org.freemarker:freemarker
* Delegate the following dependency versions to the GeoTools BOM:
  * apache commons (io, logging, collections, text, dbcp)
  * guava
  * h2
  * hsqldb
  * httpclient
  * imageio-ext
  * jackson-annotations
  * jackson-core
  * jackson-databind
  * jai_core/codec
  * jai_imageio
  * jaiext
  * jts
  * log4j-1.2-api
  * log4j-api
  * log4j-core
  * log4j-jcl
  * log4j-jul
  * log4j-slf4j2-impl
  * ojdbc14
  * postgresql
  * slf4j-api
@groldan groldan force-pushed the geoserver3/geotools-bom branch from 9188560 to 672cdfa Compare June 16, 2025 12:33
@groldan
Copy link
Member Author

groldan commented Jun 16, 2025

I would like to merge this PR and tackle the concern raised in this comment as a separate issue, for which I've created this ticket: https://osgeo-org.atlassian.net/browse/GEOT-7777

@groldan
Copy link
Member Author

groldan commented Jun 16, 2025

Downstream QA Build PR ready: geotools/geotools#5246

Copy link
Contributor

@jodygarnett jodygarnett left a comment

Choose a reason for hiding this comment

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

Thanks!

@jodygarnett jodygarnett merged commit 93bfa0d into GeoWebCache:main Jun 16, 2025
11 checks passed
@groldan groldan deleted the geoserver3/geotools-bom branch June 16, 2025 20:31
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