You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by Dennis Lundberg <de...@apache.org> on 2007/12/22 10:06:05 UTC
Re: svn commit: r605723 - /maven/shared/trunk/maven-archiver/pom.xml
Which of the declared dependencies is the problem?
Did you fix the problem there too?
olamy@apache.org 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: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
Re: svn commit: r605723 - /maven/shared/trunk/maven-archiver/pom.xml
Posted by Dennis Lundberg <de...@apache.org>.
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 <de...@apache.org>:
>> Which of the declared dependencies is the problem?
>> Did you fix the problem there too?
>>
>> olamy@apache.org 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: 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
>
>
--
Dennis Lundberg
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
Re: svn commit: r605723 - /maven/shared/trunk/maven-archiver/pom.xml
Posted by Olivier Lamy <ol...@apache.org>.
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 <de...@apache.org>:
> Which of the declared dependencies is the problem?
> Did you fix the problem there too?
>
> olamy@apache.org 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: 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