[ 
https://issues.apache.org/jira/browse/DERBY-5504?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13151629#comment-13151629
 ] 

Dag H. Wanvik commented on DERBY-5504:
--------------------------------------

Patch looks good to me, +1. Do you plan to do likewise for the replication 
tests in a follow-up patch?
                
> SecureServerTest, Driver40UnbootedTest and replication tests not prepared for 
> space in java.home
> ------------------------------------------------------------------------------------------------
>
>                 Key: DERBY-5504
>                 URL: https://issues.apache.org/jira/browse/DERBY-5504
>             Project: Derby
>          Issue Type: Bug
>          Components: Test
>    Affects Versions: 10.9.0.0
>            Reporter: Knut Anders Hatlen
>            Assignee: Knut Anders Hatlen
>         Attachments: d5504-1a.diff
>
>
> SecureServerTest, Driver40UnbootedTest and the replication tests fork JVMs by 
> calling Runtime.exec() methods that take the command as a String parameter, 
> not a String[]. This causes problems like the ones seen in DERBY-5490 when 
> there's a space in ${java.home}.
> The tests should be rewritten to use the helper method 
> BaseTestCase.execJavaCmd(), which handles this case correctly.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to