Attachment 118368 Details for Bug 255813 – patch

Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

[patch] patch

255813_a.txt (text/plain), 2.14 KB, created by Raheel Ashraf

on 2008-11-20 10:04:12 EST

(hide)

Description:

Filename:

MIME Type:

Creator: Raheel Ashraf

Created: 2008-11-20 10:04:12 EST

Size: 2.14 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.tptp.platform.jvmti.client
>Index: src/org/eclipse/tptp/platform/jvmti/client/internal/launcher/TIDelegateHelper.java
>===================================================================
>RCS file: /cvsroot/tptp/platform/org.eclipse.tptp.platform.jvmti.client/src/org/eclipse/tptp/platform/jvmti/client/internal/launcher/TIDelegateHelper.java,v
>retrieving revision 1.74
>diff -u -r1.74 TIDelegateHelper.java
>--- src/org/eclipse/tptp/platform/jvmti/client/internal/launcher/TIDelegateHelper.java	11 Jun 2008 21:41:25 -0000	1.74
>+++ src/org/eclipse/tptp/platform/jvmti/client/internal/launcher/TIDelegateHelper.java	20 Nov 2008 15:00:47 -0000
>@@ -300,8 +300,30 @@
> 			if(value.toString().indexOf("probe") != -1)
> 			{
> 				TI_JVM_FLAG = TI_PROBE_JVM_FLAG;
>-				String VMArgs = launchConfiguration.getAttribute("executableArgs", "");
>-				TI_JVM_FLAG = VMArgs + " " + TI_JVM_FLAG ;
>+				String VMArgs = launchConfiguration.getAttribute("executab1leArgs", "");
>+				//255813 need to deal with the Remote Java Application use case
>+				//where the specified classpath needs to be added to the bootclasspath
>+				String classpath = launchConfiguration.getAttribute(IProfileLaunchConfigurationConstants.ATTR_CLASSPATH, (String)null);
>+				if(classpath != null) {
>+					StringBuffer bootClassPath = new StringBuffer();
>+					bootClassPath.append("-Xbootclasspath/p:"); //$NON-NLS-1$
>+					/* If this is a Windows operating system, add quotes to the beginning and end of
>+					 * the value passed to bootclasspath.  This is done in case there are spaces in the
>+					 * path(s) specified. */
>+					boolean isWindows = false;
>+					if (System.getProperty("os.name").toLowerCase().indexOf("windows") != -1) //$NON-NLS-1$ //$NON-NLS-2$
>+					{
>+						bootClassPath.append("\""); //$NON-NLS-1$
>+						isWindows = true;
>+					}
>+					bootClassPath.append(classpath);
>+					if (isWindows)
>+					{
>+						bootClassPath.append("\""); //$NON-NLS-1$
>+					}
>+					VMArgs += " " + bootClassPath.toString(); //$NON-NLS-1$
>+				}
>+				TI_JVM_FLAG = VMArgs + " " + TI_JVM_FLAG ; //$NON-NLS-1$
> 				isCGProf = false;
> 				break;
> 			}

Actions: View | Diff

Attachments on bug 255813: 118295 | 118368