castor

Spring major release discrepancy prevents successful build of jpa-extensions-it

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.3.2
  • Fix Version/s: 1.3.2
  • Component/s: JDO tests
  • Labels:
    None
  • Environment:
    castor trunk
    eclipse 3.5.2 / plain maven 2
    OS X 10.6.3 / Windows XP SP 3
  • Number of attachments :
    2

Description

Steps to reproduce
Check out castor trunk or make a clean build

Expected
All sub-projects will successfully be built

Actual
jpa-extensions-it fails with
Caused by: org.springframework.beans.factory.BeanDefinitionStoreException: Unexpected exception parsing XML document fro
m class path resource [org/castor/jpa/scenario/one_to_many/spring-config.xml]; nested exception is java.lang.NoSuchMetho
dError: org.springframework.beans.MutablePropertyValues.add(Ljava/lang/String;Ljava/lang/Object;)Lorg/springframework/be
ans/MutablePropertyValues;

Additional
The issue is caused by different versions of spring-beans:

  • 2.5.x is is defined in the main POM
  • castor's spring-orm depends on 3.0.2.RELEASE
    and the former one shades/hides the latter one.
    This can be reproduced on both, the command line and in eclipse.
  1. Text File
    patch.txt
    01/May/10 8:08 AM
    6 kB
    Andras Hatvani
  2. Text File
    patch.txt
    01/May/10 6:41 AM
    52 kB
    Andras Hatvani

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Andras Hatvani added a comment -

Attached patch:

  • Reformatted POMs with Spring upgraded to 3.0.2.RELEASE
  • .classpath updated to refer to Spring 3.0.2.RELEASE
  • Extended both, the POMs and .classpath by spring-asm and spring-expression
Show
Andras Hatvani added a comment - Attached patch: Reformatted POMs with Spring upgraded to 3.0.2.RELEASE .classpath updated to refer to Spring 3.0.2.RELEASE Extended both, the POMs and .classpath by spring-asm and spring-expression
Hide
Permalink
Werner Guttmann added a comment -

Lookng at this right now ....

Show
Werner Guttmann added a comment - Lookng at this right now ....
Hide
Permalink
Werner Guttmann added a comment -

Andras, can you please re-create your patch with just the minimal changes. Currently, you seem to have auto-formatted the POMs, which makes it hard for us to consolidate things.

Show
Werner Guttmann added a comment - Andras, can you please re-create your patch with just the minimal changes. Currently, you seem to have auto-formatted the POMs, which makes it hard for us to consolidate things.
Hide
Permalink
Ralf Joachim added a comment -

I already check the patch and don't see anything why it should not be committed.

Show
Ralf Joachim added a comment - I already check the patch and don't see anything why it should not be committed.
Hide
Permalink
Andras Hatvani added a comment -

Patch without formatting changes

Show
Andras Hatvani added a comment - Patch without formatting changes
Hide
Permalink
Werner Guttmann added a comment -

Ralf, I am still getting exceptions related to CGLIB when trying to execute a test case .. ,

Show
Werner Guttmann added a comment - Ralf, I am still getting exceptions related to CGLIB when trying to execute a test case .. ,
Hide
Permalink
Werner Guttmann added a comment -

Okay, I just found the problem, and will commit the changes to .classpath/POM(s) in a few minutes.

Show
Werner Guttmann added a comment - Okay, I just found the problem, and will commit the changes to .classpath/POM(s) in a few minutes.

People

  • Assignee:
    Werner Guttmann
    Reporter:
    Andras Hatvani
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: