From 2682f801dfe90a85a7bf4d3ec2a51136280a2dac Mon Sep 17 00:00:00 2001
From: mmetc <92726601+mmetc@users.noreply.github.com>
Date: Fri, 5 Apr 2024 14:57:33 +0200
Subject: [PATCH] windows: fix data file update (remove before rename) (#2930)

---
 pkg/cwhub/dataset.go | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/pkg/cwhub/dataset.go b/pkg/cwhub/dataset.go
index 97fd9c5a0..921361e3f 100644
--- a/pkg/cwhub/dataset.go
+++ b/pkg/cwhub/dataset.go
@@ -4,9 +4,11 @@ import (
 	"errors"
 	"fmt"
 	"io"
+	"io/fs"
 	"net/http"
 	"os"
 	"path/filepath"
+	"runtime"
 	"time"
 
 	"github.com/sirupsen/logrus"
@@ -65,6 +67,18 @@ func downloadFile(url string, destPath string) error {
 	// TODO: use a better way to communicate this
 	fmt.Printf("updated %s\n", filepath.Base(destPath))
 
+	if runtime.GOOS == "windows" {
+		// On Windows, rename will fail if the destination file already exists
+		// so we remove it first.
+		err = os.Remove(destPath)
+		switch {
+		case errors.Is(err, fs.ErrNotExist):
+			break
+		case err != nil:
+			return err
+		}
+	}
+
 	if err = os.Rename(tmpFileName, destPath); err != nil {
 		return err
 	}