Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade geotools to 21.2 and remove the geotools submodule #2676

Merged
merged 5 commits into from
Aug 19, 2019

Conversation

groldan
Copy link
Member

@groldan groldan commented Aug 15, 2019

Upgrade to GeoTools 21.2 (latest release)

  • Main geotools version upgraded from 19.4 (latest with JTS from vividsolutions) to the latest release. Changes mostly related to refactor com.vividsolutions.jts to org.locationtech.jts.
  • geowebcache-webapp: declares <gt.version>14.1</gt.version> to match the geotools version used by geowebcache 1.8.1 and avoid possible runtime clashes.
  • extractorapp: removes support for exporting to MIF/TAB formats, since they're not supported in GeoTools anymore.
  • mapfishapp: migration without issues

Remove GeoTools submodule:

The GeoTools submodule was stuck at version 9.2 for extractorapp to support exporting to MapInfo MIF file format. With the upgrade of extractorapp and the removal of support for MIF, the geotools submodule is no longer needed.

Upgrade extractorapp to the latest GeoTools version.

Removes support for exporting to MIF/TAB formats, since
they're not supported in GeoTools anymore.
The GeoTools submodule was stuck at version 9.2
for extractorapp to support exporting to MapInfo MIF
file format. With the upgrade of extractorapp and
the removal of support for MIF, the geotools submodule
is no longer needed.
@fvanderbiest
Copy link
Member

Thanks Gabriel. Will look into it asap.

These 2 lines should also be removed:

["mif", "Mif/Mid"],
["tab", "TAB"],

@fvanderbiest fvanderbiest merged commit 937d127 into georchestra:master Aug 19, 2019
fvanderbiest added a commit that referenced this pull request Aug 19, 2019
@fvanderbiest
Copy link
Member

These 2 lines should also be removed:

Done with f8692c9

@fvanderbiest fvanderbiest added this to the 19.06 milestone Aug 19, 2019
@groldan groldan deleted the chore/upgrade_geotools_21.2 branch August 23, 2019 15:52
@fvanderbiest fvanderbiest modified the milestones: 19.09, 19.12 Oct 22, 2019
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.

None yet

2 participants