[ https://issues.apache.org/jira/browse/SUREFIRE-1391?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16105881#comment-16105881 ]
Hudson commented on SUREFIRE-1391: ---------------------------------- SUCCESS: Integrated in Jenkins build maven-surefire-dev #23 (See [https://builds.apache.org/job/maven-surefire-dev/23/]) SUREFIRE-1391: Eliminate redundant call to process (tibor17: [http://git-wip-us.apache.org/repos/asf/?p=maven-surefire.git&a=commit&h=26527d10c4835c254f8d7d2efb2227a30697f674]) * (edit) maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/SurefireProperties.java > System Property variables processed twice in calculation of effective > properties > -------------------------------------------------------------------------------- > > Key: SUREFIRE-1391 > URL: https://issues.apache.org/jira/browse/SUREFIRE-1391 > Project: Maven Surefire > Issue Type: Bug > Components: Maven Surefire Plugin > Reporter: Andrew Cohen > Assignee: Michael Osipov > Priority: Trivial > Fix For: 2.20.1 > > Attachments: > 0001-SUREFIRE-1391-Eliminate-redundant-call-to-process-sy.patch > > Original Estimate: 10m > Remaining Estimate: 10m > > System property variables are processed twice in the calculation of effective > properties. There is obviously no adverse impact on behavior, just > unnecessary processing. > {code:java} > static SurefireProperties calculateEffectiveProperties( Properties > systemProperties, > Map<String, > String> systemPropertyVariables, > Properties > userProperties, SurefireProperties props ) > { > SurefireProperties result = new SurefireProperties(); > result.copyPropertiesFrom( systemProperties ); > result.copyPropertiesFrom( props ); > copyProperties( result, systemPropertyVariables ); > copyProperties( result, systemPropertyVariables ); > // We used to take all of our system properties and dump them in with > the > // user specified properties for SUREFIRE-121, causing SUREFIRE-491. > // Not gonna do THAT any more... instead, we only propagate those > system properties > // that have been explicitly specified by the user via -Dkey=value on > the CLI > result.copyPropertiesFrom( userProperties ); > return result; > } > {code} -- This message was sent by Atlassian JIRA (v6.4.14#64029)