castor
  1. castor
  2. CASTOR-3048

A maven building error : castor-xml has dependency on castor-codegen

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.3.2
    • Component/s: None
    • Labels:
      None
    • Number of attachments :
      1

      Description

      I have downloaded the Castor source code(version 8822, the newest version now) from SVN [1]. When I built Castor with Maven, I got a 'failed building', and the error message was The projects in the reactor contain a cyclic reference: Edge between 'Vertex

      {label='org.codehaus.castor:castor-codegen'}

      ' and 'Vertex

      {label='org.codehaus.castor:castor-xml'}

      ' introduces to cycle in the graph org.codehaus.castor:castor-xml --> org.codehaus.castor:castor-codegen --> org.codehaus.castor:castor-xml.

      The castor-xml should not depend on castor-codegen. In pom.xml of castor-xml there is no direct dependency on castor-codegen but there may be an indirect dependency through maven plugin related to code generation for mapping.xsd. I'm not sure what's the best point to break this cycle but my feeling is that we should depend on a final version of maven plugin which then depends on a released version of castor-codegen. (Comments from Ralf, thanks)

      I have made a small patch for it (see attachment), which made the maven 'compile' command sucess. I have not understood the whole Castor project yet, so the small patch may be not right.

      [1] http://svn.codehaus.org/castor/castor/trunk

        Activity

        Hide
        Werner Guttmann added a comment -

        Will take care of this ....

        Show
        Werner Guttmann added a comment - Will take care of this ....
        Hide
        Werner Guttmann added a comment -

        Hi, I have just reverted this change, things should work now without any problems.

        Show
        Werner Guttmann added a comment - Hi, I have just reverted this change, things should work now without any problems.

          People

          • Assignee:
            Werner Guttmann
            Reporter:
            Wensheng Dou
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: