This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 106620 - [Profiler] Can not pass JVM option containing space enclosed in quotes
Summary: [Profiler] Can not pass JVM option containing space enclosed in quotes
Status: VERIFIED FIXED
Alias: None
Product: serverplugins
Classification: Unclassified
Component: Sun Appserver 9 (show other bugs)
Version: 6.x
Hardware: PC Windows XP
: P3 blocker (vote)
Assignee: Nitya Doraisamy
URL:
Keywords:
Depends on:
Blocks: 102323
  Show dependency tree
 
Reported: 2007-06-14 10:19 UTC by J Bachorik
Modified: 2007-10-01 08:52 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:

Attachments
Proposed patch (2.47 KB, patch)
2007-06-14 10:21 UTC, J Bachorik 		Details | Diff
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description J Bachorik 2007-06-14 10:19:19 UTC 
I need to pass a JVM option similar to "C:\path with space\myfolder" - it must be quoted for the sakes of consistency as
we pass the same argument to other servers as well as J2SE applications.
The resulting - <jvm-options>"C:\path with space\myfolder"</jvm-options> - causes the glassfish startup to fail
while slighlty modified - <jvm-options>C:\path with space\myfolder</jvm-options> - works well

This is specific to windows only - on U*x systems it works with quotes as well as without them

I'll attach a proposed patch to the DomainEditor.java which would resolve this problem on the plugin side (even if it
seems more likely to be glassfish related bug)
Comment 1 J Bachorik 2007-06-14 10:21:24 UTC 
Created attachment 43673 [details]
Proposed patch
Comment 2 Vince Kraemer 2007-06-14 14:22:27 UTC 
Yardus: Thanks for the patch.
Nitya: please evaluate the patch and integrate it.
Comment 3 Vince Kraemer 2007-06-14 14:50:44 UTC 
DomainEditor has a Tester in the qa-functional test suite. Please update it to check the patch, too.
Comment 4 Nitya Doraisamy 2007-06-14 21:47:17 UTC 
Will test and integrate the patch.
Comment 5 Nitya Doraisamy 2007-06-19 19:58:11 UTC 
Tested the provided patch. Looks good. Will check in ASAP.

The QA Functional tests are not working on my machine, so that checkin will follow later.
Comment 6 Nitya Doraisamy 2007-06-19 20:00:52 UTC 
Checking in DomainEditor.java;
/cvs/serverplugins/sun/appsrv81/src/org/netbeans/modules/j2ee/sun/ide/j2ee/DomainEditor.java,v  <--  DomainEditor.java
new revision: 1.5; previous revision: 1.4
done

Downgrading to P3 till QA Functional tests are updated
Comment 7 Vince Kraemer 2007-09-14 19:40:35 UTC 
what is up with this issue? have you updated the QA functional test?
Comment 8 Jiri Prox 2007-09-17 21:36:29 UTC 
Obsolete milestone, please reevaluate
Comment 10 J Bachorik 2007-10-01 08:52:57 UTC 
verified