Re: reviewing Maven Wrapper before releasing

2021-12-08 Thread Hervé BOUTEMY
there is MVNW_REPOURL system property https://maven.apache.org/wrapper/#Using_a_Maven_Repository_Manager with a fix done in https://issues.apache.org/jira/browse/MWRAPPER-18 (showing concrete usage output) Is it what you expected? Regards, Hervé Le jeudi 9 décembre 2021, 00:59:41 CET Dan Tran

Re: reviewing Maven Wrapper before releasing

2021-12-08 Thread Dan Tran
sorry for hi-jack this thread. is it possible to configure mvnw to use a new location of distributionUrl on the fly? (without touching maven-wrapper.properties) Thanks -D On Wed, Dec 8, 2021 at 11:31 AM Slawomir Jaranowski wrote: > In current project configuration it is a bug rather ... We

Re: reviewing Maven Wrapper before releasing

2021-12-08 Thread Slawomir Jaranowski
In current project configuration it is a bug rather ... We can use a wrapper with java 5 but we can't install it because the plugin requires java 8 ... śr., 8 gru 2021 o 20:18 Xeno Amess napisał(a): > +1 for forcing more people quit java 5,6,7, as even 17 is out. > 8 is the minimum version norma

Re: reviewing Maven Wrapper before releasing

2021-12-08 Thread Xeno Amess
+1 for forcing more people quit java 5,6,7, as even 17 is out. 8 is the minimum version normal people can accept,others be zombies who do not need to be maintained IMO. They can use original wrapper plugin anyway. If we do not force them when beginning,it is harder to add the liminition later. Xe

Re: reviewing Maven Wrapper before releasing

2021-12-08 Thread Slawomir Jaranowski
I created issues with the proposition of improvement: https://issues.apache.org/jira/browse/MWRAPPER-31 https://issues.apache.org/jira/browse/MWRAPPER-32 https://issues.apache.org/jira/browse/MWRAPPER-33 please assign a version to fix ... if it is applicable. śr., 8 gru 2021 o 19:47 Hervé BOUTEM

Re: reviewing Maven Wrapper before releasing

2021-12-08 Thread Hervé BOUTEMY
because it's as it was imported: https://github.com/takari/maven-wrapper/blob/master/pom.xml#L21 I refactored but kept everything as equivalent as possible we can update the value if it has an interest, with associated Jira issue Regards, Hervé Le mercredi 8 décembre 2021, 19:14:55 CET Slawomir

Re: reviewing Maven Wrapper before releasing

2021-12-08 Thread Slawomir Jaranowski
Next question: - why use java 5 in maven-wrapper module .. it is not possible build with java > 8 [1] [1] https://ci-maven.apache.org/job/Maven/job/maven-box/job/maven-wrapper/job/master/ śr., 8 gru 2021 o 19:05 Hervé BOUTEMY napisał(a): > Le mercredi 8 décembre 2021, 15:52:46 CET Robert Scho

Re: reviewing Maven Wrapper before releasing

2021-12-08 Thread Hervé BOUTEMY
Le mercredi 8 décembre 2021, 15:52:46 CET Robert Scholte a écrit : > With mvn verify -Prun-its all ITs fail on my system. This should work > out-of-the-box, so I'll need to investigate the issue. > > And I'm pretty sure we can remove the cli package: you can't pass System > properties to Maven, on

Re: reviewing Maven Wrapper before releasing

2021-12-08 Thread Hervé BOUTEMY
MWRAPPER-29 and MWRAPPER-30 merged: thank you Slawomir Le mercredi 8 décembre 2021, 16:07:48 CET Slawomir Jaranowski a écrit : > https://issues.apache.org/jira/browse/MWRAPPER-29 > > śr., 8 gru 2021 o 15:52 Robert Scholte napisał(a): > > With mvn verify -Prun-its all ITs fail on my system. This

Re: reviewing Maven Wrapper before releasing

2021-12-08 Thread Slawomir Jaranowski
https://issues.apache.org/jira/browse/MWRAPPER-29 śr., 8 gru 2021 o 15:52 Robert Scholte napisał(a): > With mvn verify -Prun-its all ITs fail on my system. This should work > out-of-the-box, so I'll need to investigate the issue. > > And I'm pretty sure we can remove the cli package: you can't p

Re: reviewing Maven Wrapper before releasing

2021-12-08 Thread Robert Scholte
With mvn verify -Prun-its all ITs fail on my system. This should work out-of-the-box, so I'll need to investigate the issue. And I'm pretty sure we can remove the cli package: you can't pass System properties to Maven, only arguments. And they should be passed as is to mvn. IT's should confir