fix(config): try catch unlink after load (#9577)

bluwy · web-flow · commit d35a1e2b828d · 2022-08-08T14:15:05.000+02:00
diff --git a/packages/vite/src/node/config.ts b/packages/vite/src/node/config.ts
@@ -1039,7 +1039,11 @@ async function loadConfigFromBundledFile(
     try {
       return (await dynamicImport(fileUrl)).default
     } finally {
-      fs.unlinkSync(fileNameTmp)
+      try {
+        fs.unlinkSync(fileNameTmp)
+      } catch {
+        // already removed if this function is called twice simultaneously
+      }
     }
   }
   // for cjs, we can register a custom loader via `_require.extensions`

Original file line number	Diff line number	Diff line change
`@@ -1039,7 +1039,11 @@ async function loadConfigFromBundledFile(`
`1039`	`1039`	`try {`
`1040`	`1040`	`return (await dynamicImport(fileUrl)).default`
`1041`	`1041`	`} finally {`
`1042`		`- fs.unlinkSync(fileNameTmp)`
	`1042`	`+ try {`
	`1043`	`+ fs.unlinkSync(fileNameTmp)`
	`1044`	`+ } catch {`
	`1045`	`+ // already removed if this function is called twice simultaneously`
	`1046`	`+ }`
`1043`	`1047`	`}`
`1044`	`1048`	`}`
`1045`	`1049`	// for cjs, we can register a custom loader via `_require.extensions`