[jdt-debug-dev] Patches on bug #36836

["Ias" <iasandcb@xxxxxxxxxx>]

Let me introduce the necessity of the feature: my plugin is designed for a
web application server, which needs to use some jars as boot classes. One
of my colleagues is developing a web application using javax.crypto
package, and the application doesn't work without /p option under the WAS
and the plugin. 

I made some prototyped patches regarding
https://bugs.eclipse.org/bugs/show_bug.cgi?id=36836

Index: VMRunnerConfiguration.java
===================================================================
RCS file:
/home/eclipse/org.eclipse.jdt.launching/launching/org/eclipse/jdt/launching/
VMRunnerConfiguration.java,v
retrieving revision 1.16
diff -u -r1.16 VMRunnerConfiguration.java
--- VMRunnerConfiguration.java	10 Mar 2003 21:59:28 -0000	1.16
+++ VMRunnerConfiguration.java	24 Apr 2003 14:18:12 -0000
@@ -34,6 +34,21 @@
 	
 	private static final String[] fgEmpty= new String[0];
 	
+	public static final String BOOTCLASSPATH_A_OPTION = "/a";
+	public static final String BOOTCLASSPATH_P_OPTION = "/p";
+	
+	private String fBootClassPathOption;
+	
+	public void setBootClassPathOption(String option)
+	{
+		fBootClassPathOption = option;
+	}
+	
+	public String getBootClassPathOption()
+	{
+		return fBootClassPathOption;
+	}
+	
 	/**
 	 * Creates a new configuration for launching a VM to run the given
main class
 	 * using the given class path.

Index: StandardVMDebugger.java
===================================================================
RCS file:
/home/eclipse/org.eclipse.jdt.launching/launching/org/eclipse/jdt/internal/l
aunching/StandardVMDebugger.java,v
retrieving revision 1.25
diff -u -r1.25 StandardVMDebugger.java
--- StandardVMDebugger.java	10 Mar 2003 21:59:28 -0000	1.25
+++ StandardVMDebugger.java	24 Apr 2003 14:18:46 -0000
@@ -148,7 +148,8 @@
 		String[] bootCP= config.getBootClassPath();
 		if (bootCP != null) {
 			if (bootCP.length > 0) {
-				arguments.add("-Xbootclasspath:" +
convertClassPath(bootCP)); //$NON-NLS-1$
+				String option =
config.getBootClassPathOption();
+				arguments.add("-Xbootclasspath" + (option
== null ? "" : option) + ":" + convertClassPath(bootCP)); //$NON-NLS-1$
 			} else {
 				// empty
 				arguments.add("-Xbootclasspath:"); //$NON-
NLS-1$

------

I tested the built launching.jar and it proved to work fine with my plugin
project like the following snippet:

		VMRunnerConfiguration vmConfig =
			new VMRunnerConfiguration(classToLaunch, classpath);
...

		vmConfig.setBootClassPath(bootClasspath);	
	
vmConfig.setBootClassPathOption(VMRunnerConfiguration.BOOTCLASSPATH_P_OPTION
);
...

Now he is happily working with his application and my plugin :-)

Thanks,

Ias.
 
===========================================================
Lee, Changshin (Korean name)
Ias (International name)
               Company Web Site: http://www.tmax.co.kr
               Personal Web Site: http://www.iasandcb.pe.kr
---------------------------------------------------------
Senior Researcher & Emerging Technology Evangelist
JCP member - http://jcp.org/en/participation/members/L
R&D Institute
Tmax Soft, Inc. 
JCP member - http://jcp.org/en/participation/members/T
==========================================================