Author: mkleint
Date: Sun Jul  2 10:53:27 2006
New Revision: 418629

URL: http://svn.apache.org/viewvc?rev=418629&view=rev
Log:
added comment about System.getProperties

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=418629&r1=418628&r2=418629&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
 Sun Jul  2 10:53:27 2006
@@ -865,6 +865,7 @@
         // TODO: Clean this up...we're using this to 'jump' the interpolation 
step for model properties not expressed in XML.
         //  [BP] - Can this above comment be explained?
         // We don't need all the project methods that are added over those in 
the model, but we do need basedir
+        // mkleint - using System.getProperties() is almost definitely bad for 
embedding.
         Map context = new HashMap( System.getProperties() );
 
         if ( projectDir != null )


Reply via email to