|
»
»
MavenProject.addArtifact()
|
View:
New views
6 Messages
—
Rating Filter:
Alert me
|
 |
MavenProject.addArtifact()
by Vincent Massol
::
Rate this Message:
Hi,
Is it possible to have a Maven.addArtifact() method added? Someone reported a bug on the clover plugin saying that a dependency was missing on one project. Here's the code that we currently have: Set set = new HashSet( this.project.getDependencyArtifacts() ); set.add( cloverArtifact ); this.project.setDependencyArtifacts( set ); I'd rather replace this with a addArtifact() call as the setDependencyArtrifacts() call above sounds dangerous (we could omit artifacts). It appears that this is probably the problem we have. Anyone has any idea what type of artifact we could be missing with the code above? The person's code builds fine with a normal build; it's just when clover plugin is run that it fails. Thanks -Vincent ___________________________________________________________________________ Nouveau : téléphonez moins cher avec Yahoo! Messenger ! Découvez les tarifs exceptionnels pour appeler la France et l'international. Téléchargez sur http://fr.messenger.yahoo.com --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscribe@... For additional commands, e-mail: dev-help@... |
|
|
Re: MavenProject.addArtifact()
by brettporter
::
Rate this Message:
That code looks fine, it also works in Cobertura. Can you get more
details about the dependency? - Brett Vincent Massol wrote: > Hi, > > Is it possible to have a Maven.addArtifact() method added? > > Someone reported a bug on the clover plugin saying that a dependency was > missing on one project. Here's the code that we currently have: > > Set set = new HashSet( this.project.getDependencyArtifacts() ); > set.add( cloverArtifact ); > this.project.setDependencyArtifacts( set ); > > I'd rather replace this with a addArtifact() call as the > setDependencyArtrifacts() call above sounds dangerous (we could omit > artifacts). It appears that this is probably the problem we have. > > Anyone has any idea what type of artifact we could be missing with the code > above? The person's code builds fine with a normal build; it's just when > clover plugin is run that it fails. > > Thanks > -Vincent > > > > > > > ___________________________________________________________________________ > Nouveau : téléphonez moins cher avec Yahoo! Messenger ! Découvez les tarifs exceptionnels pour appeler la France et l'international. > Téléchargez sur http://fr.messenger.yahoo.com > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscribe@... > For additional commands, e-mail: dev-help@... > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscribe@... For additional commands, e-mail: dev-help@... |
|
|
RE: MavenProject.addArtifact()
by Vincent Massol
::
Rate this Message:
> -----Original Message----- > From: Brett Porter [mailto:brett@...] > Sent: vendredi 31 mars 2006 00:26 > To: Maven Developers List > Subject: Re: MavenProject.addArtifact() > > That code looks fine, it also works in Cobertura. Can you get more > details about the dependency? I got more information and apparently the missing dependency is brought transitively. Is it possible that the getDependencyArtifacts() below does not include transitive dependencies? Thanks -Vincent > Vincent Massol wrote: > > Hi, > > > > Is it possible to have a Maven.addArtifact() method added? > > > > Someone reported a bug on the clover plugin saying that a dependency was > > missing on one project. Here's the code that we currently have: > > > > Set set = new HashSet( this.project.getDependencyArtifacts() ); > > set.add( cloverArtifact ); > > this.project.setDependencyArtifacts( set ); > > > > I'd rather replace this with a addArtifact() call as the > > setDependencyArtrifacts() call above sounds dangerous (we could omit > > artifacts). It appears that this is probably the problem we have. > > > > Anyone has any idea what type of artifact we could be missing with the > code > > above? The person's code builds fine with a normal build; it's just when > > clover plugin is run that it fails. > > > > Thanks > > -Vincent > > > > > > > > > > > > > > > __________________________________________________________________________ > _ > > Nouveau : téléphonez moins cher avec Yahoo! Messenger ! Découvez les > tarifs exceptionnels pour appeler la France et l'international. > > Téléchargez sur http://fr.messenger.yahoo.com > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: dev-unsubscribe@... > > For additional commands, e-mail: dev-help@... > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscribe@... > For additional commands, e-mail: dev-help@... ___________________________________________________________________________ Nouveau : téléphonez moins cher avec Yahoo! Messenger ! Découvez les tarifs exceptionnels pour appeler la France et l'international. Téléchargez sur http://fr.messenger.yahoo.com --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscribe@... For additional commands, e-mail: dev-help@... |
|
|
Re: MavenProject.addArtifact()
by brettporter
::
Rate this Message:
That's correct. If you want the transitive dependencies, use getArtifacts().
However, for the setting, you need to setDependencyArtifacts due to the way they are constructed. An add artifact method would be very helpful in a future release. - Brett Vincent Massol wrote: > >> -----Original Message----- >> From: Brett Porter [mailto:brett@...] >> Sent: vendredi 31 mars 2006 00:26 >> To: Maven Developers List >> Subject: Re: MavenProject.addArtifact() >> >> That code looks fine, it also works in Cobertura. Can you get more >> details about the dependency? > > I got more information and apparently the missing dependency is brought > transitively. Is it possible that the getDependencyArtifacts() below does > not include transitive dependencies? > > Thanks > -Vincent > >> Vincent Massol wrote: >>> Hi, >>> >>> Is it possible to have a Maven.addArtifact() method added? >>> >>> Someone reported a bug on the clover plugin saying that a dependency was >>> missing on one project. Here's the code that we currently have: >>> >>> Set set = new HashSet( this.project.getDependencyArtifacts() ); >>> set.add( cloverArtifact ); >>> this.project.setDependencyArtifacts( set ); >>> >>> I'd rather replace this with a addArtifact() call as the >>> setDependencyArtrifacts() call above sounds dangerous (we could omit >>> artifacts). It appears that this is probably the problem we have. >>> >>> Anyone has any idea what type of artifact we could be missing with the >> code >>> above? The person's code builds fine with a normal build; it's just when >>> clover plugin is run that it fails. >>> >>> Thanks >>> -Vincent >>> >>> >>> >>> >>> >>> >>> >> __________________________________________________________________________ >> _ >>> Nouveau : téléphonez moins cher avec Yahoo! Messenger ! Découvez les >> tarifs exceptionnels pour appeler la France et l'international. >>> Téléchargez sur http://fr.messenger.yahoo.com >>> >>> --------------------------------------------------------------------- >>> To unsubscribe, e-mail: dev-unsubscribe@... >>> For additional commands, e-mail: dev-help@... >>> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: dev-unsubscribe@... >> For additional commands, e-mail: dev-help@... > > > > > > > ___________________________________________________________________________ > Nouveau : téléphonez moins cher avec Yahoo! Messenger ! Découvez les tarifs exceptionnels pour appeler la France et l'international. > Téléchargez sur http://fr.messenger.yahoo.com > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscribe@... > For additional commands, e-mail: dev-help@... > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscribe@... For additional commands, e-mail: dev-help@... |
|
|
RE: MavenProject.addArtifact()
by Vincent Massol
::
Rate this Message:
> -----Original Message----- > From: Brett Porter [mailto:brett@...] > Sent: samedi 1 avril 2006 10:36 > To: Maven Developers List > Subject: Re: MavenProject.addArtifact() > > That's correct. If you want the transitive dependencies, use > getArtifacts(). Does it mean that when using setDependencyArtifacts() the deps are no longer transitively resolved after that? > However, for the setting, you need to setDependencyArtifacts due to the > way they are constructed. > > An add artifact method would be very helpful in a future release. Added in jira: http://jira.codehaus.org/browse/MNG-2197 Thanks -Vincent > Vincent Massol wrote: > > > >> -----Original Message----- > >> From: Brett Porter [mailto:brett@...] > >> Sent: vendredi 31 mars 2006 00:26 > >> To: Maven Developers List > >> Subject: Re: MavenProject.addArtifact() > >> > >> That code looks fine, it also works in Cobertura. Can you get more > >> details about the dependency? > > > > I got more information and apparently the missing dependency is brought > > transitively. Is it possible that the getDependencyArtifacts() below > does > > not include transitive dependencies? > > > > Thanks > > -Vincent > > > >> Vincent Massol wrote: > >>> Hi, > >>> > >>> Is it possible to have a Maven.addArtifact() method added? > >>> > >>> Someone reported a bug on the clover plugin saying that a dependency > was > >>> missing on one project. Here's the code that we currently have: > >>> > >>> Set set = new HashSet( this.project.getDependencyArtifacts() > ); > >>> set.add( cloverArtifact ); > >>> this.project.setDependencyArtifacts( set ); > >>> > >>> I'd rather replace this with a addArtifact() call as the > >>> setDependencyArtrifacts() call above sounds dangerous (we could omit > >>> artifacts). It appears that this is probably the problem we have. > >>> > >>> Anyone has any idea what type of artifact we could be missing with the > >> code > >>> above? The person's code builds fine with a normal build; it's just > when > >>> clover plugin is run that it fails. > >>> > >>> Thanks > >>> -Vincent > >>> > >>> > >>> > >>> > >>> > >>> > >>> > >> > __________________________________________________________________________ > >> _ > >>> Nouveau : téléphonez moins cher avec Yahoo! Messenger ! Découvez les > >> tarifs exceptionnels pour appeler la France et l'international. > >>> Téléchargez sur http://fr.messenger.yahoo.com > >>> > >>> --------------------------------------------------------------------- > >>> To unsubscribe, e-mail: dev-unsubscribe@... > >>> For additional commands, e-mail: dev-help@... > >>> > >> --------------------------------------------------------------------- > >> To unsubscribe, e-mail: dev-unsubscribe@... > >> For additional commands, e-mail: dev-help@... > > > > > > > > > > > > > > > __________________________________________________________________________ > _ > > Nouveau : téléphonez moins cher avec Yahoo! Messenger ! Découvez les > tarifs exceptionnels pour appeler la France et l'international. > > Téléchargez sur http://fr.messenger.yahoo.com > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: dev-unsubscribe@... > > For additional commands, e-mail: dev-help@... > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscribe@... > For additional commands, e-mail: dev-help@... ___________________________________________________________________________ Nouveau : téléphonez moins cher avec Yahoo! Messenger ! Découvez les tarifs exceptionnels pour appeler la France et l'international. Téléchargez sur http://fr.messenger.yahoo.com --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscribe@... For additional commands, e-mail: dev-help@... |
|
|
Re: MavenProject.addArtifact()
by brettporter
::
Rate this Message:
Vincent Massol wrote:
> >> -----Original Message----- >> From: Brett Porter [mailto:brett@...] >> Sent: samedi 1 avril 2006 10:36 >> To: Maven Developers List >> Subject: Re: MavenProject.addArtifact() >> >> That's correct. If you want the transitive dependencies, use >> getArtifacts(). > > Does it mean that when using setDependencyArtifacts() the deps are no longer > transitively resolved after that? No, they are transitively resolved, but if you are using getDependencyArtifacts for anything, you only get the direct dependencies (which will include anything added to the list that is reset with setDependencyArtifacts). - Brett --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscribe@... For additional commands, e-mail: dev-help@... |
| Free embeddable forum powered by Nabble | Forum Help |
