KurlesHS/restic
1
0
Fork 0
forked from TrueCloudLab/restic
Code Pull requests Activity

self-update: Fix handling of --output on windows

Browse source 
The code always assumed that the upgrade happens in place. Thus writing
the upgrade to a separate file fails, when trying to remove the file
stored at that location.
This commit is contained in:
Michael Eischer 2023-01-20 23:13:04 +01:00
parent 0f398b82e3
commit 34e67e3510
1 changed files with 7 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
internal/selfupdate/download_windows.go
View file

@ -7,11 +7,18 @@ import (
"fmt" "fmt"
"os" "os"
"path/filepath" "path/filepath"
"github.com/restic/restic/internal/errors"
) )
// Rename (rather than remove) the running version. The running binary will be locked // Rename (rather than remove) the running version. The running binary will be locked
// on Windows and cannot be removed while still executing. // on Windows and cannot be removed while still executing.
func removeResticBinary(dir, target string) error { func removeResticBinary(dir, target string) error {
// nothing to do if the target does not exist
if _, err := os.Stat(target); errors.Is(err, os.ErrNotExist) {
return nil
}
backup := filepath.Join(dir, filepath.Base(target)+".bak") backup := filepath.Join(dir, filepath.Base(target)+".bak")
if _, err := os.Stat(backup); err == nil { if _, err := os.Stat(backup); err == nil {
_ = os.Remove(backup) _ = os.Remove(backup)