svn commit: r594300 - /maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java

[jd...@apache.org]

Author: jdcasey
Date: Mon Nov 12 13:17:00 2007
New Revision: 594300

URL: http://svn.apache.org/viewvc?rev=594300&view=rev
Log:
Moving a comment to a place where it will be more useful.

Modified:
    maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java

Modified: maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java?rev=594300&r1=594299&r2=594300&view=diff
==============================================================================
--- maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java (original)
+++ maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java Mon Nov 12 13:17:00 2007
@@ -275,7 +275,6 @@
         // this snippet of code here.
         // ----------------------------------------------------------------------
 
-        // TODO: such a call in MavenMetadataSource too - packaging not really the intention of type
         Artifact projectArtifact = project.getArtifact();
 
         String projectId = safeVersionlessKey( project.getGroupId(), project.getArtifactId() );
@@ -852,6 +851,7 @@
 
         project.setActiveProfiles( activeProfiles );
 
+        // TODO: such a call in MavenMetadataSource too - packaging not really the intention of type
         // TODO: maybe not strictly correct, while we should enfore that packaging has a type handler of the same id, we don't
         Artifact projectArtifact = artifactFactory.createBuildArtifact( project.getGroupId(), project.getArtifactId(),
             project.getVersion(), project.getPackaging() );