fix: fix nil pointer execution if no proper packagemanager selected

This commit is contained in:
Patryk Hegenberg 2025-01-19 13:26:48 +01:00
parent cc3cc5c916
commit 3b1799397e
3 changed files with 9 additions and 2 deletions

View file

@ -28,7 +28,7 @@ var installCmd = &cobra.Command{
if err := osManager.Install([]string{packageName}); err != nil { if err := osManager.Install([]string{packageName}); err != nil {
log.Printf("error: %v\n", err) log.Printf("error: %v\n", err)
} }
case "homebrew": case "homebrew", "brew":
manager = &pm.HomebrewManager{} manager = &pm.HomebrewManager{}
case "cargo": case "cargo":
manager = &pm.CargoManager{} manager = &pm.CargoManager{}
@ -38,6 +38,7 @@ var installCmd = &cobra.Command{
manager = &pm.FlatpakManager{} manager = &pm.FlatpakManager{}
default: default:
fmt.Println("No PackageManager found") fmt.Println("No PackageManager found")
return
} }
if err := manager.InstallPackage(packageName); err != nil { if err := manager.InstallPackage(packageName); err != nil {
log.Printf("error: %v\n", err) log.Printf("error: %v\n", err)

View file

@ -1,6 +1,7 @@
package cmd package cmd
import ( import (
"fmt"
"log" "log"
"system_setup_tool/utils" "system_setup_tool/utils"
@ -27,7 +28,7 @@ var removeCmd = &cobra.Command{
if err := osManager.Install([]string{packageName}); err != nil { if err := osManager.Install([]string{packageName}); err != nil {
log.Printf("error: %v\n", err) log.Printf("error: %v\n", err)
} }
case "homebrew": case "homebrew", "brew":
manager = &pm.HomebrewManager{} manager = &pm.HomebrewManager{}
case "cargo": case "cargo":
manager = &pm.CargoManager{} manager = &pm.CargoManager{}
@ -36,6 +37,8 @@ var removeCmd = &cobra.Command{
case "flatpak": case "flatpak":
manager = &pm.FlatpakManager{} manager = &pm.FlatpakManager{}
default: default:
fmt.Println("No PackageManager found")
return
} }
if err := manager.RemovePackage(packageName); err != nil { if err := manager.RemovePackage(packageName); err != nil {
log.Printf("error: %v\n", err) log.Printf("error: %v\n", err)

View file

@ -27,6 +27,9 @@ var searchCmd = &cobra.Command{
} }
if managerName != "" { if managerName != "" {
if managerName == "brew" {
managerName = "homebrew"
}
for _, m := range managers { for _, m := range managers {
if strings.EqualFold(m.Name(), managerName) { if strings.EqualFold(m.Name(), managerName) {
searchAndDisplayResults(m, packageName) searchAndDisplayResults(m, packageName)