Merge pull request #7 from sourcegraph/nelsona/windows-no-fsync

nelsonjr · web-flow · commit 345a8df15ded · 2023-06-23T17:17:57.000-07:00
Rename directories on Windows does not need syscall.Fsync() on rename.
diff --git a/internal/fileutil/fileutil.go b/internal/fileutil/fileutil.go
@@ -1,3 +1,6 @@
+//go:build !windows
+// +build !windows
+
 package fileutil
 
 import (
diff --git a/internal/fileutil/fileutil_windows.go b/internal/fileutil/fileutil_windows.go
@@ -0,0 +1,14 @@
+//go:build windows
+// +build windows
+
+package fileutil
+
+import (
+	"os"
+)
+
+// RenameAndSync will do an os.Rename followed by fsync to ensure the rename
+// is recorded
+func RenameAndSync(oldpath, newpath string) error {
+	return os.Rename(oldpath, newpath)
+}
