Skip to content

Conversation

@fibsifan
Copy link
Contributor

@fibsifan fibsifan commented Jul 2, 2025

Task dependency optimizations

  • The bootstrapPlatform task no longer depends on the downloadAndVerify tasks. Instead, the unpacking tasks depend on the downloadAndVerify tasks (if they're configured, as before).

Plugin Updates

  • SAP Commerce Cloud version updated to 2211.38
  • Gradle version updated to 8.14.2
  • SAP Commerce Gradle Plugin versions updated to 4.1.0
  • Download Plugin version not updated, because version 5.6.0 had issues with the SAP download portal (too many redirects)

@fibsifan fibsifan force-pushed the task-dependencies branch from d11e97b to 7705560 Compare July 2, 2025 07:53
@mpern
Copy link
Member

mpern commented Dec 3, 2025

Hi @fibsifan!

I hope you don't mind me piggy-backing on this PR to bump everything to Gradle 9.x + plugin 5.x + jdk21 versions.

Disclaimer: once I find the time for it.
I definitely won't be mad if you are faster 😉

@fibsifan
Copy link
Contributor Author

fibsifan commented Dec 6, 2025

@mpern Good Idea! - first step (updating the version numbers) was easy.

But now I'm stuck: the -b command line parameter for gradle is gone in gradle 9 - and it is used for bootstrapping. I think a different approach for bootstrapping is now needed.

I didn't have an idea immediately, but maybe you have?

@mpern
Copy link
Member

mpern commented Dec 9, 2025

Some initial ideas after 10min brainstorming with a rubber duck:

  • Conditionally apply(from = "bootstrap.gradle.kts") in build.gradle. kts
    • chicken-and-egg problem, maybe: cannot apply Commerce plugin because no manifest.json exists yet?
  • build - > build.template bootstrap - > build; add task to bootstrap to also generate the real/final build
    • can I overwrite the currently executing build.gradle.kts? 🤔

@fibsifan
Copy link
Contributor Author

I like the first idea and already had and immediately tossed the second one 😅

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.

2 participants