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

fix: upgrade gradle version 8.4 to 8.5 #2471

Merged

Conversation

miettal
Copy link
Contributor

@miettal miettal commented Dec 25, 2023

close #2458

gradle versions up to 8.4 has caching-problem and
In openms, build-clean-build steps fails by last build step.
https://github.com/girasolenergy/openems-fork/actions/runs/7206191185/job/19630540280

This changes fix the problem by updating gradle version 8.4 to 8.5

Test is here. it work.
https://github.com/girasolenergy/openems-fork/actions/runs/7282794004/job/19845714357

See also

Copy link

Code Coverage

@sfeilmeier
Copy link
Contributor

Thank you for digging into this! According to https://github.com/gradle/gradle/releases/tag/v8.5.0 the official way to upgrade gradle is via:

./gradlew wrapper --gradle-version=8.5

This command not only updates .gradle-wrapper/gradle-wrapper.properties but also the gradlew script. Could you please apply it on your fix?

Thanks & Regards,
Stefan

@miettal
Copy link
Contributor Author

miettal commented Dec 26, 2023

Thank you feedback!

Its very helpful information to me because I'm not familiar with java-related ecosystem.
I'll continue to work to complete PR.

gradle versions up to 8.4 has chacing-problem and
In openms, build-clean-build steps fails by last build.
https://github.com/girasolenergy/openems-fork/actions/runs/7206191185/job/19630540280

This changes fix the problem by updating gradle version 8.4 to 8.5

Test is here. it work.
https://github.com/girasolenergy/openems-fork/actions/runs/7282794004/job/19845714357

See also
 - OpenEMS#2458
 - gradle/gradle#20830

> ./gradlew wrapper --gradle-version=8.5
@miettal miettal force-pushed the bugfix/update-gradle8.4-to-8.5 branch from 535451e to 36d172d Compare December 27, 2023 01:36
Copy link

Code Coverage

@miettal
Copy link
Contributor Author

miettal commented Dec 27, 2023

@sfeilmeier I updated commit by following command. could you review it?

./gradlew wrapper --gradle-version=8.5

@sfeilmeier sfeilmeier merged commit 08af711 into OpenEMS:develop Dec 29, 2023
2 checks passed
@miettal miettal deleted the bugfix/update-gradle8.4-to-8.5 branch February 12, 2024 05:14
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.

gradle re-build fails
2 participants