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() );