Camel
  1. Camel
  2. CAMEL-4357

Move org.apache.camel.Main to org.apache.camel.main.Main

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.8.0
    • Fix Version/s: 2.9.0
    • Component/s: camel-core
    • Labels:
      None
    • Estimated Complexity:
      Unknown

      Description

      I am currently looking into the dependencies betwen packages in camel-core. The packages org.apache.camel and org.apache.camel.spi form the camel api.
      So I am trying to make them not depend on other packages from camel-core. One problem there is the starter class Main. It needs access to impl packages as it needs to start camel. So it should not live in org.apache.camel.

      I propose to move it to org.apache.camel.main. To not break anything right now I will create a deprecated class Main in org.apache.camel that extends the moved Main. We can remove the deprecated version in camel 3.0

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        2d 23h 56m 1 Christian Schneider 22/Aug/11 14:34
        Hide
        Christian Schneider added a comment -

        Ok. Will add that tomorrow

        Show
        Christian Schneider added a comment - Ok. Will add that tomorrow
        Hide
        Claus Ibsen added a comment -

        There is a CS error as the license header is missing in one file.

        When adding a new package in camel-core, a package.html file should be included, which briefly summarizes the package. See some of the other for examples to copy.

        And in the camel-core/pom.xml file there is some javadoc grouping of packages. I guess the new main package is to be added as well.

        Show
        Claus Ibsen added a comment - There is a CS error as the license header is missing in one file. When adding a new package in camel-core, a package.html file should be included, which briefly summarizes the package. See some of the other for examples to copy. And in the camel-core/pom.xml file there is some javadoc grouping of packages. I guess the new main package is to be added as well.
        Christian Schneider made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Christian Schneider added a comment -

        The one in camel-spring is not problematic as it is in a non API package org.apache.camel.spring

        Show
        Christian Schneider added a comment - The one in camel-spring is not problematic as it is in a non API package org.apache.camel.spring
        Hide
        Claus Ibsen added a comment -

        There is also a Main class in camel-spring. It was actually started there as the first Main class we offered in Camel.

        Show
        Claus Ibsen added a comment - There is also a Main class in camel-spring. It was actually started there as the first Main class we offered in Camel.
        Christian Schneider created issue -

          People

          • Assignee:
            Christian Schneider
            Reporter:
            Christian Schneider
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development