Good point!
.mvn/maven.config is handled by MavenCli and should not require forked
execution.
.mvn/jvm.config is handled by mvn shell script and does require forked
execution. Verifier "auto" mode does not currently consider presence of
.mvn/jvm.config, so we'll either need fix Verifier or force
ok, no issue found, then embedded mode merged to master
later switching back to non-embedded is easy if necessary
Regards,
Hervé
Le samedi 25 mars 2017, 17:32:38 CEST Hervé BOUTEMY a écrit :
> thanks for the complements
> FTR, I checked and added pointers to code
>
> Regards,
>
> Hervé
>
>
Does it force a fork if there is a .mvn/maven.config file to be picked up?
On Tue 28 Mar 2017 at 07:07, Hervé BOUTEMY wrote:
> ok, no issue found, then embedded mode merged to master
>
> later switching back to non-embedded is easy if necessary
>
> Regards,
>
> Hervé
>
>
On 28 March 2017 at 11:56, Igor Fedorenko wrote:
> Good point!
>
> .mvn/maven.config is handled by MavenCli and should not require forked
> execution.
>
> .mvn/jvm.config is handled by mvn shell script and does require forked
> execution. Verifier "auto" mode does not
Looked fine to me when I found it just after writing my version!
Merge away!
On Tue 28 Mar 2017 at 20:31, Robert Scholte wrote:
> I've created a branch for MNG-6185, ready to be merged.
>
> Robert
>
> On Fri, 24 Mar 2017 11:45:14 +0100, Stephen Connolly
>
Github user ifedorenko commented on the issue:
https://github.com/apache/maven/pull/110
spam?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
GitHub user devopsharish opened a pull request:
https://github.com/apache/maven/pull/110
plexus
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/devopsharish/maven master
Alternatively you can review and apply these changes as
Just to confirm: on Windows in all three cases the tests are skipped when
following the steps to reproduce MNG-6198
Robert
On Mon, 27 Mar 2017 22:08:06 +0200, Stephen Connolly
wrote:
Found another issue with the shell script: MNG-6198 (would be good if
Am 2017-03-27 um 19:58 schrieb Stephen Connolly:
My mng-6195 branch is passing all ITs
https://builds.apache.org/view/Maven/job/maven-3.x-jenkinsfile/job/mng-6195/
Can somebody test it on FreeBSD (check that .mvn folder is resolved
correctly when using -f to specify a path outside the current
I've created a branch for MNG-6185, ready to be merged.
Robert
On Fri, 24 Mar 2017 11:45:14 +0100, Stephen Connolly
wrote:
Thanks. My analysis was correct on the Windows mvn.cmd then ;-)
On Tue 28 Mar 2017 at 19:09, Robert Scholte wrote:
> Just to confirm: on Windows in all three cases the tests are skipped when
> following the steps to reproduce MNG-6198
>
> Robert
>
> On Mon, 27 Mar 2017
11 matches
Mail list logo