Mailing-List: contact issues-help@maven.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@maven.apache.org
Received-SPF: neutral (herse.apache.org: local policy)
Message-ID: 
 <51273936.1173473952862.JavaMail.haus-jira@codehaus01.managed.contegix.com>
Date: Fri, 9 Mar 2007 14:59:12 -0600 (CST)
From: "Vincent Massol (JIRA)" <jira@codehaus.org>
To: issues@maven.apache.org
Subject: [jira] Reopened: (SUREFIRE-101) Plugin not longer sets system
 properties when forking is on and debugging information is not correct
In-Reply-To: 
 <72568200.1152378922613.JavaMail.haus-jira@codehaus01.managed.contegix.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit


     [ http://jira.codehaus.org/browse/SUREFIRE-101?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Vincent Massol reopened SUREFIRE-101:
-------------------------------------


Reopening as no longer convinced this is fixed. We're getting the issue again on Cargo after removing our <version>2.1.3</version> which is the last version where it worked

> Plugin not longer sets system properties when forking is on and debugging information is not correct
> ----------------------------------------------------------------------------------------------------
>
>                 Key: SUREFIRE-101
>                 URL: http://jira.codehaus.org/browse/SUREFIRE-101
>             Project: Maven Surefire
>          Issue Type: Bug
>    Affects Versions: 2.0 (2.2 plugin)
>            Reporter: Vincent Massol
>         Assigned To: Vincent Massol
>            Priority: Blocker
>             Fix For: 2.3
>
>         Attachments: MSUREFIRE-145.zip
>
>
> The following code is in SurefirePlugin.java:
> {code:java}
>         processSystemProperties( !fork.isForking() );
>         if ( getLog().isDebugEnabled() )
>         {
>             showMap( systemProperties, "system property" );
>         }
> {code}
> 2 problems:
> 1) fork.isForking() is false when forking is enabled and thus system properties are not set:
> {code:java}
>     protected void processSystemProperties( boolean setInSystem )
>     {
> [...]
>         if ( setInSystem )
>         {
>             // Add all system properties configured by the user
>             Iterator iter = systemProperties.keySet().iterator();
>             while ( iter.hasNext() )
>             {
>                 String key = (String) iter.next();
>                 String value = systemProperties.getProperty( key );
>                 System.setProperty( key, value );
>             }
>         }
>     }
> {code}
> 2)  showMap() is called regardless of whether the system properties are set or not, leading to the following kind of misleading logs:
> {noformat}
> [...]
> [DEBUG] Setting system property [cargo.jetty4x.port]=[8280]
> [DEBUG] Setting system property [cargo.jetty6x.port]=[8280]
> [DEBUG] Setting system property [cargo.jetty5x.port]=[8280]
> [...]
> {noformat}
> Those properties are actually NOT set.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira