mv - do not futimes on symbolic links

Benjamin Pasero · Benjamin Pasero · commit 25b96b66d651 · 2019-03-27T11:34:53.000+01:00
diff --git a/src/vs/base/node/extfs.ts b/src/vs/base/node/extfs.ts
@@ -304,12 +304,12 @@ export function mv(source: string, target: string, callback: (error: Error | nul
 			return callback(err);
 		}
 
-		fs.stat(target, (error, stat) => {
+		fs.lstat(target, (error, stat) => {
 			if (error) {
 				return callback(error);
 			}
 
-			if (stat.isDirectory()) {
+			if (stat.isDirectory() || stat.isSymbolicLink()) {
 				return callback(null);
 			}
 

Original file line number	Diff line number	Diff line change
`@@ -304,12 +304,12 @@ export function mv(source: string, target: string, callback: (error: Error \| nul`
`304`	`304`	`return callback(err);`
`305`	`305`	`}`
`306`	`306`
`307`		`- fs.stat(target, (error, stat) => {`
	`307`	`+ fs.lstat(target, (error, stat) => {`
`308`	`308`	`if (error) {`
`309`	`309`	`return callback(error);`
`310`	`310`	`}`
`311`	`311`
`312`		`- if (stat.isDirectory()) {`
	`312`	`+ if (stat.isDirectory() \|\| stat.isSymbolicLink()) {`
`313`	`313`	`return callback(null);`
`314`	`314`	`}`
`315`	`315`