fix: fix nil pointer execution if no proper packagemanager selected
This commit is contained in:
parent
cc3cc5c916
commit
98f8213953
3 changed files with 9 additions and 2 deletions
|
|
@ -28,7 +28,7 @@ var installCmd = &cobra.Command{
|
|||
if err := osManager.Install([]string{packageName}); err != nil {
|
||||
log.Printf("error: %v\n", err)
|
||||
}
|
||||
case "homebrew":
|
||||
case "homebrew", "brew":
|
||||
manager = &pm.HomebrewManager{}
|
||||
case "cargo":
|
||||
manager = &pm.CargoManager{}
|
||||
|
|
@ -38,6 +38,7 @@ var installCmd = &cobra.Command{
|
|||
manager = &pm.FlatpakManager{}
|
||||
default:
|
||||
fmt.Println("No PackageManager found")
|
||||
return
|
||||
}
|
||||
if err := manager.InstallPackage(packageName); err != nil {
|
||||
log.Printf("error: %v\n", err)
|
||||
|
|
|
|||
|
|
@ -1,6 +1,7 @@
|
|||
package cmd
|
||||
|
||||
import (
|
||||
"fmt"
|
||||
"log"
|
||||
"system_setup_tool/utils"
|
||||
|
||||
|
|
@ -27,7 +28,7 @@ var removeCmd = &cobra.Command{
|
|||
if err := osManager.Install([]string{packageName}); err != nil {
|
||||
log.Printf("error: %v\n", err)
|
||||
}
|
||||
case "homebrew":
|
||||
case "homebrew", "brew":
|
||||
manager = &pm.HomebrewManager{}
|
||||
case "cargo":
|
||||
manager = &pm.CargoManager{}
|
||||
|
|
@ -36,6 +37,8 @@ var removeCmd = &cobra.Command{
|
|||
case "flatpak":
|
||||
manager = &pm.FlatpakManager{}
|
||||
default:
|
||||
fmt.Println("No PackageManager found")
|
||||
return
|
||||
}
|
||||
if err := manager.RemovePackage(packageName); err != nil {
|
||||
log.Printf("error: %v\n", err)
|
||||
|
|
|
|||
|
|
@ -27,6 +27,9 @@ var searchCmd = &cobra.Command{
|
|||
}
|
||||
|
||||
if managerName != "" {
|
||||
if managerName == "brew" {
|
||||
managerName = "homebrew"
|
||||
}
|
||||
for _, m := range managers {
|
||||
if strings.EqualFold(m.Name(), managerName) {
|
||||
searchAndDisplayResults(m, packageName)
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue