[GitHub] [maven] mthmulders commented on pull request #602: Maven.config from start scripts

[GitBox <gi...@apache.org>]

mthmulders commented on pull request #602:
URL: https://github.com/apache/maven/pull/602#issuecomment-981747872


   From [MNG-7193](https://issues.apache.org/jira/browse/MNG-7193):
   > `MAVEN_ARGS` is overwritten by `maven.config` is overwritten by CLI arguments.
   
   However, if I build Maven on this branch and test with 
   
   ```console
   $ cat .mvn/maven.config 
   -V
   -DskipTests=true
   
   $ MAVEN_ARGS="-DskipTests=false" mvn test 
   ```
   
   I get
   ```console
   [INFO] -------------------------------------------------------
   [INFO]  T E S T S
   [INFO] -------------------------------------------------------
   [INFO] Running foo.AppTest
   [INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.023 s - in foo.AppTest
   ```
   
   Since `maven.config` should have precedence over `MAVEN_ARGS`, I would expect the tests to be skipped. If I use the same `maven.config` and run `mvn test` (without `MAVEN_ARGS`) the tests are skipped, as expected.
   
   I did a second test, where a Maven profile upgrades a plugin; it shows the same behaviour: `MAVEN_ARGS` "wins" from `.mvn/maven.config`, despite [MNG-7193](https://issues.apache.org/jira/browse/MNG-7193) saying the reverse.


-- 
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: issues-unsubscribe@maven.apache.org

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