Camel
  1. Camel
  2. CAMEL-991

Maven goal camel:dot should use overridden <mainClass/> when specified

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.4.0
    • Fix Version/s: 1.5.0
    • Component/s: tooling
    • Labels:
      None
    • Patch Info:
      Patch Available

      Description

      Maven goal camel:dot should use overridden <mainClass/> when used as in pom.xml:

      <build>
      <plugins>
      <!-- Allows the routes to be run via 'mvn camel:run' -->
      <plugin>
      <groupId>org.apache.camel</groupId>
      <artifactId>camel-maven-plugin</artifactId>
      <version>$

      {pom.version}

      </version>
      <!-- optional, default value: org.apache.camel.spring.Main -->
      <configuration>
      <mainClass>mypackage.boot.camel.CamelStartup</mainClass>
      </configuration>
      </plugin>
      </plugins>
      </build>

        Activity

        Hide
        Claus Ibsen added a comment -

        Is useable for some of the other maven goals as well: camel:run etc.

        Show
        Claus Ibsen added a comment - Is useable for some of the other maven goals as well: camel:run etc.
        Hide
        Jonathan Anstey added a comment -

        Seb, could you re attach your patch and grant the ASF license? We cannot apply any patches that don't have this granted.

        Show
        Jonathan Anstey added a comment - Seb, could you re attach your patch and grant the ASF license? We cannot apply any patches that don't have this granted.
        Hide
        Claus Ibsen added a comment -

        Seb, could you re attach the patch with the ASF license granted. You should not attach files while creating the tickets as the radiobutton for granting this license is only possible afterwards.

        So please use "Attach File" and upload it again with the license.

        We can only accept patches with this license

        Show
        Claus Ibsen added a comment - Seb, could you re attach the patch with the ASF license granted. You should not attach files while creating the tickets as the radiobutton for granting this license is only possible afterwards. So please use "Attach File" and upload it again with the license. We can only accept patches with this license
        Hide
        Claus Ibsen added a comment -

        Jon, remember to document it on the wiki as well

        Show
        Claus Ibsen added a comment - Jon, remember to document it on the wiki as well
        Hide
        Jonathan Anstey added a comment -

        Claus, looks like camel:run already had this functionality. Will update the wiki after the patch is applied.

        Show
        Jonathan Anstey added a comment - Claus, looks like camel:run already had this functionality. Will update the wiki after the patch is applied.
        Hide
        Seb Auvray added a comment -

        Patch With the license

        Show
        Seb Auvray added a comment - Patch With the license
        Hide
        Seb Auvray added a comment -

        @Claus and Jonathan, Yes camel:run had this overriding possibility but camel:dot would'nt use it.

        Show
        Seb Auvray added a comment - @Claus and Jonathan, Yes camel:run had this overriding possibility but camel:dot would'nt use it.
        Hide
        Jonathan Anstey added a comment -

        Your patch was applied in revision 705236. Thanks!

        Show
        Jonathan Anstey added a comment - Your patch was applied in revision 705236. Thanks!
        Show
        Jonathan Anstey added a comment - Updated wiki doc here http://cwiki.apache.org/confluence/display/CAMEL/Camel+Maven+Plugin
        Hide
        Claus Ibsen added a comment -

        Closing all 1.5.0 issues

        Show
        Claus Ibsen added a comment - Closing all 1.5.0 issues

          People

          • Assignee:
            Jonathan Anstey
            Reporter:
            Seb Auvray
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 4h
              4h
              Remaining:
              Remaining Estimate - 4h
              4h
              Logged:
              Time Spent - Not Specified
              Not Specified

                Development