View | Details | Raw Unified | Return to bug 188330 | Differences between
and this patch

Collapse All | Expand All

(-)src/org/eclipse/rse/internal/services/local/files/LocalFileService.java (-6 / +9 lines)
Lines 30-35 Link Here
30
 * Kevin Doyle		(IBM)		 - [208778] [efs][api] RSEFileStore#getOutputStream() does not support EFS#APPEND
30
 * Kevin Doyle		(IBM)		 - [208778] [efs][api] RSEFileStore#getOutputStream() does not support EFS#APPEND
31
 * David McKnight   (IBM)        - [209593] [api] add support for "file permissions" and "owner" properties for unix files
31
 * David McKnight   (IBM)        - [209593] [api] add support for "file permissions" and "owner" properties for unix files
32
 * Radoslav Gerganov (ProSyst)   - [218173] [local] non-generic filters don't work
32
 * Radoslav Gerganov (ProSyst)   - [218173] [local] non-generic filters don't work
33
 * Radoslav Gerganov (ProSyst)   - [188330] Problems Copying files with $ in name
33
 ********************************************************************************/
34
 ********************************************************************************/
34
35
35
package org.eclipse.rse.internal.services.local.files;
36
package org.eclipse.rse.internal.services.local.files;
Lines 1263-1271 Link Here
1263
//		 handle special characters in source and target strings 
1264
//		 handle special characters in source and target strings 
1264
		StringBuffer srcBuf = new StringBuffer(src);
1265
		StringBuffer srcBuf = new StringBuffer(src);
1265
		StringBuffer tgtBuf = new StringBuffer(target);
1266
		StringBuffer tgtBuf = new StringBuffer(target);
1266
		handleSpecialChars(srcBuf);
1267
		// there is no need to escape special chars on Windows,
1267
		handleSpecialChars(tgtBuf);
1268
		// if the arguments are enclosed with double quotes
1268
		
1269
		if (!isWindows()) {
1270
  		handleSpecialChars(srcBuf);
1271
  		handleSpecialChars(tgtBuf);
1272
		}
1269
		src = "\"" + srcBuf.toString() + "\""; //$NON-NLS-1$ //$NON-NLS-2$
1273
		src = "\"" + srcBuf.toString() + "\""; //$NON-NLS-1$ //$NON-NLS-2$
1270
		target = "\"" + tgtBuf.toString() + "\""; //$NON-NLS-1$ //$NON-NLS-2$
1274
		target = "\"" + tgtBuf.toString() + "\""; //$NON-NLS-1$ //$NON-NLS-2$
1271
		/*
1275
		/*
Lines 1275-1282 Link Here
1275
		if (target.indexOf(' ') >= 0)
1279
		if (target.indexOf(' ') >= 0)
1276
			target = "\"" + target + "\"";
1280
			target = "\"" + target + "\"";
1277
		*/
1281
		*/
1278
		boolean isWindows = System.getProperty("os.name").toLowerCase().startsWith("win"); //$NON-NLS-1$ //$NON-NLS-2$
1282
		if (isWindows())
1279
		if (isWindows)
1280
		{
1283
		{
1281
			if (folderCopy)
1284
			if (folderCopy)
1282
			{
1285
			{
Lines 1320-1326 Link Here
1320
		{
1323
		{
1321
			Process p = null;
1324
			Process p = null;
1322
			Runtime runtime = Runtime.getRuntime();
1325
			Runtime runtime = Runtime.getRuntime();
1323
			if (isWindows)
1326
			if (isWindows())
1324
			{
1327
			{
1325
				String theShell = "cmd /C "; //$NON-NLS-1$
1328
				String theShell = "cmd /C "; //$NON-NLS-1$
1326
				p = runtime.exec(theShell + command);		
1329
				p = runtime.exec(theShell + command);		

Return to bug 188330