My bad, I misunderstood your svn comment. If plexus-utils is used directly by the classes it should be a dependency.
Olivier Lamy wrote: > No issue fixed here. > My only concerns was to add a dependency on p-u because p-u is used > directly in some classes of the project. > I know p-u comes here by transitivity from the other dependencies. > It's just to declare something we use directly in the project. > Sure not mandatory, do you want I remove this ? > > -- > Olivier > > 2007/12/22, Dennis Lundberg <[EMAIL PROTECTED]>: >> Which of the declared dependencies is the problem? >> Did you fix the problem there too? >> >> [EMAIL PROTECTED] wrote: >>> Author: olamy >>> Date: Wed Dec 19 14:32:51 2007 >>> New Revision: 605723 >>> >>> URL: http://svn.apache.org/viewvc?rev=605723&view=rev >>> Log: >>> add dependency on p-u which it's used but not declared in the dependencies >>> >>> Modified: >>> maven/shared/trunk/maven-archiver/pom.xml >>> >>> Modified: maven/shared/trunk/maven-archiver/pom.xml >>> URL: >>> http://svn.apache.org/viewvc/maven/shared/trunk/maven-archiver/pom.xml?rev=605723&r1=605722&r2=605723&view=diff >>> ============================================================================== >>> --- maven/shared/trunk/maven-archiver/pom.xml (original) >>> +++ maven/shared/trunk/maven-archiver/pom.xml Wed Dec 19 14:32:51 2007 >>> @@ -44,6 +44,11 @@ >>> <artifactId>maven-artifact</artifactId> >>> <version>2.0</version> >>> </dependency> >>> + <dependency> >>> + <groupId>org.codehaus.plexus</groupId> >>> + <artifactId>plexus-utils</artifactId> >>> + <version>1.4.9</version> >>> + </dependency> >>> </dependencies> >>> <scm> >>> >>> <connection>scm:svn:http://svn.apache.org/repos/asf/maven/shared/trunk/maven-archiver</connection> >>> >>> >>> >> >> -- >> Dennis Lundberg >> >> --------------------------------------------------------------------- >> 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] > > -- Dennis Lundberg --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
