Maven 1.x Test Plugin

Handle spaces in additional parameters correctly

Details

  • Type: Bug Bug
  • Status: Open Open
  • Priority: Minor Minor
  • Resolution: Unresolved
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None
  • Environment:
    Maven 1.0
    Test Plugin 1.6.2
    Windows XP
  • Number of attachments :
    0

Description

The following statement in the jelly script does not work if the property contains an argument with spaces, like a path name under Windows:

maven.junit.for=true
maven.junit.jvmargs=-Xbootclasspath/a:C:/Path with Spaces/dummy.jar

<u:tokenize var="listOfJvmArgs" delim=" "> ${maven.junit.jvmargs}</u:tokenize>

This breaks the junit task and no test can be executed at all.

Perhaps it would be an option to specify the delim character as a plugin property so one can set it to something other than a space if required.

Activity

Hide
Charlie Collins added a comment -

I am still using Maven 1 (for a host of reasons), and running into this issue. This would be resolved by simply making the args delimiter itself a property, as the description states.

Is this on the radar anywhere, or is M1 stuff all dormant?

Show
Charlie Collins added a comment - I am still using Maven 1 (for a host of reasons), and running into this issue. This would be resolved by simply making the args delimiter itself a property, as the description states. Is this on the radar anywhere, or is M1 stuff all dormant?
Hide
Lukas Theussl added a comment -

The m1 test plugin had a release just 2 weeks ago (http://maven.apache.org/maven-1.x/plugins/bundled/multichanges-report.html), that's the version we intent to include in m1.1-final (which will be virtually identical to the released 1.1-RC1). I doubt that there will be much activity on m1 anymore after that. But you can always try to vote for the issue or attach a patch...

Show
Lukas Theussl added a comment - The m1 test plugin had a release just 2 weeks ago (http://maven.apache.org/maven-1.x/plugins/bundled/multichanges-report.html), that's the version we intent to include in m1.1-final (which will be virtually identical to the released 1.1-RC1). I doubt that there will be much activity on m1 anymore after that. But you can always try to vote for the issue or attach a patch...

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated: