Re: seconder on other fixes for Maven 3.5.3

2018-02-11 Thread Hervé BOUTEMY
Checkstyle fixes and Java 7 code (vs Guava) merged

remaining to decide:
MNG-6069 https://github.com/apache/maven/pull/153 commons-cli updates
MNG-5378 https://github.com/apache/maven/pull/155 partial plexus-utils -> 
commons-lang

There are some discussion pending on the first one, and for the second I'm not 
sure we want to switch to commons-lang

Regards,

Hervé

Le samedi 10 février 2018, 22:24:10 CET Arnaud Héritier a écrit :
> +1 for them Hervé
> 
> Thx
> 
> On Fri, Feb 9, 2018 at 12:29 AM, Hervé BOUTEMY 
> 
> wrote:
> > Hi,
> > 
> > Here is a new list of fixes I'd like to merge:
> > 
> > - update parent pom to version 31
> > https://builds.apache.org/view/M-R/view/Maven/job/maven-> > 
> > wip/job/maven/job/upgrade_parent_pom/
> > 
> > - MNG-6255 Maven script cannot parse jvm.config with CRLF
> > https://issues.apache.org/jira/browse/MNG-6255
> > https://builds.apache.org/view/M-R/view/Maven/job/maven-> > 
> > wip/job/maven/job/MNG-6255/
> > 
> > - Jansi 1.17 upgrade, which fixes 3 Jira issues on encoding &
> > MingW/GitBash
> > https://issues.apache.org/jira/browse/MNG-6320
> > https://issues.apache.org/jira/browse/MNG-6282
> > https://issues.apache.org/jira/browse/MNG-6218
> > https://builds.apache.org/view/M-R/view/Maven/job/maven-> > 
> > wip/job/maven/job/MNG-6320_jansi-1.17/
> > 
> > - Mockito upgrade
> > https://issues.apache.org/jira/browse/MNG-6335
> > https://builds.apache.org/view/M-R/view/Maven/job/maven-> > 
> > wip/job/maven/job/MNG-6335_Mockito_upgrade/
> > 
> > 
> > Regards,
> > 
> > Hervé
> > 
> > -
> > To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> > For additional commands, e-mail: dev-h...@maven.apache.org



-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



Re: seconder on other fixes for Maven 3.5.3

2018-02-10 Thread Arnaud Héritier
+1 for them Hervé

Thx

On Fri, Feb 9, 2018 at 12:29 AM, Hervé BOUTEMY 
wrote:

> Hi,
>
> Here is a new list of fixes I'd like to merge:
>
> - update parent pom to version 31
> https://builds.apache.org/view/M-R/view/Maven/job/maven-
> wip/job/maven/job/upgrade_parent_pom/
>
> - MNG-6255 Maven script cannot parse jvm.config with CRLF
> https://issues.apache.org/jira/browse/MNG-6255
> https://builds.apache.org/view/M-R/view/Maven/job/maven-
> wip/job/maven/job/MNG-6255/
>
> - Jansi 1.17 upgrade, which fixes 3 Jira issues on encoding & MingW/GitBash
> https://issues.apache.org/jira/browse/MNG-6320
> https://issues.apache.org/jira/browse/MNG-6282
> https://issues.apache.org/jira/browse/MNG-6218
> https://builds.apache.org/view/M-R/view/Maven/job/maven-
> wip/job/maven/job/MNG-6320_jansi-1.17/
>
> - Mockito upgrade
> https://issues.apache.org/jira/browse/MNG-6335
> https://builds.apache.org/view/M-R/view/Maven/job/maven-
> wip/job/maven/job/MNG-6335_Mockito_upgrade/
>
>
> Regards,
>
> Hervé
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> For additional commands, e-mail: dev-h...@maven.apache.org
>
>


-- 
-
Arnaud Héritier
http://aheritier.net
Mail/GTalk: aheritier AT gmail DOT com
Twitter/Skype : aheritier


Re: seconder on other fixes for Maven 3.5.3

2018-02-10 Thread Hervé BOUTEMY
Le samedi 10 février 2018, 19:31:13 CET Sylwester Lachiewicz a écrit :
> Hi,
> i would like to add also:
> https://github.com/apache/maven/pull/149 to fast fix all checkstyle
> violations
+1

> MNG-6069 https://github.com/apache/maven/pull/153 commons-cli updates
> MNG-5378 https://github.com/apache/maven/pull/155 partial plexus-utils ->
> commons-lang
I don't know on these ones: review from others would be useful

> https://github.com/apache/maven/pull/156 partial replacement to Google Guava
+1

> 
> 
> I'm not sure, but a pull requests from GitHub is not always closed - maybe
> we need to update the documentation on how to contribute to Maven [1] using
> the GitHub?
with the move to GitBox, we'll be able to close the PRs

> 
> There is also more than 40 already delivered fixes [2] for review.
> 
> Regards
> Sylwester
> 
> [1]
> http://maven.apache.org/guides/development/guide-maven-development.html#Crea
> ting_and_submitting_a_patch [2] https://github.com/apache/maven/pulls
> 
> 
> pt., 9 lut 2018 o 08:19 użytkownik Stephen Connolly <
> 
> stephen.alan.conno...@gmail.com> napisał:
> > +1 on all of those
> > 
> > On Thu 8 Feb 2018 at 23:30, Hervé BOUTEMY  wrote:
> > > Hi,
> > > 
> > > Here is a new list of fixes I'd like to merge:
> > > 
> > > - update parent pom to version 31
> > 
> > https://builds.apache.org/view/M-R/view/Maven/job/maven-wip/job/maven/job/
> > upgrade_parent_pom/> 
> > > - MNG-6255 Maven script cannot parse jvm.config with CRLF
> > > https://issues.apache.org/jira/browse/MNG-6255
> > 
> > https://builds.apache.org/view/M-R/view/Maven/job/maven-wip/job/maven/job/
> > MNG-6255/> 
> > > - Jansi 1.17 upgrade, which fixes 3 Jira issues on encoding &
> > 
> > MingW/GitBash
> > 
> > > https://issues.apache.org/jira/browse/MNG-6320
> > > https://issues.apache.org/jira/browse/MNG-6282
> > > https://issues.apache.org/jira/browse/MNG-6218
> > 
> > https://builds.apache.org/view/M-R/view/Maven/job/maven-wip/job/maven/job/
> > MNG-6320_jansi-1.17/> 
> > > - Mockito upgrade
> > > https://issues.apache.org/jira/browse/MNG-6335
> > 
> > https://builds.apache.org/view/M-R/view/Maven/job/maven-wip/job/maven/job/
> > MNG-6335_Mockito_upgrade/> 
> > > Regards,
> > > 
> > > Hervé
> > > 
> > > -
> > > To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> > > For additional commands, e-mail: dev-h...@maven.apache.org
> > > 
> > > --
> > 
> > Sent from my phone



-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



Re: seconder on other fixes for Maven 3.5.3

2018-02-10 Thread Sylwester Lachiewicz
Hi,
i would like to add also:
https://github.com/apache/maven/pull/149 to fast fix all checkstyle
violations
MNG-6069 https://github.com/apache/maven/pull/153 commons-cli updates
MNG-5378 https://github.com/apache/maven/pull/155 partial plexus-utils ->
commons-lang
https://github.com/apache/maven/pull/156 partial replacement to Google Guava


I'm not sure, but a pull requests from GitHub is not always closed - maybe
we need to update the documentation on how to contribute to Maven [1] using
the GitHub?

There is also more than 40 already delivered fixes [2] for review.

Regards
Sylwester

[1]
http://maven.apache.org/guides/development/guide-maven-development.html#Creating_and_submitting_a_patch
[2] https://github.com/apache/maven/pulls


pt., 9 lut 2018 o 08:19 użytkownik Stephen Connolly <
stephen.alan.conno...@gmail.com> napisał:

> +1 on all of those
>
> On Thu 8 Feb 2018 at 23:30, Hervé BOUTEMY  wrote:
>
> > Hi,
> >
> > Here is a new list of fixes I'd like to merge:
> >
> > - update parent pom to version 31
> >
> >
> https://builds.apache.org/view/M-R/view/Maven/job/maven-wip/job/maven/job/upgrade_parent_pom/
> >
> > - MNG-6255 Maven script cannot parse jvm.config with CRLF
> > https://issues.apache.org/jira/browse/MNG-6255
> >
> >
> https://builds.apache.org/view/M-R/view/Maven/job/maven-wip/job/maven/job/MNG-6255/
> >
> > - Jansi 1.17 upgrade, which fixes 3 Jira issues on encoding &
> MingW/GitBash
> > https://issues.apache.org/jira/browse/MNG-6320
> > https://issues.apache.org/jira/browse/MNG-6282
> > https://issues.apache.org/jira/browse/MNG-6218
> >
> >
> https://builds.apache.org/view/M-R/view/Maven/job/maven-wip/job/maven/job/MNG-6320_jansi-1.17/
> >
> > - Mockito upgrade
> > https://issues.apache.org/jira/browse/MNG-6335
> >
> >
> https://builds.apache.org/view/M-R/view/Maven/job/maven-wip/job/maven/job/MNG-6335_Mockito_upgrade/
> >
> >
> > Regards,
> >
> > Hervé
> >
> > -
> > To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> > For additional commands, e-mail: dev-h...@maven.apache.org
> >
> > --
> Sent from my phone
>


Re: seconder on other fixes for Maven 3.5.3

2018-02-08 Thread Stephen Connolly
+1 on all of those

On Thu 8 Feb 2018 at 23:30, Hervé BOUTEMY  wrote:

> Hi,
>
> Here is a new list of fixes I'd like to merge:
>
> - update parent pom to version 31
>
> https://builds.apache.org/view/M-R/view/Maven/job/maven-wip/job/maven/job/upgrade_parent_pom/
>
> - MNG-6255 Maven script cannot parse jvm.config with CRLF
> https://issues.apache.org/jira/browse/MNG-6255
>
> https://builds.apache.org/view/M-R/view/Maven/job/maven-wip/job/maven/job/MNG-6255/
>
> - Jansi 1.17 upgrade, which fixes 3 Jira issues on encoding & MingW/GitBash
> https://issues.apache.org/jira/browse/MNG-6320
> https://issues.apache.org/jira/browse/MNG-6282
> https://issues.apache.org/jira/browse/MNG-6218
>
> https://builds.apache.org/view/M-R/view/Maven/job/maven-wip/job/maven/job/MNG-6320_jansi-1.17/
>
> - Mockito upgrade
> https://issues.apache.org/jira/browse/MNG-6335
>
> https://builds.apache.org/view/M-R/view/Maven/job/maven-wip/job/maven/job/MNG-6335_Mockito_upgrade/
>
>
> Regards,
>
> Hervé
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> For additional commands, e-mail: dev-h...@maven.apache.org
>
> --
Sent from my phone


seconder on other fixes for Maven 3.5.3

2018-02-08 Thread Hervé BOUTEMY
Hi,

Here is a new list of fixes I'd like to merge:

- update parent pom to version 31
https://builds.apache.org/view/M-R/view/Maven/job/maven-wip/job/maven/job/upgrade_parent_pom/

- MNG-6255 Maven script cannot parse jvm.config with CRLF
https://issues.apache.org/jira/browse/MNG-6255
https://builds.apache.org/view/M-R/view/Maven/job/maven-wip/job/maven/job/MNG-6255/

- Jansi 1.17 upgrade, which fixes 3 Jira issues on encoding & MingW/GitBash
https://issues.apache.org/jira/browse/MNG-6320
https://issues.apache.org/jira/browse/MNG-6282
https://issues.apache.org/jira/browse/MNG-6218
https://builds.apache.org/view/M-R/view/Maven/job/maven-wip/job/maven/job/MNG-6320_jansi-1.17/

- Mockito upgrade
https://issues.apache.org/jira/browse/MNG-6335
https://builds.apache.org/view/M-R/view/Maven/job/maven-wip/job/maven/job/MNG-6335_Mockito_upgrade/


Regards,

Hervé

-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org