svn commit: r731220 - /maven/surefire/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java

[dantran]

Author: dantran
Date: Sun Jan  4 03:11:33 2009
New Revision: 731220

URL: http://svn.apache.org/viewvc?rev=731220&view=rev
Log:
SUREFIRE-511: allow null systemPropertyVariable's value, otherwise user cannot 
migrate deprecated systemProperties. Forgot one check-in from the last commit

Modified:
    
maven/surefire/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java

Modified: 
maven/surefire/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java
URL: 
http://svn.apache.org/viewvc/maven/surefire/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java?rev=731220&r1=731219&r2=731220&view=diff
==============================================================================
--- 
maven/surefire/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java
 (original)
+++ 
maven/surefire/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java
 Sun Jan  4 03:11:33 2009
@@ -1040,13 +1040,12 @@
             {
                 String key = (String) i.next();
                 String value = (String) systemPropertyVariables.get( key );
-                if ( value == null )
+                //java Properties does not accept null value
+                if ( value != null )
                 {
-                    this.getLog().error( key + "'s value cannot be null." );
+                    internalSystemProperties.setProperty( key, value );
                 }
-                internalSystemProperties.setProperty( key, value );
             }
-            
         }
 
         originalSystemProperties = (Properties) System.getProperties().clone();