chore: improve error message if batt is already uninstalled

Signed-off-by: Charlie Chiang <[email protected]>
charlie0129 · Sep 27, 2023 · ac86188 · ac86188
1 parent 5134fa3
commit ac86188
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/cmd.go b/cmd.go
@@ -550,7 +550,7 @@ func NewStatusCommand() *cobra.Command {
 			cmd.Printf("  State: %s\n", bold(state))
 			cmd.Printf("  Full capacity: %s\n", bold("%.1f Wh", bat.Design/1e3))
 			cmd.Printf("  Charge rate: %s\n", bold("%.1f W", bat.ChargeRate/1e3))
-			cmd.Printf("  voltage: %s\n", bold("%.2f V", bat.DesignVoltage))
+			cmd.Printf("  Voltage: %s\n", bold("%.2f V", bat.DesignVoltage))
 
 			cmd.Println()
 

diff --git a/install.go b/install.go
@@ -78,8 +78,6 @@ func installDaemon() error {
 }
 
 func uninstallDaemon() error {
-	// TODO: revert any changes, enable battery charging and enable adapter before uninstalling
-
 	logrus.Infof("stopping batt")
 
 	// run launchctl unload /Library/LaunchDaemons/cc.chlc.batt.plist
@@ -89,14 +87,14 @@ func uninstallDaemon() error {
 		"/Library/LaunchDaemons/cc.chlc.batt.plist",
 	).Run()
 	if err != nil {
-		return fmt.Errorf("failed to unload /Library/LaunchDaemons/cc.chlc.batt.plist: %w", err)
+		return fmt.Errorf("failed to unload /Library/LaunchDaemons/cc.chlc.batt.plist: %w. Are you root?", err)
 	}
 
 	logrus.Infof("removing launch daemon")
 
 	err = os.Remove("/Library/LaunchDaemons/cc.chlc.batt.plist")
 	if err != nil {
-		return fmt.Errorf("failed to remove /Library/LaunchDaemons/cc.chlc.batt.plist: %w", err)
+		return fmt.Errorf("failed to remove /Library/LaunchDaemons/cc.chlc.batt.plist: %w. Do you have enough permissions? Is batt already uninstalled?", err)
 	}
 
 	return nil