You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by Stephen Connolly <st...@gmail.com> on 2010/05/17 23:35:32 UTC
Re: [VOTE] Release Maven install plugin version 2.3.1
+1
On 17 May 2010 23:58, Kristian Rosenvold <kr...@gmail.com>wrote:
> Hi,
>
> We solved 1 issue:
>
> http://jira.codehaus.org/secure/ReleaseNote.jspa?projectId=11136&version=16459
>
> There are still a couple of issues left in JIRA:
>
> http://jira.codehaus.org/secure/IssueNavigator.jspa?reset=true&mode=hide&jqlQuery=project+%3D+MINSTALL+AND+resolution+%3D+Unresolved+ORDER+BY+updated+DESC
>
> Staging repo:
> https://repository.apache.org/content/repositories/maven-039/
>
> Staging site:
> http://maven.apache.org/plugins/maven-install-plugin-2.3.1/
>
> Guide to testing staged releases:
> http://maven.apache.org/guides/development/guide-testing-releases.html
>
> Vote open for 72 hours.
>
> [ ] +1
> [ ] +0
> [ ] -1
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> For additional commands, e-mail: dev-help@maven.apache.org
>
>
RE: [VOTE] Release Maven install plugin version 2.3.1
Posted by Ludwig Magnusson <lu...@itcatapult.com>.
I have supplied a patch now including one test. You asked for two but
skip=false is already tested in the default install test. Is that ok? I also
added a setSkip method to the InstallMojo. I don't know if this is the right
way to go but I found no other way of setting the skip-parameter in the
test.
I also did not really understand what you menat by (+ve and -ve).
Please give me any feedback if there is something more needed.
/Ludwig
-----Original Message-----
From: Stephen Connolly [mailto:stephen.alan.connolly@gmail.com]
Sent: den 18 maj 2010 08:15
To: Maven Developers List
Subject: Re: [VOTE] Release Maven install plugin version 2.3.1
On 18 May 2010 06:52, Ludwig Magnusson <lu...@itcatapult.com> wrote:
> I don't have an official say in this but I recently submitted a patch for
> an
> issue I created that allows for a "skip" parameter to the install plugin.
I
> assigned it to the 2.4 version (2.3.1 did not exist in jira at the time).
> Is
>
FYI, If you don't have commit access, you should not be assigning a Fix
Version for JIRA issues that you create as you do not have any ability to
control when the issue gets fixed (if you had commit access you could
controld when the issue gets fixed)
> it possible to add this patch or is it too much work?
>
Given that Kristian has already staged the release it is unlikely to make it
into 2.3.1
> Check it out here:
> http://jira.codehaus.org/browse/MINSTALL-73
>
The patch is simple enough, but anyone applying the patch would really be
looking for a positive & negative integration test to ensure that things do
not break going forward.
At this stage due to the great effort that Benjamin has put in to creating
integration tests, the Maven community really wants to ensure that we
increase coverage with new features. If you add two integration test cases
(+ve and -ve) to your patch I will apply it for you.
-Stephen
> /Ludwig
>
>
> -----Original Message-----
> From: Stephen Connolly [mailto:stephen.alan.connolly@gmail.com]
> Sent: den 17 maj 2010 23:36
> To: Maven Developers List
> Subject: Re: [VOTE] Release Maven install plugin version 2.3.1
>
> +1
>
> On 17 May 2010 23:58, Kristian Rosenvold
> <kr...@gmail.com>wrote:
>
> > Hi,
> >
> > We solved 1 issue:
> >
> >
>
>
http://jira.codehaus.org/secure/ReleaseNote.jspa?projectId=11136&version=164
> 59
> >
> > There are still a couple of issues left in JIRA:
> >
> >
>
>
http://jira.codehaus.org/secure/IssueNavigator.jspa?reset=true&mode=hide&jql
>
>
Query=project+%3D+MINSTALL+AND+resolution+%3D+Unresolved+ORDER+BY+updated+DE
<http://jira.codehaus.org/secure/IssueNavigator.jspa?reset=true&mode=hide&jq
l%0AQuery=project+%3D+MINSTALL+AND+resolution+%3D+Unresolved+ORDER+BY+update
d+DE>
> SC
> >
> > Staging repo:
> > https://repository.apache.org/content/repositories/maven-039/
> >
> > Staging site:
> > http://maven.apache.org/plugins/maven-install-plugin-2.3.1/
> >
> > Guide to testing staged releases:
> > http://maven.apache.org/guides/development/guide-testing-releases.html
> >
> > Vote open for 72 hours.
> >
> > [ ] +1
> > [ ] +0
> > [ ] -1
> >
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> > For additional commands, e-mail: dev-help@maven.apache.org
> >
> >
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> For additional commands, e-mail: dev-help@maven.apache.org
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
Re: [VOTE] Release Maven install plugin version 2.3.1
Posted by Kristian Rosenvold <kr...@gmail.com>.
I'm doing a whole series of releases at the moment for supporting
multithreaded builds (and some bugs related to that), and anything
that's not already fixed (with
decent test-cases) I'm just ignoring. As I announced a week ago
(http://www.mail-archive.com/dev@maven.apache.org/msg84663.html), there
are still some more plugins
that'll are up for imminent release. And there's going to be another
bunch after that too (deploy, release and a few more).
Your patch looked good enough. Make tests like Stephen says and I'll
review it. But it's too late for 2.3.1.
Kristian
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
Re: [VOTE] Release Maven install plugin version 2.3.1
Posted by Stephen Connolly <st...@gmail.com>.
On 18 May 2010 06:52, Ludwig Magnusson <lu...@itcatapult.com> wrote:
> I don't have an official say in this but I recently submitted a patch for
> an
> issue I created that allows for a "skip" parameter to the install plugin. I
> assigned it to the 2.4 version (2.3.1 did not exist in jira at the time).
> Is
>
FYI, If you don't have commit access, you should not be assigning a Fix
Version for JIRA issues that you create as you do not have any ability to
control when the issue gets fixed (if you had commit access you could
controld when the issue gets fixed)
> it possible to add this patch or is it too much work?
>
Given that Kristian has already staged the release it is unlikely to make it
into 2.3.1
> Check it out here:
> http://jira.codehaus.org/browse/MINSTALL-73
>
The patch is simple enough, but anyone applying the patch would really be
looking for a positive & negative integration test to ensure that things do
not break going forward.
At this stage due to the great effort that Benjamin has put in to creating
integration tests, the Maven community really wants to ensure that we
increase coverage with new features. If you add two integration test cases
(+ve and -ve) to your patch I will apply it for you.
-Stephen
> /Ludwig
>
>
> -----Original Message-----
> From: Stephen Connolly [mailto:stephen.alan.connolly@gmail.com]
> Sent: den 17 maj 2010 23:36
> To: Maven Developers List
> Subject: Re: [VOTE] Release Maven install plugin version 2.3.1
>
> +1
>
> On 17 May 2010 23:58, Kristian Rosenvold
> <kr...@gmail.com>wrote:
>
> > Hi,
> >
> > We solved 1 issue:
> >
> >
>
> http://jira.codehaus.org/secure/ReleaseNote.jspa?projectId=11136&version=164
> 59
> >
> > There are still a couple of issues left in JIRA:
> >
> >
>
> http://jira.codehaus.org/secure/IssueNavigator.jspa?reset=true&mode=hide&jql
>
> Query=project+%3D+MINSTALL+AND+resolution+%3D+Unresolved+ORDER+BY+updated+DE<http://jira.codehaus.org/secure/IssueNavigator.jspa?reset=true&mode=hide&jql%0AQuery=project+%3D+MINSTALL+AND+resolution+%3D+Unresolved+ORDER+BY+updated+DE>
> SC
> >
> > Staging repo:
> > https://repository.apache.org/content/repositories/maven-039/
> >
> > Staging site:
> > http://maven.apache.org/plugins/maven-install-plugin-2.3.1/
> >
> > Guide to testing staged releases:
> > http://maven.apache.org/guides/development/guide-testing-releases.html
> >
> > Vote open for 72 hours.
> >
> > [ ] +1
> > [ ] +0
> > [ ] -1
> >
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> > For additional commands, e-mail: dev-help@maven.apache.org
> >
> >
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> For additional commands, e-mail: dev-help@maven.apache.org
>
>
RE: [VOTE] Release Maven install plugin version 2.3.1
Posted by Ludwig Magnusson <lu...@itcatapult.com>.
I don't have an official say in this but I recently submitted a patch for an
issue I created that allows for a "skip" parameter to the install plugin. I
assigned it to the 2.4 version (2.3.1 did not exist in jira at the time). Is
it possible to add this patch or is it too much work?
Check it out here:
http://jira.codehaus.org/browse/MINSTALL-73
/Ludwig
-----Original Message-----
From: Stephen Connolly [mailto:stephen.alan.connolly@gmail.com]
Sent: den 17 maj 2010 23:36
To: Maven Developers List
Subject: Re: [VOTE] Release Maven install plugin version 2.3.1
+1
On 17 May 2010 23:58, Kristian Rosenvold
<kr...@gmail.com>wrote:
> Hi,
>
> We solved 1 issue:
>
>
http://jira.codehaus.org/secure/ReleaseNote.jspa?projectId=11136&version=164
59
>
> There are still a couple of issues left in JIRA:
>
>
http://jira.codehaus.org/secure/IssueNavigator.jspa?reset=true&mode=hide&jql
Query=project+%3D+MINSTALL+AND+resolution+%3D+Unresolved+ORDER+BY+updated+DE
SC
>
> Staging repo:
> https://repository.apache.org/content/repositories/maven-039/
>
> Staging site:
> http://maven.apache.org/plugins/maven-install-plugin-2.3.1/
>
> Guide to testing staged releases:
> http://maven.apache.org/guides/development/guide-testing-releases.html
>
> Vote open for 72 hours.
>
> [ ] +1
> [ ] +0
> [ ] -1
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> For additional commands, e-mail: dev-help@maven.apache.org
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org