Skip to content

Commit

Permalink
Merge pull request #2940 from ActiveState/mitchell/dx-1699
Browse files Browse the repository at this point in the history
Use installation package for directory determination.
  • Loading branch information
mitchell-as authored Dec 13, 2023
2 parents 25b620b + c586708 commit 285fb93
Showing 1 changed file with 6 additions and 3 deletions.
9 changes: 6 additions & 3 deletions internal/runners/clean/run_win.go
Original file line number Diff line number Diff line change
Expand Up @@ -117,11 +117,14 @@ func removeInstall(logFile string, params *UninstallParams, cfg *config.Instance
return locale.WrapError(err, "err_state_exec")
}

// Schedule removal of the entire branch name directory.
// Schedule removal of the entire install directory.
// This is because Windows often thinks the installation.InstallDirMarker and
// constants.StateInstallerCmd files are still in use.
branchDir := filepath.Dir(filepath.Dir(stateExec))
paths := []string{stateExec, branchDir}
installDir, err := installation.InstallPathFromExecPath()
if err != nil {
return errs.Wrap(err, "Could not get installation path")
}
paths := []string{stateExec, installDir}
if params.All {
paths = append(paths, cfg.ConfigPath()) // also remove the config directory
}
Expand Down

0 comments on commit 285fb93

Please sign in to comment.