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