Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade to Gradle 7.2 and add feature variants for Jakarta XML parsing support #2962

Merged
merged 2 commits into from
Dec 7, 2021

Conversation

chrisdennis
Copy link
Member

This PR updates the Ehcache build to use Gradle 7.2. At the same time it adds support for lightweight (same artifact) and heavyweight (split sources and artifacts) feature variants to support the optional use of Jakarta namespaced libraries for XML parsing.

Along with the Gradle 7.2 build changes come some associated (mandated?) dependency scoping changes:

  • org.ehcache.modules:ehcache-107
    • javax.cache:cache-api : providedcompile
  • org.ehcache.modules.ehcache-xml
    • javax.xml.bind:jaxb-api : <absent>compile
    • org.glassfish.jaxb:jaxb-runtime : providedruntime
    • jakarta.xml.bind:jakarta.xml.bind-api : <absent>compile (optional)
  • org.ehcache:ehcache
    • org.slf4j:slf4j-api : compileruntime
    • javax.cache:cache-api : providedcompile
    • org.glassfish.jaxb:jaxb-runtime : providedruntime
  • org.ehcache:ehcache-clustered
    • org.slf4j:slf4j-api : compileruntime
    • org.ehcache:ehcache : compileruntime

@cljohnso
Copy link
Contributor

cljohnso commented Dec 2, 2021

What's the reason behind the transition from compile to runtime for SLF4J?

@chrisdennis
Copy link
Member Author

I think this is triggered by the move to the maven-publish plugin. The slf4j-api dependency has always been implementation... which means it's not needed to compile against Ehcache3... which means the correct maven scope should aways have been runtime.

@chrisdennis chrisdennis force-pushed the jakarta-variants branch 3 times, most recently from 2062777 to f0ea3b4 Compare December 6, 2021 18:08
Copy link
Contributor

@cljohnso cljohnso left a comment

Choose a reason for hiding this comment

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

Finished a pass but have not checked differences between the version I checked out and the current one.

ehcache-api/build.gradle Show resolved Hide resolved
ehcache/build.gradle Show resolved Hide resolved
ehcache/build.gradle Outdated Show resolved Hide resolved
Copy link
Contributor

@cljohnso cljohnso left a comment

Choose a reason for hiding this comment

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

Good to go unless you have additional changes you want to make.

 * org.ehcache.modules:ehcache-xml
 * org.ehcache.modules:ehcache-107
 * org.ehcache:ehcache
@cljohnso cljohnso merged commit 8e830d0 into ehcache:master Dec 7, 2021
@chrisdennis chrisdennis deleted the jakarta-variants branch December 15, 2021 20:15
@chrisdennis chrisdennis added this to the 3.10.0 milestone Mar 10, 2022
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