You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by be...@apache.org on 2008/11/20 16:25:02 UTC
svn commit: r719253 -
/maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/builder/ArtifactModelContainerFactory.java
Author: bentmann
Date: Thu Nov 20 07:25:01 2008
New Revision: 719253
URL: http://svn.apache.org/viewvc?rev=719253&view=rev
Log:
[MNG-3863] Maven erroneously assumes "org.apache.maven.plugins" for dependencies without groupId
Modified:
maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/builder/ArtifactModelContainerFactory.java
Modified: maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/builder/ArtifactModelContainerFactory.java
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/builder/ArtifactModelContainerFactory.java?rev=719253&r1=719252&r2=719253&view=diff
==============================================================================
--- maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/builder/ArtifactModelContainerFactory.java (original)
+++ maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/builder/ArtifactModelContainerFactory.java Thu Nov 20 07:25:01 2008
@@ -123,9 +123,17 @@
}
if ( groupId == null )
{
- groupId = "org.apache.maven.plugins";
- // throw new IllegalArgumentException("properties does not contain group id. Artifact ID = "
- // + artifactId + ", Version = " + version);
+ if ( ProjectUri.Build.Plugins.Plugin.xUri.equals( uri )
+ || ProjectUri.Build.PluginManagement.Plugins.Plugin.xUri.equals( uri )
+ || ProjectUri.Reporting.Plugins.Plugin.xUri.equals( uri ) )
+ {
+ groupId = "org.apache.maven.plugins";
+ }
+ else
+ {
+ throw new IllegalArgumentException( "Properties do not contain group id. Artifact ID = "
+ + artifactId + ", Version = " + version );
+ }
}
if ( artifactId == null )