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.

Sign up for GitHub

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

Jump to bottom

Upgrade to JTS ≥1.17.0 #2

Closed
stokwisz opened this issue Nov 24, 2021 · 3 comments
Closed

Upgrade to JTS ≥1.17.0 #2

stokwisz opened this issue Nov 24, 2021 · 3 comments

Comments

@stokwisz
Copy link

stokwisz commented Nov 24, 2021 •
edited
Loading

With release of JTS 1.17.0 binary incompatible change was introduced to some method of Polygon:

Change Polygon getExteriorRing and getInteriorRingN accessors to return LinearRing.
This is a binary incompatible change to the method signature. Recompilation is necessary. No source code changes are required.

This issue should be addressed by PR's upgrading dependency to JTS to have it at least in v1.17.0.
This was referenced Nov 24, 2021
Closed
Closed
@stokwisz
Copy link
Author

Hi @clementgayvallet, would it be possible to look this issue regarding updating JTS dependency? Is there something that I can do on my own to speed up a bit of the review process? Thanks in advance for your input here. ;-)

@clementgayvallet
Copy link
Contributor

Hi @stokwisz, I upgraded JTS to its latest version. Let me know if it fixes your issue. :)

@stokwisz
Copy link
Author

stokwisz commented Dec 1, 2022

Sorry for delayed answer – everything looks good. Thank you for your effort.

Please decide on your own what to do with open PRs:

@Joxit Joxit closed this as completed Dec 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Joxit @clementgayvallet @stokwisz