Skip to content

BUILD-8928 Fix Windows Gradle build#73

Merged
matemoln merged 1 commit intomasterfrom
fix/mm/BUILD-8928
Sep 2, 2025
Merged

BUILD-8928 Fix Windows Gradle build#73
matemoln merged 1 commit intomasterfrom
fix/mm/BUILD-8928

Conversation

@matemoln
Copy link
Copy Markdown
Contributor

@matemoln matemoln commented Sep 2, 2025

BUILD-8928

  • add --console plain argument to set_project_version, the gradle command hangs without this
  • unset JAVA_HOME environment variable to fix an issue on GitHub hosted Windows runners, where JAVA_HOME is set by default and Gradle prioritizes this JDK instead of the JDK from the path

@matemoln matemoln requested a review from a team September 2, 2025 07:10
@matemoln matemoln changed the title BUILD-8928 Fix Windows Gradle build - add --console plain argument to set_project_version, the gradle command hangs without this - unset JAVA_HOME environment variable to fix an issue on GitHub hosted Windows runners, where JAVA_HOME is set by default and Gradle prioritizes this JDK instead of the JDK from the path BUILD-8928 Fix Windows Gradle build Sep 2, 2025
- add --console plain argument to set_project_version, the gradle command hangs without this
- unset JAVA_HOME environment variable to fix an issue on GitHub hosted Windows runners, where JAVA_HOME is set by default and Gradle prioritizes this JDK instead of the JDK from the path
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Sep 2, 2025

🤖 Pull Request summary

Improves Gradle build script reliability by adding daemon control and fixing Windows CI issues.

• Enhanced Gradle properties command with --no-daemon --console plain flags for more consistent output parsing
• Added unset JAVA_HOME in main function to resolve JDK selection issues on GitHub Windows runners
• Updated corresponding test mock to match new Gradle command signature
• Minor formatting cleanup (removed extra blank line)

Review focus: Verify the JAVA_HOME unset doesn't break local development environments where it may be intentionally set.

💬 Please send your feedback

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@matemoln matemoln merged commit 30fb307 into master Sep 2, 2025
15 of 18 checks passed
@matemoln matemoln deleted the fix/mm/BUILD-8928 branch September 2, 2025 10:16
matemoln added a commit that referenced this pull request Sep 3, 2025
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.

2 participants