Skip to content

Commit

Permalink
Resolve home dirs to absolute paths in whisky_darwin.go
Browse files Browse the repository at this point in the history
  • Loading branch information
jtatum committed Dec 17, 2024
1 parent 26f8a7e commit 61d488a
Showing 1 changed file with 12 additions and 2 deletions.
14 changes: 12 additions & 2 deletions backend/installfinders/launchers/whisky/whisky_darwin.go
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,12 @@ func getWhiskyBottlesPath() (string, error) {
return "", fmt.Errorf("failed to get user home dir: %w", err)
}

defaultBottlesPath := filepath.Join(homeDir, whiskyDefaultBottlesRelativePath)
absHomeDir, err := filepath.Abs(homeDir)
if err != nil {
return "", fmt.Errorf("failed to get absolute path of user home dir: %w", err)
}

defaultBottlesPath := filepath.Join(absHomeDir, whiskyDefaultBottlesRelativePath)

var bottlesPath string

Expand All @@ -93,7 +98,12 @@ func getWhiskyBottlesPath() (string, error) {
if err != nil {
slog.Error("failed to parse Whisky config file", slog.Any("error", err))
} else {
bottlesPath = config.DefaultBottleLocation
if bottlesPath != "" {
bottlesPath, err = filepath.Abs(config.DefaultBottleLocation)
if err != nil {
return "", fmt.Errorf("failed to get absolute path of Whisky bottles path: %w", err)
}
}
}
}

Expand Down

0 comments on commit 61d488a

Please sign in to comment.