Bug 11444 and bug 11599 - for non x86, allow non-exe executables and for attach don't ask for install file RCL_2_4
authordadubrow
Tue, 06 Jul 2010 11:42:41 -0500
branchRCL_2_4
changeset 1575 8ced3afefe5b
parent 1573 660c89efde83
child 1580 60a1e2151c91
Bug 11444 and bug 11599 - for non x86, allow non-exe executables and for attach don't ask for install file
debuggercdi/com.nokia.cdt.debug.launch/src/com/nokia/cdt/internal/debug/launch/LaunchPlugin.java
debuggercdi/com.nokia.cdt.debug.launch/src/com/nokia/cdt/internal/debug/launch/newwizard/DebugRunProcessSection.java
--- a/debuggercdi/com.nokia.cdt.debug.launch/src/com/nokia/cdt/internal/debug/launch/LaunchPlugin.java	Fri Jul 02 16:58:26 2010 -0500
+++ b/debuggercdi/com.nokia.cdt.debug.launch/src/com/nokia/cdt/internal/debug/launch/LaunchPlugin.java	Tue Jul 06 11:42:41 2010 -0500
@@ -222,20 +222,19 @@
 	}
 	
 	private boolean getExePathsFromProjectAndDetermineIfX86(IProject project, List<IPath> exePaths) {
+		// changed logic to allow paths for executables for !isExecutable(path) for non-x86
+		// this allows new plug-and-play wizard to create attach configs for imported dlls (see bug 11444)
 		boolean isX86 = false;
         ICProject cProject = CoreModel.getDefault().create(project);
         if (cProject != null) {
 			try {
     			for (IBinary bin : cProject.getBinaryContainer().getBinaries()) {
-    				if (bin.isExecutable()) {
-    					IPath path = bin.getResource().getLocation();
-    					
-    					if (isEmulatorBinaryPath(path)) {
-							isX86 = true;
-						}
+					IPath path = bin.getResource().getLocation();
+					
+					isX86 = isX86 || isEmulatorBinaryPath(path); // only check once
 
+    				if (!isX86 || bin.isExecutable())
     					exePaths.add(path);
-    				}
     			}
 			} catch (CModelException e) {
 				log(e);
--- a/debuggercdi/com.nokia.cdt.debug.launch/src/com/nokia/cdt/internal/debug/launch/newwizard/DebugRunProcessSection.java	Fri Jul 02 16:58:26 2010 -0500
+++ b/debuggercdi/com.nokia.cdt.debug.launch/src/com/nokia/cdt/internal/debug/launch/newwizard/DebugRunProcessSection.java	Tue Jul 06 11:42:41 2010 -0500
@@ -76,7 +76,7 @@
 		}
 		
 		if (data.getExeSelection().equals(EExeSelection.ATTACH_TO_PROCESS)) {
-			data.setInstallPackage(true);
+			data.setInstallPackage(false);
 		} else {
 			Boolean detectedSysTrk = data.isSysTRKConnection(); // TRUE, FALSE, or null
 			boolean isSysTrk = detectedSysTrk == Boolean.TRUE || (detectedSysTrk == null && data.isInternalLayout());