Integrated: Add missing inputs to tasks in gradle build

Jorn Vernee jvernee at openjdk.org
Fri Feb 9 22:44:02 UTC 2024


On Fri, 9 Feb 2024 14:59:53 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:

> I recently re-did some of the task dependencies of the gradle build: https://github.com/openjdk/jextract/pull/200
> 
> From my initial testing for that PR, and reading online, I was under the impression that tasks would re-run when the outputs of a dependency changes. After additional use & testing, this turns out to be not the case and we need to explicitly set task inputs any way. This PR does that. 
> 
> I also changed the `copyLibClang` task from `Copy` to `Sync`. If a file is removed from the source directory, `Copy` will not remove it from the output, but `Sync` will. This helps ensure that we don't package any stale files into the jextract jmod (which might create include conflicts). This is also something I noticed during more thorough testing.
> 
> Finally, I noticed that on Windows we were copying `clang.exe` to the jextract jmod archive, since we copy from the `bin` dir on Windows, which contains both the `libclang.dll` and the `clang.exe` file. I've added an exclude for `clang.exe`.

This pull request has now been integrated.

Changeset: 545db09f
Author:    Jorn Vernee <jvernee at openjdk.org>
URL:       https://git.openjdk.org/jextract/commit/545db09f3d1d2a7aadeea96494ba088e68e19e9f
Stats:     6 lines in 1 file changed: 5 ins; 0 del; 1 mod

Add missing inputs to tasks in gradle build

Reviewed-by: mcimadamore

-------------

PR: https://git.openjdk.org/jextract/pull/204


More information about the jextract-dev mailing list