Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Commit 75252e1

Browse files
committed
HHH-19723 Upgrade from wildfly-transaction-client-jakarta 2.x to wildfly-transaction-client 3.x in hibernate-testing
1 parent 0a76cbe commit 75252e1

File tree

3 files changed

+21
-4
lines changed

3 files changed

+21
-4
lines changed

‎hibernate-core/hibernate-core.gradle

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,13 @@ dependencies {
6363

6464
testRuntimeOnly libs.byteBuddy
6565
testRuntimeOnly testLibs.weld
66-
testRuntimeOnly testLibs.wildFlyTxnClient
66+
implementation(libs.logging)
67+
testRuntimeOnly(testLibs.wildFlyTxnClient) {
68+
// For some reason this dependency brings in an incompatible version of JBoss Logging in the compile classpath,
69+
// even though we specify our own dependency already
70+
// *and* this is supposed to be a runtime-only dep.
71+
exclude group: 'org.jboss.logging'
72+
}
6773
testImplementation libs.jandex
6874
testImplementation jakartaLibs.jsonb
6975
testImplementation libs.jackson

‎hibernate-testing/hibernate-testing.gradle

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,18 @@ dependencies {
4141

4242
implementation libs.hibernateModels
4343
implementation libs.jandex
44-
implementation testLibs.wildFlyTxnClient
44+
implementation(libs.logging)
45+
// Note: while this dependency isn't explicitly used, removing it will lead to using JBossStandAloneJtaPlatform
46+
// by default in tests, which will apparently lead to leaks detected by HibernateClassLoaderLeaksTest,
47+
// which will fail.
48+
// JBossStandAloneJtaPlatform is old and likely wouldn't work with Jakarta EE anyway,
49+
// so I won't try to fix it, but we might want to remove it in the next minor,
50+
// and then we can remove the dependency below.
51+
implementation(testLibs.wildFlyTxnClient) {
52+
// For some reason this dependency brings in an incompatible version of JBoss Logging in the compile classpath,
53+
// even though we specify our own dependency already.
54+
exclude group: 'org.jboss.logging'
55+
}
4556
implementation testLibs.junit5Engine
4657
implementation testLibs.junit5Launcher
4758

‎settings.gradle

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -182,7 +182,7 @@ dependencyResolutionManagement {
182182
def shrinkwrapVersion = version "shrinkwrap", "1.2.6"
183183
def shrinkwrapDescriptorsVersion = version "shrinkwrapDescriptors", "2.0.0"
184184
def weldVersion = version "weld", "6.0.3.Final"
185-
def wildFlyTxnClientVersion = version "wildFlyTxnClient", "2.0.0.Final"
185+
def wildFlyTxnClientVersion = version "wildFlyTxnClient", "3.0.5.Final"
186186

187187
def jfrUnitVersion = version "jfrUnit", "1.0.0.Alpha2"
188188

@@ -212,7 +212,7 @@ dependencyResolutionManagement {
212212

213213
library( "jbossJta", "org.jboss.narayana.jta", "narayana-jta" ).versionRef( jbossJtaVersion )
214214
library( "jbossTxSpi", "org.jboss", "jboss-transaction-spi" ).versionRef( jbossTxSpiVersion )
215-
library( "wildFlyTxnClient", "org.wildfly.transaction", "wildfly-transaction-client-jakarta" ).versionRef( wildFlyTxnClientVersion )
215+
library( "wildFlyTxnClient", "org.wildfly.transaction", "wildfly-transaction-client" ).versionRef( wildFlyTxnClientVersion )
216216

217217
library( "weld", "org.jboss.weld.se", "weld-se-shaded" ).versionRef( weldVersion )
218218

0 commit comments

Comments
(0)

AltStyle によって変換されたページ (->オリジナル) /