Yes, we should add it temporarily. I normally wouldn't use a snapshot but the new archiver has a feature needed by mdep and they asked me to test it before they do a release. It should be very short term.
-----Original Message----- From: Kenney Westerhof [mailto:[EMAIL PROTECTED] Sent: Friday, June 15, 2007 7:30 AM To: Maven Developers List Subject: Re: svn commit: r547483 - in /maven/plugins/trunk/maven-dependency-plugin: ./ src/test/java/org/apache/maven/plugin/dependency/ src/test/java/org/apache/maven/plugin/dependency/testUtils/ src/test/java/org/apache/maven/plugin/dependency/utils/filters Yup, I did the same change locally to test something. Normally we don't want to depend on plexus snapshots since it imposes a restriction on the ability to release. I think we should add the repo in the pom where we need a plexus snapshot, and remove it as soon as the dependency is released. Putting it in a parent or root pom will promote use of snapshots. my 2 cents, Kenney Mark Hobson wrote: > On 15/06/07, [EMAIL PROTECTED] <[EMAIL PROTECTED]> wrote: >> Author: brianf >> Date: Thu Jun 14 18:50:14 2007 >> New Revision: 547483 >> >> URL: http://svn.apache.org/viewvc?view=rev&rev=547483 >> Log: >> MDEP-59: updated dependencies to get the new unarchiver for rar. Had >> to move around the tests to avoid new heap size issues. >> >> Added: >> >> maven/plugins/trunk/maven-dependency-plugin/src/test/java/org/apache/maven/plugin/dependency/TestUnpackDependenciesMojo2.java >> >> >> Modified: >> maven/plugins/trunk/maven-dependency-plugin/pom.xml >> >> maven/plugins/trunk/maven-dependency-plugin/src/test/java/org/apache/maven/plugin/dependency/TestUnpackDependenciesMojo.java >> >> >> >> maven/plugins/trunk/maven-dependency-plugin/src/test/java/org/apache/maven/plugin/dependency/testUtils/ArtifactStubFactory.java >> >> >> >> maven/plugins/trunk/maven-dependency-plugin/src/test/java/org/apache/maven/plugin/dependency/utils/filters/TestTypeFilter.java >> >> >> >> Modified: maven/plugins/trunk/maven-dependency-plugin/pom.xml >> URL: >> http://svn.apache.org/viewvc/maven/plugins/trunk/maven-dependency-plugin/pom.xml?view=diff&rev=547483&r1=547482&r2=547483 >> >> >> ============================================================================== >> >> >> --- maven/plugins/trunk/maven-dependency-plugin/pom.xml (original) >> +++ maven/plugins/trunk/maven-dependency-plugin/pom.xml Thu Jun 14 >> 18:50:14 2007 >> @@ -138,13 +138,18 @@ >> <groupId>org.apache.maven.shared</groupId> >> >> <artifactId>maven-plugin-testing-harness</artifactId> >> <scope>test</scope> >> - <version>1.0-beta-1</version> >> + <version>1.1-SNAPSHOT</version> >> </dependency> >> <dependency> >> <groupId>org.codehaus.plexus</groupId> >> <artifactId>plexus-archiver</artifactId> >> - <version>1.0-alpha-8</version> >> + <version>1.0-alpha-9-SNAPSHOT</version> > [snip] > > Should we not add the codehaus snapshot repo to the poms somewhere? A > fresh checkout doesn't build since it can't find plexus snapshots. > > Mark > > --------------------------------------------------------------------- > 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]
