Modify

Opened 5 years ago

Closed 4 years ago

Last modified 4 years ago

#20669 closed enhancement (fixed)

[PATCH] Please upgrade the GeoTools plugin to 25.0 and the JTS plugin to 1.18.1

Reported by: anonymous Owned by: Don-vip
Priority: major Milestone:
Component: Plugin geotools Version:
Keywords: upgrade Cc:

Description

The GeoTools plugin could be upgraded to 25.0.
The JTS plugin could be upgraded to 1.18.1.
Make sure the ImportImagePlugin plugin is upgraded to use GeoTools version 25.0

Attachments (2)

20669.patch (17.4 KB ) - added by taylor.smock 4 years ago.
20669.2.patch (26.4 KB ) - added by taylor.smock 4 years ago.
Additional excludes for ivy

Download all attachments as: .zip

Change History (8)

comment:1 by Don-vip, 5 years ago

Status: newassigned

comment:2 by taylor.smock, 4 years ago

Summary: Please upgrade the GeoTools plugin to 25.0 and the JTS plugin to 1.18.1[WIP PATCH] Please upgrade the GeoTools plugin to 25.0 and the JTS plugin to 1.18.1

The attached patch does the following:

  • Convert geotools to ivy
    • ivy ignores the following packages:
      • org=javax.xml.bind with modules jaxb-api, jaxb-runtime which come from the jaxb plugin
      • org=com.github.spotbugs
      • org=org.geotools.demo
      • org=org.geotools with module gt-imagemosaic (was not added in 22.0 version)
      • org=org.locationtech.jts module=jts-core (jts plugin)
      • org=org.locationtech.jts.io module=jts-io-common (jts plugin)
      • org=org.ejml (ejml plugin)
      • Various items that are in the apache-commons plugin
  • Deletes the lib directory

The patch is currently WIP since I haven't (yet) cleaned out everything that is unnecessary. The old geotools jar was 14M, and the new one is 25M.

by taylor.smock, 4 years ago

Attachment: 20669.patch added

by taylor.smock, 4 years ago

Attachment: 20669.2.patch added

Additional excludes for ivy

comment:3 by taylor.smock, 4 years ago

In attachment:20669.2.patch, the following packages were excluded:

  • Guava (was also excluded for 22.0)
  • Hatbox (was also excluded for 22.0)
  • imageio-ext-cog-reader (was not present in 22.0, excluded due to dependency on ehcache, 1.6M)
  • org.eclipse.emf (was not present in 22.0, excluded due to core being 1.3M)

Altogether, the changes reduced the jar file size from 26M to 18M (3M larger geotools at 22.0). I do need to load up the plugins that depend upon geotools, and see what happens, but I think I'll be able to move this from WIP PATCH to PATCH status.

comment:4 by taylor.smock, 4 years ago

Summary: [WIP PATCH] Please upgrade the GeoTools plugin to 25.0 and the JTS plugin to 1.18.1[PATCH] Please upgrade the GeoTools plugin to 25.0 and the JTS plugin to 1.18.1

comment:5 by Don-vip, 4 years ago

Resolution: fixed
Status: assignedclosed

In 35898/osm:

fix #20669 - upgrade the GeoTools plugin to 25.0 + convert to ivy (patch by taylor.smock)

comment:6 by Don-vip, 4 years ago

In 35900/osm:

see #20669 - dist

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Don-vip.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.