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

Update gradle-build.yml / gradle-build-action #189

Merged
merged 5 commits into from Sep 6, 2023

Conversation

magro
Copy link
Contributor

@magro magro commented Sep 6, 2023

This should improve the issue with builds failing with an OOME, which might be related to gradle/gradle#23215, which could be solved with gradle 8.3 (used by the new gradle-build-action according to gradle/gradle-build-action@v2.7.1...v2.8.0).

Update after some builds and more changes: The final solution here is to set jvmArgs to "-Xmx2048m" for the test task.

This hopefully improves the issue with builds failing with an OOME, which might be related to gradle/gradle#23215, which could be solved with gradle 8.3 (used by the new gradle-build-action according to gradle/gradle-build-action@v2.7.1...v2.8.0).
@github-actions
Copy link

github-actions bot commented Sep 6, 2023

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ EDITORCONFIG editorconfig-checker 3 0 0.01s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security

Update to gradle 8.3
Prevent
```
Unable to make protected final java.lang.Class java.lang.ClassLoader.defineClass(java.lang.String,byte[],int,int,java.security.ProtectionDomain) throws java.lang.ClassFormatError accessible: module java.base does not "opens java.lang" to unnamed module @238e0d81
```
Remove args to set Xmx that seemingly don't work
Set Xmx via jvmArgs for test task
@magro magro merged commit 7eabc21 into main Sep 6, 2023
3 checks passed
@magro magro deleted the update-gradle-build-action branch September 6, 2023 22:36
magro added a commit that referenced this pull request Sep 20, 2023
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

1 participant