Attachment 73576 Details for Bug 147044 – proposed fix

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

[patch] proposed fix

147044.patch (text/plain), 1.64 KB, created by Michael Rennie

on 2007-07-11 14:14:44 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Michael Rennie

Created: 2007-07-11 14:14:44 EDT

Size: 1.64 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.ant.ui
>Index: Ant Tools Support/org/eclipse/ant/internal/ui/launchConfigurations/AntJRETab.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.ant.ui/Ant Tools Support/org/eclipse/ant/internal/ui/launchConfigurations/AntJRETab.java,v
>retrieving revision 1.42
>diff -u -r1.42 AntJRETab.java
>--- Ant Tools Support/org/eclipse/ant/internal/ui/launchConfigurations/AntJRETab.java	19 Oct 2006 22:35:56 -0000	1.42
>+++ Ant Tools Support/org/eclipse/ant/internal/ui/launchConfigurations/AntJRETab.java	11 Jul 2007 18:12:59 -0000
>@@ -184,13 +184,19 @@
> 			 	
> 			 	//null out the vm type and jre container path to get the default vm install from JavaRuntime
> 			 	copy.setAttribute(IJavaLaunchConfigurationConstants.ATTR_VM_INSTALL_TYPE, (String)null);
>-			 	copy.setAttribute(IJavaLaunchConfigurationConstants.ATTR_JRE_CONTAINER_PATH, (String)null);
>+			 	String oldpath = copy.getAttribute(IJavaLaunchConfigurationConstants.ATTR_JRE_CONTAINER_PATH, (String)null);
>+			 	if(oldpath != null) {
>+			 		copy.setAttribute(IJavaLaunchConfigurationConstants.ATTR_JRE_CONTAINER_PATH, (String)null);
>+			 	}
> 			 	IVMInstall defaultVMInstall= getDefaultVMInstall(copy);
> 			 	if (defaultVMInstall != null) {
> 			 		//update if required
> 			 		setDefaultVMInstallAttributes(defaultVMInstall, copy);
> 			 	}
>-				if (copy.isDirty() && !copy.isReadOnly()) {
>+			 	if(oldpath != null) {
>+			 		copy.setAttribute(IJavaLaunchConfigurationConstants.ATTR_JRE_CONTAINER_PATH, oldpath);
>+			 	}
>+			 	if (copy.isDirty() && !copy.isReadOnly()) {
> 					configuration= copy.doSave();
> 				}
> 			 }

Actions: View | Diff

Attachments on bug 147044: 73576 | 134659 | 163007