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

        Christian Schneider created issue -
        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.
        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
        Christian Schneider made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        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.
        Hide
        Christian Schneider added a comment -

        Ok. Will add that tomorrow

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

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development