[arquillian-issues] [JBoss JIRA] (ARQ-2131) Fix for multiple JVM arguments not released

[Gerhard Poul (JIRA)]

    [ https://issues.jboss.org/browse/ARQ-2131?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13442934#comment-13442934 ] 

Gerhard Poul commented on ARQ-2131:
-----------------------------------

[~aslak] What do you think? Would you be able to push a new beta out?

> Fix for multiple JVM arguments not released
> -------------------------------------------
>
>                 Key: ARQ-2131
>                 URL: https://issues.jboss.org/browse/ARQ-2131
>             Project: Arquillian
>          Issue Type: Bug
>          Components: WebSphere Containers
>         Environment: {code}arquillian-container-was-1.0.0.Beta2{code}
> {{arquillian.xml}} snippet:
> {code:xml}  <container qualifier="liberty-ci-managed">
>     <configuration>
>       <property name="deployType">xml</property>
>       <property name="javaVmArguments">-Dsetting1=x -Dsetting2=y</property>
>     </configuration>
>   </container>{code}
>            Reporter: Benjamin Marwell
>            Assignee: Gerhard Poul
>             Fix For: was_1.0.0.next
>
>
> Hello,
> h2. expected output
> Process started with (from Debug):
> {{FINER: Starting server with command: [java, -Dsetting1=x, -Dsetting2=y, -javaagent:lib/bootstrap-agent.jar, -jar, lib/ws-launch.jar, defaultServer]}}
> h2. actual output
> note the missing comma betweed setting1 and setting2:
> {{FINER: Starting server with command: [java, -Dsetting1=x -Dsetting2=y, -javaagent:lib/bootstrap-agent.jar, -jar, lib/ws-launch.jar, defaultServer]}}
> h2. Solution
> Already exists. This commit is currently not in a release:
> https://github.com/arquillian/arquillian-container-was/commit/a1a77fe92418f49943a97a57a4dd1ff7ebc2bd25#diff-957685487719c29ce4c464fc2ec9cf39
> What can't be seen from here: this commit is needed to have multiple JVM args. Otherwise all jvm args are concatenated into a single String.
> So, the bug is fixed in trunk, but not released.
> h2. Proposed test case
> It is also missing a test  case.
> Please add a test case with two or more {{-D}}-Parameters and check in your servlet if they are set.
> Thank you very much!



--
This message was sent by Atlassian JIRA
(v7.2.3#72005)