Skip to content

Commit 86eba90

Browse files
author
Stevo Slavic
committed
Fix JibxMarshallerTests failing on Windows
Before this change JibxMarshallerTests would fail on Windows with error message explaining that JiBX compiler generated classes are not found on classpath for binding with name 'binding'. Tests would execute well on Linux. Actual root cause of this bug is found to be in JiBX 1.1.5 release that is used to build Spring. Binding name can be explicitly specified in JiBX binding file. If omitted, when generating classes JiBX compiler as fall-back mechanism tries to derive binding name from binding file name. That logic had bug which gets manifested when configured binding file path has mixed Windows and Linux style file separators, as in case when mentioned JibxMarshallerTests are executed on Windows platform. This change introduces one of the two possible solutions - since this bug was fixed in JiBX 1.2 (see JIBX-441 for more details), dependency to JiBX is upgraded from 1.1.5 to the latest 1.2.3 release. For build to pass with this change, public access to 1.2.3 version of JiBX has to be granted on SpringSource libs-release repository (http://repo.springsource.org/libs-release). Issue: SPR-8360
1 parent 0ba73d8 commit 86eba90

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

build.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -241,7 +241,7 @@ project('spring-oxm') {
241241
compile "commons-lang:commons-lang:2.5"
242242
compile("com.thoughtworks.xstream:xstream:1.3.1", optional)
243243
compile("com.sun.xml.bind:jaxb-impl:2.1.7", optional)
244-
compile("org.jibx:jibx-run:1.1.5", optional)
244+
compile("org.jibx:jibx-run:1.2.3", optional)
245245
compile("org.apache.xmlbeans:xmlbeans:2.4.0", optional)
246246
compile("org.codehaus.castor:castor-xml:1.3.2", optional)
247247
testCompile "org.codehaus.jettison:jettison:1.0.1"

spring-oxm/oxm.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ dependencies {
99
castor "velocity:velocity:1.5"
1010
xjc "com.sun.xml:com.springsource.com.sun.tools.xjc:2.1.7"
1111
xmlbeans "org.apache.xmlbeans:com.springsource.org.apache.xmlbeans:2.4.0"
12-
jibx "org.jibx:jibx-bind:1.1.5"
12+
jibx "org.jibx:jibx-bind:1.2.3"
1313
jibx "bcel:bcel:5.1"
1414
}
1515

0 commit comments

Comments
 (0)