Skip to content

Commit

Permalink
Fix windows file suffix in updater and pmctl
Browse files Browse the repository at this point in the history
  • Loading branch information
dhaavi committed May 22, 2019
1 parent 7bbfdf2 commit 81eee56
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 3 deletions.
11 changes: 10 additions & 1 deletion pmctl/run.go
Original file line number Diff line number Diff line change
Expand Up @@ -93,6 +93,11 @@ func run(identifier string, cmd *cobra.Command, filterDatabaseFlag bool) error {
args = os.Args[3:]
}

// adapt identifier
if windows() {
identifier += ".exe"
}

// run
for {
file, err := getFile(identifier)
Expand All @@ -101,7 +106,7 @@ func run(identifier string, cmd *cobra.Command, filterDatabaseFlag bool) error {
}

// check permission
if runtime.GOOS != "windows" {
if !windows() {
info, err := os.Stat(file.Path())
if err != nil {
return fmt.Errorf("%s failed to get file info on %s: %s", logPrefix, file.Path(), err)
Expand Down Expand Up @@ -179,3 +184,7 @@ func run(identifier string, cmd *cobra.Command, filterDatabaseFlag bool) error {
fmt.Printf("%s %s completed successfully\n", logPrefix, identifier)
return nil
}

func windows() bool {
return runtime.GOOS == "windows"
}
10 changes: 8 additions & 2 deletions updates/updater.go
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ import (
"errors"
"io/ioutil"
"path/filepath"
"runtime"
"time"

"github.com/Safing/portbase/log"
Expand All @@ -23,8 +24,13 @@ func updater() {

func CheckForUpdates() error {

// be sure that pmctl is part of the "used" updates
_, err := GetLocalFile("pmctl/pmctl")
// ensure core components are updated
var err error
if runtime.GOOS == "windows" {
_, err = GetPlatformFile("pmctl/pmctl.exe")
} else {
_, err = GetPlatformFile("pmctl/pmctl")
}
if err != nil {
log.Errorf("updates: failed to mark pmctl/pmctl as used to ensure updates: %s", err)
}
Expand Down

0 comments on commit 81eee56

Please sign in to comment.