Skip to content

Conversation

@sdedic
Copy link
Member

@sdedic sdedic commented Aug 6, 2025

During debugging of other bugs, a gradle project load failed on me (some unrelated error) for a top-level project that does not have an build.gradle but just settings.gradle. The code that consumes error reports from gradle process threw a NPE - this PR just adds a fallback to settings if buildscript is not present - and finally probably "null" string if even settings file does not exist.

@sdedic sdedic added this to the NB28 milestone Aug 6, 2025
@sdedic sdedic requested review from MartinBalin and lkishalmi August 6, 2025 14:01
@sdedic sdedic self-assigned this Aug 6, 2025
@sdedic sdedic added the Gradle [ci] enable "build tools" tests label Aug 6, 2025
Copy link
Contributor

@lkishalmi lkishalmi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@sdedic sdedic merged commit 078fb05 into apache:master Aug 7, 2025
33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Gradle [ci] enable "build tools" tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants