Attachment 155688 Details for Bug 263689 – CDI-only patch

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

[patch] CDI-only patch

MacCDDirectorySpaces-CDI-patch.txt (text/plain), 1.94 KB, created by Marc-André Laperle

on 2010-01-10 16:42:28 EST

(hide)

Description:

Filename:

MIME Type:

Creator: Marc-André Laperle

Created: 2010-01-10 16:42:28 EST

Size: 1.94 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.cdt.debug.mi.core
>Index: mi/org/eclipse/cdt/debug/mi/core/command/factories/macos/MacOSMIEnvironmentCD.java
>===================================================================
>RCS file: /cvsroot/tools/org.eclipse.cdt/all/org.eclipse.cdt.debug.mi.core/mi/org/eclipse/cdt/debug/mi/core/command/factories/macos/MacOSMIEnvironmentCD.java,v
>retrieving revision 1.1
>diff -u -r1.1 MacOSMIEnvironmentCD.java
>--- mi/org/eclipse/cdt/debug/mi/core/command/factories/macos/MacOSMIEnvironmentCD.java	17 May 2006 16:31:23 -0000	1.1
>+++ mi/org/eclipse/cdt/debug/mi/core/command/factories/macos/MacOSMIEnvironmentCD.java	10 Jan 2010 21:32:05 -0000
>@@ -1,5 +1,5 @@
> /**********************************************************************
>- * Copyright (c) 2006 Nokia and others.
>+ * Copyright (c) 2006, 2010 Nokia and others.
>  * All rights reserved. This program and the accompanying materials
>  * are made available under the terms of the Eclipse Public License v1.0
>  * which accompanies this distribution, and is available at
>@@ -7,6 +7,7 @@
>  * 
>  * Contributors: 
>  * Nokia - Initial API and implementation
>+ * Marc-Andre Laperle - fix for bug 263689 (spaces in directory)
> ***********************************************************************/
> package org.eclipse.cdt.debug.mi.core.command.factories.macos;
> 
>@@ -16,13 +17,15 @@
> 
> 	public MacOSMIEnvironmentCD(String miVersion, String path) {
> 		super(miVersion, path);
>-		this.setOperation("cd");//$NON-NLS-1$
>+		this.setOperation("-environment-cd");//$NON-NLS-1$
> 	}
> 	
> 	protected String parametersToString() {
> 		String[] parameters = getParameters();
> 		if (parameters != null && parameters.length == 1) {
>-			return '"' + parameters[0] + '"';
>+			// To handle spaces in the path, the command string has this format:
>+			// -environment-cd "\"/path with spaces\""
>+			return "\"\\\"" + parameters[0] + "\\\"\""; //$NON-NLS-1$ //$NON-NLS-2$
> 		}
> 		return super.parametersToString();		
> 	}

Actions: View | Diff

Attachments on bug 263689: 124726 | 139339 | 139457 | 155688 | 160124 | 160879 | 161013