Re: [VOTE] Release Maven Ant Tasks 2.0.7
Le vendredi 29 juin 2007, Hervé BOUTEMY a écrit : Le vendredi 29 juin 2007, Hervé BOUTEMY a écrit : patch MANTTASKS-76_site-release.diff added to MANTTASKS-76 added patch MANTTASKS-76_license.diff too, while we're at it... are these patches ok to get the release out, or should I change something? Hervé Ready for the release! Hervé Le jeudi 28 juin 2007, Jason van Zyl a écrit : On 28 Jun 07, at 1:54 PM 28 Jun 07, Hervé BOUTEMY wrote: Le jeudi 28 juin 2007, Jason van Zyl a écrit : Thanks Herve, I forgot about adding that even though you reminded me last night. Think you might make a site patch for the release notes regarding these issues? Do you mean adding release notes for Maven Ant Tasks like for Maven Core http://maven.apache.org/release-notes.html ? The change for localRepository is documented at the end of the doc, in Type ReferencelocalRepository section. But it is not really easy to find it... We should probably make separate release notes for the ant tasks. On 27 Jun 07, at 11:29 PM 27 Jun 07, Hervé BOUTEMY wrote: Hi, Just a little note on this release. There are a lot of bugfixes (SNAPSHOT downloading, for example), new features (VersionMapper, remoteRepository support for pom and install- provider tasks) documented on http://maven.apache.org/ant-tasks.html. But there are 2 incompatible changes that you must know before upgrading, to fix your Ant build files: 1. MANTTASKS-44: localRepository path=.../ instead of localRepository location=/ for Ant 1.7 compatibility (location attribute is used by Ant now) 2. MANTTASKS-65: central is not automatically added any more if a remoteRepository is set: the code has been changed to work as the documentation said Everything else should work as previously (with less bugs) Regards Hervé Le jeudi 28 juin 2007, Jason van Zyl a écrit : Hi, Repost of the vote after fixing the issues with the uber JAR that was produced. Hervé and I have tried them and they appear to work fine. The staging repository is here: http://people.apache.org/~jvanzyl/maven-ant-tasks-2.0.7-staging- repository/ The uber jar that people will want to try is here: http://people.apache.org/~jvanzyl/maven-ant-tasks-2.0.7-staging- repository/org/apache/maven/maven-ant-tasks/2.0.7/ Road map: http://jira.codehaus.org/secure/ReleaseNote.jspa? projectId=11533styleName=Htmlversion=13521 Thanks, Jason -- Jason van Zyl Founder and PMC Chair, Apache Maven jason at sonatype dot com -- -- - -- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] --- - - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] Thanks, Jason -- Jason van Zyl Founder and PMC Chair, Apache Maven jason at sonatype dot com -- - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] Thanks, Jason -- Jason van Zyl Founder and PMC Chair, Apache Maven jason at sonatype dot com -- - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [VOTE] Release Maven Ant Tasks 2.0.7
On 2 Jul 07, at 2:20 PM 2 Jul 07, Hervé BOUTEMY wrote: Le vendredi 29 juin 2007, Hervé BOUTEMY a écrit : Le vendredi 29 juin 2007, Hervé BOUTEMY a écrit : patch MANTTASKS-76_site-release.diff added to MANTTASKS-76 added patch MANTTASKS-76_license.diff too, while we're at it... are these patches ok to get the release out, or should I change something? Yup, I will do it tonight. Hervé Ready for the release! Hervé Le jeudi 28 juin 2007, Jason van Zyl a écrit : On 28 Jun 07, at 1:54 PM 28 Jun 07, Hervé BOUTEMY wrote: Le jeudi 28 juin 2007, Jason van Zyl a écrit : Thanks Herve, I forgot about adding that even though you reminded me last night. Think you might make a site patch for the release notes regarding these issues? Do you mean adding release notes for Maven Ant Tasks like for Maven Core http://maven.apache.org/release-notes.html ? The change for localRepository is documented at the end of the doc, in Type ReferencelocalRepository section. But it is not really easy to find it... We should probably make separate release notes for the ant tasks. On 27 Jun 07, at 11:29 PM 27 Jun 07, Hervé BOUTEMY wrote: Hi, Just a little note on this release. There are a lot of bugfixes (SNAPSHOT downloading, for example), new features (VersionMapper, remoteRepository support for pom and install- provider tasks) documented on http://maven.apache.org/ant-tasks.html. But there are 2 incompatible changes that you must know before upgrading, to fix your Ant build files: 1. MANTTASKS-44: localRepository path=.../ instead of localRepository location=/ for Ant 1.7 compatibility (location attribute is used by Ant now) 2. MANTTASKS-65: central is not automatically added any more if a remoteRepository is set: the code has been changed to work as the documentation said Everything else should work as previously (with less bugs) Regards Hervé Le jeudi 28 juin 2007, Jason van Zyl a écrit : Hi, Repost of the vote after fixing the issues with the uber JAR that was produced. Hervé and I have tried them and they appear to work fine. The staging repository is here: http://people.apache.org/~jvanzyl/maven-ant-tasks-2.0.7- staging- repository/ The uber jar that people will want to try is here: http://people.apache.org/~jvanzyl/maven-ant-tasks-2.0.7- staging- repository/org/apache/maven/maven-ant-tasks/2.0.7/ Road map: http://jira.codehaus.org/secure/ReleaseNote.jspa? projectId=11533styleName=Htmlversion=13521 Thanks, Jason -- Jason van Zyl Founder and PMC Chair, Apache Maven jason at sonatype dot com -- --- --- - -- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] --- - - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] Thanks, Jason -- Jason van Zyl Founder and PMC Chair, Apache Maven jason at sonatype dot com -- - --- - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] -- --- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] Thanks, Jason -- Jason van Zyl Founder and PMC Chair, Apache Maven jason at sonatype dot com -- --- -- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] Thanks, Jason -- Jason van Zyl Founder and PMC Chair, Apache Maven jason at sonatype dot com -- - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [VOTE] Release Maven Ant Tasks 2.0.7
patch MANTTASKS-76_site-release.diff added to MANTTASKS-76 Ready for the release! Hervé Le jeudi 28 juin 2007, Jason van Zyl a écrit : On 28 Jun 07, at 1:54 PM 28 Jun 07, Hervé BOUTEMY wrote: Le jeudi 28 juin 2007, Jason van Zyl a écrit : Thanks Herve, I forgot about adding that even though you reminded me last night. Think you might make a site patch for the release notes regarding these issues? Do you mean adding release notes for Maven Ant Tasks like for Maven Core http://maven.apache.org/release-notes.html ? The change for localRepository is documented at the end of the doc, in Type ReferencelocalRepository section. But it is not really easy to find it... We should probably make separate release notes for the ant tasks. On 27 Jun 07, at 11:29 PM 27 Jun 07, Hervé BOUTEMY wrote: Hi, Just a little note on this release. There are a lot of bugfixes (SNAPSHOT downloading, for example), new features (VersionMapper, remoteRepository support for pom and install- provider tasks) documented on http://maven.apache.org/ant-tasks.html. But there are 2 incompatible changes that you must know before upgrading, to fix your Ant build files: 1. MANTTASKS-44: localRepository path=.../ instead of localRepository location=/ for Ant 1.7 compatibility (location attribute is used by Ant now) 2. MANTTASKS-65: central is not automatically added any more if a remoteRepository is set: the code has been changed to work as the documentation said Everything else should work as previously (with less bugs) Regards Hervé Le jeudi 28 juin 2007, Jason van Zyl a écrit : Hi, Repost of the vote after fixing the issues with the uber JAR that was produced. Hervé and I have tried them and they appear to work fine. The staging repository is here: http://people.apache.org/~jvanzyl/maven-ant-tasks-2.0.7-staging- repository/ The uber jar that people will want to try is here: http://people.apache.org/~jvanzyl/maven-ant-tasks-2.0.7-staging- repository/org/apache/maven/maven-ant-tasks/2.0.7/ Road map: http://jira.codehaus.org/secure/ReleaseNote.jspa? projectId=11533styleName=Htmlversion=13521 Thanks, Jason -- Jason van Zyl Founder and PMC Chair, Apache Maven jason at sonatype dot com -- --- -- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] Thanks, Jason -- Jason van Zyl Founder and PMC Chair, Apache Maven jason at sonatype dot com -- - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] Thanks, Jason -- Jason van Zyl Founder and PMC Chair, Apache Maven jason at sonatype dot com -- - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [VOTE] Release Maven Ant Tasks 2.0.7
That makes sense. I don't use this much myself, but I was concerned that it may pop up as an inconsistency in Mavenland. It's a good point to remember that we're not dealing with strictly Maven users, though. -john On Jun 28, 2007, at 4:53 PM, Jason van Zyl wrote: On 28 Jun 07, at 1:33 PM 28 Jun 07, Hervé BOUTEMY wrote: Le jeudi 28 juin 2007, John Casey a écrit : On Jun 28, 2007, at 2:29 AM, Hervé BOUTEMY wrote: 2. MANTTASKS-65: central is not automatically added any more if a remoteRepository is set: the code has been changed to work as the documentation said This seems inconsistent with the way Maven as a whole works. Are you saying that the central repository is not used if you specify an alternative repository in your build.xml? Yes, exactly The super/root POM in a normal Maven build ensures that the central repository is always used, unless it's mirrored to a different location in the settings.xml. It may not be the first repository checked, but it's always there. I don't think this is bad in Ant where there is little magic and things are entirely procedural. I think having to specify anything used is more natural to people who use Ant. What's the reasoning behind making this work differently, rather than just fixing the documentation? good question. I remember having asked whether I should fix the code or the documentation, and was replied to fix the code. But I can't find any more a reference of where I asked, and who replied, and why I was confident in this reply... Today I'd choose the other solution: fix the doc (and since then, I added to the tasks' doc explainations about mirrorOf). Should we stop the release and have a vote on this? I don't think so. I think the explicit specification is a good thing when using Ant myself. Hervé Aside from this, I'm +1. -john --- John Casey Committer and PMC Member, Apache Maven mail: jdcasey at commonjava dot org blog: http://www.ejlife.net/blogs/john - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] Thanks, Jason -- Jason van Zyl Founder and PMC Chair, Apache Maven jason at sonatype dot com -- - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] --- John Casey Committer and PMC Member, Apache Maven mail: jdcasey at commonjava dot org blog: http://www.ejlife.net/blogs/john
Re: [VOTE] Release Maven Ant Tasks 2.0.7
Hi, Just a little note on this release. There are a lot of bugfixes (SNAPSHOT downloading, for example), new features (VersionMapper, remoteRepository support for pom and install-provider tasks) documented on http://maven.apache.org/ant-tasks.html. But there are 2 incompatible changes that you must know before upgrading, to fix your Ant build files: 1. MANTTASKS-44: localRepository path=.../ instead of localRepository location=/ for Ant 1.7 compatibility (location attribute is used by Ant now) 2. MANTTASKS-65: central is not automatically added any more if a remoteRepository is set: the code has been changed to work as the documentation said Everything else should work as previously (with less bugs) Regards Hervé Le jeudi 28 juin 2007, Jason van Zyl a écrit : Hi, Repost of the vote after fixing the issues with the uber JAR that was produced. Hervé and I have tried them and they appear to work fine. The staging repository is here: http://people.apache.org/~jvanzyl/maven-ant-tasks-2.0.7-staging- repository/ The uber jar that people will want to try is here: http://people.apache.org/~jvanzyl/maven-ant-tasks-2.0.7-staging- repository/org/apache/maven/maven-ant-tasks/2.0.7/ Road map: http://jira.codehaus.org/secure/ReleaseNote.jspa? projectId=11533styleName=Htmlversion=13521 Thanks, Jason -- Jason van Zyl Founder and PMC Chair, Apache Maven jason at sonatype dot com -- - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [VOTE] Release Maven Ant Tasks 2.0.7
Thanks Herve, I forgot about adding that even though you reminded me last night. Think you might make a site patch for the release notes regarding these issues? On 27 Jun 07, at 11:29 PM 27 Jun 07, Hervé BOUTEMY wrote: Hi, Just a little note on this release. There are a lot of bugfixes (SNAPSHOT downloading, for example), new features (VersionMapper, remoteRepository support for pom and install- provider tasks) documented on http://maven.apache.org/ant-tasks.html. But there are 2 incompatible changes that you must know before upgrading, to fix your Ant build files: 1. MANTTASKS-44: localRepository path=.../ instead of localRepository location=/ for Ant 1.7 compatibility (location attribute is used by Ant now) 2. MANTTASKS-65: central is not automatically added any more if a remoteRepository is set: the code has been changed to work as the documentation said Everything else should work as previously (with less bugs) Regards Hervé Le jeudi 28 juin 2007, Jason van Zyl a écrit : Hi, Repost of the vote after fixing the issues with the uber JAR that was produced. Hervé and I have tried them and they appear to work fine. The staging repository is here: http://people.apache.org/~jvanzyl/maven-ant-tasks-2.0.7-staging- repository/ The uber jar that people will want to try is here: http://people.apache.org/~jvanzyl/maven-ant-tasks-2.0.7-staging- repository/org/apache/maven/maven-ant-tasks/2.0.7/ Road map: http://jira.codehaus.org/secure/ReleaseNote.jspa? projectId=11533styleName=Htmlversion=13521 Thanks, Jason -- Jason van Zyl Founder and PMC Chair, Apache Maven jason at sonatype dot com -- - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] Thanks, Jason -- Jason van Zyl Founder and PMC Chair, Apache Maven jason at sonatype dot com -- - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [VOTE] Release Maven Ant Tasks 2.0.7
On Jun 28, 2007, at 2:29 AM, Hervé BOUTEMY wrote: 2. MANTTASKS-65: central is not automatically added any more if a remoteRepository is set: the code has been changed to work as the documentation said This seems inconsistent with the way Maven as a whole works. Are you saying that the central repository is not used if you specify an alternative repository in your build.xml? The super/root POM in a normal Maven build ensures that the central repository is always used, unless it's mirrored to a different location in the settings.xml. It may not be the first repository checked, but it's always there. What's the reasoning behind making this work differently, rather than just fixing the documentation? Aside from this, I'm +1. -john --- John Casey Committer and PMC Member, Apache Maven mail: jdcasey at commonjava dot org blog: http://www.ejlife.net/blogs/john
Re: [VOTE] Release Maven Ant Tasks 2.0.7
+1, as long as the release notes include the notes mentioned. Also, for future releases, the POM and one source file have an outdated license: pom.xml: ~ Copyright 2005-2006 The Apache Software Foundation. src/main/java/org/apache/maven/artifact/ant/VersionMapper.java: * Copyright 2001-2007 The Apache Software Foundation. It should be the Licensed to... header that is on the other source files. Cheers, Brett On 28/06/2007, at 4:15 PM, Jason van Zyl wrote: Hi, Repost of the vote after fixing the issues with the uber JAR that was produced. Hervé and I have tried them and they appear to work fine. The staging repository is here: http://people.apache.org/~jvanzyl/maven-ant-tasks-2.0.7-staging- repository/ The uber jar that people will want to try is here: http://people.apache.org/~jvanzyl/maven-ant-tasks-2.0.7-staging- repository/org/apache/maven/maven-ant-tasks/2.0.7/ Road map: http://jira.codehaus.org/secure/ReleaseNote.jspa? projectId=11533styleName=Htmlversion=13521 Thanks, Jason -- Jason van Zyl Founder and PMC Chair, Apache Maven jason at sonatype dot com -- - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [VOTE] Release Maven Ant Tasks 2.0.7
Le jeudi 28 juin 2007, John Casey a écrit : On Jun 28, 2007, at 2:29 AM, Hervé BOUTEMY wrote: 2. MANTTASKS-65: central is not automatically added any more if a remoteRepository is set: the code has been changed to work as the documentation said This seems inconsistent with the way Maven as a whole works. Are you saying that the central repository is not used if you specify an alternative repository in your build.xml? Yes, exactly The super/root POM in a normal Maven build ensures that the central repository is always used, unless it's mirrored to a different location in the settings.xml. It may not be the first repository checked, but it's always there. What's the reasoning behind making this work differently, rather than just fixing the documentation? good question. I remember having asked whether I should fix the code or the documentation, and was replied to fix the code. But I can't find any more a reference of where I asked, and who replied, and why I was confident in this reply... Today I'd choose the other solution: fix the doc (and since then, I added to the tasks' doc explainations about mirrorOf). Should we stop the release and have a vote on this? Hervé Aside from this, I'm +1. -john --- John Casey Committer and PMC Member, Apache Maven mail: jdcasey at commonjava dot org blog: http://www.ejlife.net/blogs/john - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [VOTE] Release Maven Ant Tasks 2.0.7
On 28 Jun 07, at 1:33 PM 28 Jun 07, Hervé BOUTEMY wrote: Le jeudi 28 juin 2007, John Casey a écrit : On Jun 28, 2007, at 2:29 AM, Hervé BOUTEMY wrote: 2. MANTTASKS-65: central is not automatically added any more if a remoteRepository is set: the code has been changed to work as the documentation said This seems inconsistent with the way Maven as a whole works. Are you saying that the central repository is not used if you specify an alternative repository in your build.xml? Yes, exactly The super/root POM in a normal Maven build ensures that the central repository is always used, unless it's mirrored to a different location in the settings.xml. It may not be the first repository checked, but it's always there. I don't think this is bad in Ant where there is little magic and things are entirely procedural. I think having to specify anything used is more natural to people who use Ant. What's the reasoning behind making this work differently, rather than just fixing the documentation? good question. I remember having asked whether I should fix the code or the documentation, and was replied to fix the code. But I can't find any more a reference of where I asked, and who replied, and why I was confident in this reply... Today I'd choose the other solution: fix the doc (and since then, I added to the tasks' doc explainations about mirrorOf). Should we stop the release and have a vote on this? I don't think so. I think the explicit specification is a good thing when using Ant myself. Hervé Aside from this, I'm +1. -john --- John Casey Committer and PMC Member, Apache Maven mail: jdcasey at commonjava dot org blog: http://www.ejlife.net/blogs/john - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] Thanks, Jason -- Jason van Zyl Founder and PMC Chair, Apache Maven jason at sonatype dot com -- - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [VOTE] Release Maven Ant Tasks 2.0.7
Le jeudi 28 juin 2007, Jason van Zyl a écrit : Thanks Herve, I forgot about adding that even though you reminded me last night. Think you might make a site patch for the release notes regarding these issues? Do you mean adding release notes for Maven Ant Tasks like for Maven Core http://maven.apache.org/release-notes.html ? The change for localRepository is documented at the end of the doc, in Type ReferencelocalRepository section. But it is not really easy to find it... On 27 Jun 07, at 11:29 PM 27 Jun 07, Hervé BOUTEMY wrote: Hi, Just a little note on this release. There are a lot of bugfixes (SNAPSHOT downloading, for example), new features (VersionMapper, remoteRepository support for pom and install- provider tasks) documented on http://maven.apache.org/ant-tasks.html. But there are 2 incompatible changes that you must know before upgrading, to fix your Ant build files: 1. MANTTASKS-44: localRepository path=.../ instead of localRepository location=/ for Ant 1.7 compatibility (location attribute is used by Ant now) 2. MANTTASKS-65: central is not automatically added any more if a remoteRepository is set: the code has been changed to work as the documentation said Everything else should work as previously (with less bugs) Regards Hervé Le jeudi 28 juin 2007, Jason van Zyl a écrit : Hi, Repost of the vote after fixing the issues with the uber JAR that was produced. Hervé and I have tried them and they appear to work fine. The staging repository is here: http://people.apache.org/~jvanzyl/maven-ant-tasks-2.0.7-staging- repository/ The uber jar that people will want to try is here: http://people.apache.org/~jvanzyl/maven-ant-tasks-2.0.7-staging- repository/org/apache/maven/maven-ant-tasks/2.0.7/ Road map: http://jira.codehaus.org/secure/ReleaseNote.jspa? projectId=11533styleName=Htmlversion=13521 Thanks, Jason -- Jason van Zyl Founder and PMC Chair, Apache Maven jason at sonatype dot com -- - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] Thanks, Jason -- Jason van Zyl Founder and PMC Chair, Apache Maven jason at sonatype dot com -- - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [VOTE] Release Maven Ant Tasks 2.0.7
Le jeudi 28 juin 2007, Jason van Zyl a écrit : On 28 Jun 07, at 1:33 PM 28 Jun 07, Hervé BOUTEMY wrote: Le jeudi 28 juin 2007, John Casey a écrit : On Jun 28, 2007, at 2:29 AM, Hervé BOUTEMY wrote: 2. MANTTASKS-65: central is not automatically added any more if a remoteRepository is set: the code has been changed to work as the documentation said This seems inconsistent with the way Maven as a whole works. Are you saying that the central repository is not used if you specify an alternative repository in your build.xml? Yes, exactly The super/root POM in a normal Maven build ensures that the central repository is always used, unless it's mirrored to a different location in the settings.xml. It may not be the first repository checked, but it's always there. I don't think this is bad in Ant where there is little magic and things are entirely procedural. I think having to specify anything used is more natural to people who use Ant. What's the reasoning behind making this work differently, rather than just fixing the documentation? good question. I remember having asked whether I should fix the code or the documentation, and was replied to fix the code. But I can't find any more a reference of where I asked, and who replied, and why I was confident in this reply... Today I'd choose the other solution: fix the doc (and since then, I added to the tasks' doc explainations about mirrorOf). Should we stop the release and have a vote on this? I don't think so. I think the explicit specification is a good thing when using Ant myself. I remember now who said me to fix the code, and why I didn't ask why :) Hervé Aside from this, I'm +1. -john --- John Casey Committer and PMC Member, Apache Maven mail: jdcasey at commonjava dot org blog: http://www.ejlife.net/blogs/john - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] Thanks, Jason -- Jason van Zyl Founder and PMC Chair, Apache Maven jason at sonatype dot com -- - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [VOTE] Release Maven Ant Tasks 2.0.7
On 28 Jun 07, at 1:54 PM 28 Jun 07, Hervé BOUTEMY wrote: Le jeudi 28 juin 2007, Jason van Zyl a écrit : Thanks Herve, I forgot about adding that even though you reminded me last night. Think you might make a site patch for the release notes regarding these issues? Do you mean adding release notes for Maven Ant Tasks like for Maven Core http://maven.apache.org/release-notes.html ? The change for localRepository is documented at the end of the doc, in Type ReferencelocalRepository section. But it is not really easy to find it... We should probably make separate release notes for the ant tasks. On 27 Jun 07, at 11:29 PM 27 Jun 07, Hervé BOUTEMY wrote: Hi, Just a little note on this release. There are a lot of bugfixes (SNAPSHOT downloading, for example), new features (VersionMapper, remoteRepository support for pom and install- provider tasks) documented on http://maven.apache.org/ant-tasks.html. But there are 2 incompatible changes that you must know before upgrading, to fix your Ant build files: 1. MANTTASKS-44: localRepository path=.../ instead of localRepository location=/ for Ant 1.7 compatibility (location attribute is used by Ant now) 2. MANTTASKS-65: central is not automatically added any more if a remoteRepository is set: the code has been changed to work as the documentation said Everything else should work as previously (with less bugs) Regards Hervé Le jeudi 28 juin 2007, Jason van Zyl a écrit : Hi, Repost of the vote after fixing the issues with the uber JAR that was produced. Hervé and I have tried them and they appear to work fine. The staging repository is here: http://people.apache.org/~jvanzyl/maven-ant-tasks-2.0.7-staging- repository/ The uber jar that people will want to try is here: http://people.apache.org/~jvanzyl/maven-ant-tasks-2.0.7-staging- repository/org/apache/maven/maven-ant-tasks/2.0.7/ Road map: http://jira.codehaus.org/secure/ReleaseNote.jspa? projectId=11533styleName=Htmlversion=13521 Thanks, Jason -- Jason van Zyl Founder and PMC Chair, Apache Maven jason at sonatype dot com -- --- -- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] Thanks, Jason -- Jason van Zyl Founder and PMC Chair, Apache Maven jason at sonatype dot com -- - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] Thanks, Jason -- Jason van Zyl Founder and PMC Chair, Apache Maven jason at sonatype dot com -- - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]