Skip to content

Downgrade Gradle 9.0 to 8.12 for gretty compatibility#311

Merged
manolo merged 1 commit intov25from
fix/vaadin-dev-jetty
Feb 16, 2026
Merged

Downgrade Gradle 9.0 to 8.12 for gretty compatibility#311
manolo merged 1 commit intov25from
fix/vaadin-dev-jetty

Conversation

@manolo
Copy link
Copy Markdown
Member

@manolo manolo commented Feb 16, 2026

Summary

  • Gretty 4.1.10 uses the archivePath property which was removed in Gradle 9
  • This causes jettyStartWar to fail with: Could not get unknown property 'archivePath' for task ':war'
  • Neither Gretty 4.1.10 nor 5.0.1 support Gradle 9
  • Downgrade to Gradle 8.12 until gretty adds Gradle 9 support

Test plan

  • Run ./gradlew jettyStart (dev mode) — verify it starts correctly
  • Run ./gradlew clean build && ./gradlew jettyStartWar (prod mode) — verify it starts correctly

Gretty 4.1.10 uses the archivePath property which was removed in
Gradle 9, causing jettyStartWar to fail with:
"Could not get unknown property 'archivePath' for task ':war'"

Gretty does not yet support Gradle 9 (neither 4.1.10 nor 5.0.1),
so downgrade to Gradle 8.12 until gretty adds Gradle 9 support.
manolo added a commit to vaadin/platform-in-test-script that referenced this pull request Feb 16, 2026
Gretty uses archivePath which was removed in Gradle 9. Neither
gretty 4.1.10 nor 5.0.1 support Gradle 9. Downgrade wrapper to 8.12.
See vaadin/base-starter-gradle#311.
@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 16, 2026

Test Results

0 tests   - 5   0 ✅  - 5   0s ⏱️ -6s
0 suites  - 1   0 💤 ±0 
0 files    - 1   0 ❌ ±0 

Results for commit 3f7b240. ± Comparison against base commit 517c95f.

♻️ This comment has been updated with latest results.

@manolo manolo merged commit 992671f into v25 Feb 16, 2026
2 of 8 checks passed
@manolo manolo deleted the fix/vaadin-dev-jetty branch February 16, 2026 13:35
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.

1 participant