You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by "jamesnetherton (via GitHub)" <gi...@apache.org> on 2024/03/18 07:51:12 UTC

[I] Optimize the Maven release [camel-quarkus]

jamesnetherton opened a new issue, #5887:
URL: https://github.com/apache/camel-quarkus/issues/5887

   ### Description
   
   We deploy a lot of redundant stuff.
   
   * integration-test-groups/*
   * integration-tests-jvm/*
   * perf-regression
   
   The only test JARs that need to be deployed are in the integration-tests module.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] Optimize the Maven release [camel-quarkus]

Posted by "jamesnetherton (via GitHub)" <gi...@apache.org>.
jamesnetherton commented on issue #5887:
URL: https://github.com/apache/camel-quarkus/issues/5887#issuecomment-2102151640

   This appears to be trickier than it first seems. I was hoping to just add a property for `<maven.deploy.skip>true</maven.deploy.skip>` to the parent of the modules mentioned above.
   
   But due to the parent that the sub-modules use, it has no effect. You have to explicitly add the property in each sub-module pom.xml. Which is a pain to maintain.
   
   So I thought of adding something extra to the `cq-maven-plugin` format mojo that could configure the property automatically like:
   
   ```xml
   <mavenDeploySkip>
       <basedir>${project.basedir}</basedir>
       <includes>
           <include>integration-test-groups/**/pom.xml</include>
           <include>integration-tests-jvm/**/pom.xml</include>
           <include>tooling/perf-regression/pom.xml</include>
       </includes>
   </mavenDeploySkip>
   ```
   
   It [works](https://github.com/jamesnetherton/cq-maven-plugin/commit/b0a60ca3663ce94e828773e43875bef5aa108294). Just not sure if that's the right way to go. WDYT @ppalaga?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org