You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by Robert Scholte <rf...@apache.org> on 2020/05/22 20:36:56 UTC

second apache-maven-wrapper

I've reached the point where apache-maven-wrapper is ready to be merged into master.
The code contains all original commits and a few from me to get them step by step into core.

Related tickets:
https://issues.apache.org/jira/browse/MNG-5937

https://issues.apache.org/jira/browse/MNG-6914


https://github.com/apache/maven/pull/349

https://github.com/apache/maven-integration-testing/pull/62


thanks,
Robert

Re: second apache-maven-wrapper

Posted by Arnaud Héritier <ah...@gmail.com>.
ok no worries, it was a nice to have if it wasn't yet on master.

On Sat, May 23, 2020 at 1:04 PM Karl Heinz Marbaise <kh...@gmx.de>
wrote:

>
> HI,
>
> On 23.05.20 12:51, Arnaud Héritier wrote:
> > LGTM
> > Rebasing the code could be great if not too difficult
>
> Unfortunately the code is already on master with all commits....
>
> Kind regards
> Karl Heinz Marbaise
>
> >
> > On Sat, May 23, 2020 at 12:32 PM Enrico Olivelli <eo...@gmail.com>
> > wrote:
> >
> >> Approved on github
> >>
> >> +1
> >> I have run thru the diff.
> >> It looks very well
> >>
> >> Longing for testing it on a real project
> >>
> >> The fact that you won't have to commit mave wrapper jar to local
> sources is
> >> great
> >>
> >>
> >> Enrico
> >>
> >> Il Ven 22 Mag 2020, 22:56 Manfred Moser <ma...@simpligility.com> ha
> >> scritto:
> >>
> >>> Awesome! +1
> >>>
> >>> Robert Scholte wrote on 2020-05-22 13:36 (GMT -07:00):
> >>>
> >>>> I've reached the point where apache-maven-wrapper is ready to be
> merged
> >>> into
> >>>> master.
> >>>> The code contains all original commits and a few from me to get them
> >>> step by
> >>>> step into core.
> >>>>
> >>>> Related tickets:
> >>>> https://issues.apache.org/jira/browse/MNG-5937
> >>>>
> >>>> https://issues.apache.org/jira/browse/MNG-6914
> >>>>
> >>>>
> >>>> https://github.com/apache/maven/pull/349
> >>>>
> >>>> https://github.com/apache/maven-integration-testing/pull/62
> >>>>
> >>>>
> >>>> thanks,
> >>>> Robert
>


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

Re: second apache-maven-wrapper

Posted by Karl Heinz Marbaise <kh...@gmx.de>.
HI,

On 23.05.20 12:51, Arnaud Héritier wrote:
> LGTM
> Rebasing the code could be great if not too difficult

Unfortunately the code is already on master with all commits....

Kind regards
Karl Heinz Marbaise

>
> On Sat, May 23, 2020 at 12:32 PM Enrico Olivelli <eo...@gmail.com>
> wrote:
>
>> Approved on github
>>
>> +1
>> I have run thru the diff.
>> It looks very well
>>
>> Longing for testing it on a real project
>>
>> The fact that you won't have to commit mave wrapper jar to local sources is
>> great
>>
>>
>> Enrico
>>
>> Il Ven 22 Mag 2020, 22:56 Manfred Moser <ma...@simpligility.com> ha
>> scritto:
>>
>>> Awesome! +1
>>>
>>> Robert Scholte wrote on 2020-05-22 13:36 (GMT -07:00):
>>>
>>>> I've reached the point where apache-maven-wrapper is ready to be merged
>>> into
>>>> master.
>>>> The code contains all original commits and a few from me to get them
>>> step by
>>>> step into core.
>>>>
>>>> Related tickets:
>>>> https://issues.apache.org/jira/browse/MNG-5937
>>>>
>>>> https://issues.apache.org/jira/browse/MNG-6914
>>>>
>>>>
>>>> https://github.com/apache/maven/pull/349
>>>>
>>>> https://github.com/apache/maven-integration-testing/pull/62
>>>>
>>>>
>>>> thanks,
>>>> Robert

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

Re: second apache-maven-wrapper

Posted by Arnaud Héritier <ah...@gmail.com>.
LGTM
Rebasing the code could be great if not too difficult

On Sat, May 23, 2020 at 12:32 PM Enrico Olivelli <eo...@gmail.com>
wrote:

> Approved on github
>
> +1
> I have run thru the diff.
> It looks very well
>
> Longing for testing it on a real project
>
> The fact that you won't have to commit mave wrapper jar to local sources is
> great
>
>
> Enrico
>
> Il Ven 22 Mag 2020, 22:56 Manfred Moser <ma...@simpligility.com> ha
> scritto:
>
> > Awesome! +1
> >
> > Robert Scholte wrote on 2020-05-22 13:36 (GMT -07:00):
> >
> > > I've reached the point where apache-maven-wrapper is ready to be merged
> > into
> > > master.
> > > The code contains all original commits and a few from me to get them
> > step by
> > > step into core.
> > >
> > > Related tickets:
> > > https://issues.apache.org/jira/browse/MNG-5937
> > >
> > > https://issues.apache.org/jira/browse/MNG-6914
> > >
> > >
> > > https://github.com/apache/maven/pull/349
> > >
> > > https://github.com/apache/maven-integration-testing/pull/62
> > >
> > >
> > > thanks,
> > > Robert
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> > For additional commands, e-mail: dev-help@maven.apache.org
> >
> >
>


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

Re: second apache-maven-wrapper

Posted by Enrico Olivelli <eo...@gmail.com>.
Approved on github

+1
I have run thru the diff.
It looks very well

Longing for testing it on a real project

The fact that you won't have to commit mave wrapper jar to local sources is
great


Enrico

Il Ven 22 Mag 2020, 22:56 Manfred Moser <ma...@simpligility.com> ha
scritto:

> Awesome! +1
>
> Robert Scholte wrote on 2020-05-22 13:36 (GMT -07:00):
>
> > I've reached the point where apache-maven-wrapper is ready to be merged
> into
> > master.
> > The code contains all original commits and a few from me to get them
> step by
> > step into core.
> >
> > Related tickets:
> > https://issues.apache.org/jira/browse/MNG-5937
> >
> > https://issues.apache.org/jira/browse/MNG-6914
> >
> >
> > https://github.com/apache/maven/pull/349
> >
> > https://github.com/apache/maven-integration-testing/pull/62
> >
> >
> > thanks,
> > Robert
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> For additional commands, e-mail: dev-help@maven.apache.org
>
>

Re: second apache-maven-wrapper

Posted by Manfred Moser <ma...@simpligility.com>.
Awesome! +1 

Robert Scholte wrote on 2020-05-22 13:36 (GMT -07:00):

> I've reached the point where apache-maven-wrapper is ready to be merged into
> master.
> The code contains all original commits and a few from me to get them step by
> step into core.
> 
> Related tickets:
> https://issues.apache.org/jira/browse/MNG-5937
> 
> https://issues.apache.org/jira/browse/MNG-6914
> 
> 
> https://github.com/apache/maven/pull/349
> 
> https://github.com/apache/maven-integration-testing/pull/62
> 
> 
> thanks,
> Robert

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