Skip to content

GROOVY-11859: fix JarJarTask annotation for Windows compatibility#2380

Merged
paulk-asert merged 1 commit intoapache:masterfrom
jamesfredley:GROOVY-11859
Feb 23, 2026
Merged

GROOVY-11859: fix JarJarTask annotation for Windows compatibility#2380
paulk-asert merged 1 commit intoapache:masterfrom
jamesfredley:GROOVY-11859

Conversation

@jamesfredley
Copy link
Contributor

https://issues.apache.org/jira/browse/GROOVY-11859

JarJarTask.untouchedFiles is a List<String> of glob patterns (e.g. CliBuilder*), not file paths. The @InputFiles @Classpath annotations cause Gradle to resolve these patterns as literal file paths on Windows, failing the build. Changed to @Input since these are plain string values.

Copilot AI review requested due to automatic review settings February 15, 2026 23:34
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Fixes GROOVY-11859 by correcting Gradle task input annotations in JarJarTask so Windows builds don’t fail when untouchedFiles contains glob patterns rather than real file paths.

Changes:

  • Replaces @InputFiles @Classpath on untouchedFiles with @Input to treat the value as plain strings for up-to-date checking/caching.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 66.7172%. Comparing base (9b4ec31) to head (a33437f).

Additional details and impacted files

Impacted file tree graph

@@                Coverage Diff                 @@
##               master      #2380        +/-   ##
==================================================
+ Coverage     66.7154%   66.7172%   +0.0017%     
- Complexity      29846      29848         +2     
==================================================
  Files            1382       1382                
  Lines          116180     116180                
  Branches        20491      20491                
==================================================
+ Hits            77510      77512         +2     
  Misses          32323      32323                
+ Partials         6347       6345         -2     

see 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@paulk-asert paulk-asert merged commit f02b9bc into apache:master Feb 23, 2026
20 of 21 checks passed
@paulk-asert
Copy link
Contributor

Merged, thanks!

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.

4 participants