W-23475822 fix(build): declare nativeCompile output directory for up-to-date checks#124
Merged
Merged
Conversation
…to-date checks The GraalVM nativeCompile task did not declare its output directory, so Gradle could not reliably compute up-to-date status or resolve the native artifacts across projects. Declare outputs.dir on every nativeCompile task in subprojects so incremental builds and cross-project dependency resolution work correctly. Extracted from the native-bindings branch (feat/new-native-bindings) as an independent build-correctness fix. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
martincousido
approved these changes
Jul 16, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Extracted from #119 (feat/new-native-bindings) as an independent build-correctness fix.
Problem
The GraalVM
nativeCompiletask did not declare its output directory, so Gradle could not reliably compute up-to-date status or resolve the native artifacts across projects.Fix
Declare
outputs.dir("${project.buildDir}/native/nativeCompile")on everynativeCompiletask insubprojects(rootbuild.gradle), so incremental builds and cross-project dependency resolution work correctly.Notes
master; ~6 lines, no dependency on the new C/Go/Rust bindings in Native-bindings #119.