MovingBlocks / DestinationSol

Official continuation of Destination Sol, the great fun little arcade space shooter from http://sourceforge.net/projects/destinationsol/ Modules live at https://github.com/DestinationSol/

Apache License 2.0

325 stars 122 forks source link

build: update groovy wrapper for Gradle 7 #680

Closed BenjaminAmos closed 1 year ago

BenjaminAmos commented 1 year ago

Description

This pull request updates the groovy wrapper to a version compatible with Gradle 7. It includes the changes in MovingBlocks/groovy-wrapper#7.

Testing

If groovyw runs again, then it should work. You can try testing with groovyw module get formic.

Notes

The groovy wrapper is (and always has been) a binary file, so I'd only merge this if you trust it. If there's a way of making a reproducable build of the wrapper, then that would help for validation.

Cervator commented 1 year ago

I've now officially also made a formal new wrapper release - https://github.com/MovingBlocks/groovy-wrapper/releases/tag/v2.1.0 :-)

Can move forward either merging this or dropping in the new jar from there then fixing up #673 👍

Feel free to go for it or I'll circle around to this eventually, just out of time for tonight :-)

BenjaminAmos commented 1 year ago

I managed to reproduce the groovy-wrapper.jar contents from the release byte-for-byte, so I've changed to using that one. It's easier to replicate than the one I uploaded before. Thanks for the release! I'll merge this in now.

(The build failed for unrelated reasons.)