review comments

DataDog · Jan 11, 2023 · 53f9090 · 53f9090
1 parent a348c2d
commit 53f9090
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/comp/systray/systray/systray.go b/comp/systray/systray/systray.go
@@ -92,7 +92,7 @@ var (
 
 // newSystray creates a new systray component, which will start and stop based on
 // the fx Lifecycle
-func newSystray(deps dependencies) (Component, error) {
+func newSystray(deps dependencies) Component {
 	// fx init
 	s := &systray{
 		log:        deps.Log,
@@ -115,7 +115,7 @@ func newSystray(deps dependencies) (Component, error) {
 		s.isUserAdmin = isAdmin
 	}
 
-	return s, nil
+	return s
 }
 
 // start hook has a fx enforced timeout, so don't do long running things

diff --git a/tasks/systray.py b/tasks/systray.py
@@ -90,6 +90,6 @@ def clean(ctx):
     # remove the bin/agent folder
     print("Remove systray executable")
     try:
-        os.remove('./bin/agent/ddtray.exe')
+        os.remove(os.path.join(BIN_PATH, bin_name("ddtray")))
     except Exception as e:
         print(e)