nit

cmd/control/control.go

@@ -244,9 +244,6 @@ func newStartCommand() *cobra.Command {
 		false,
 		"true to assign dynamic ports",
 	)
-	if err := cmd.MarkPersistentFlagRequired("avalanchego-path"); err != nil {
-		panic(err)
-	}
 	return cmd
 }
 

tests/e2e/e2e_test.go

@@ -498,11 +498,11 @@ var _ = ginkgo.Describe("[Start/Remove/Restart/Add/Stop]", func() {
 	})
 
 	ginkgo.It("can start", func() {
-		ginkgo.By("start request with invalid exec path should fail", func() {
+		ginkgo.By("start request with empty exec path should fail", func() {
 			ctx, cancel := context.WithTimeout(context.Background(), 2*time.Minute)
 			_, err := cli.Start(ctx, "")
 			cancel()
-			gomega.Ω(err.Error()).Should(gomega.ContainSubstring(utils.ErrInvalidExecPath.Error()))
+			gomega.Ω(err.Error()).Should(gomega.ContainSubstring(utils.ErrEmptyExecPath.Error()))
 		})
 
 		ginkgo.By("start request with invalid exec path should fail", func() {

utils/utils.go

@@ -47,14 +47,14 @@ func NetworkIDFromGenesis(genesis []byte) (uint32, error) {
 }
 
 var (
-	ErrInvalidExecPath = errors.New("avalanche exec is invalid")
+	ErrEmptyExecPath   = errors.New("avalanche exec is not defined")
 	ErrNotExists       = errors.New("avalanche exec not exists")
 	ErrNotExistsPlugin = errors.New("plugin exec not exists")
 )
 
 func CheckExecPath(exec string) error {
 	if exec == "" {
-		return ErrInvalidExecPath
+		return ErrEmptyExecPath
 	}
 	_, err := os.Stat(exec)
 	if err != nil {

utils/utils_test.go

@@ -71,7 +71,7 @@ func TestCheckExecPath(t *testing.T) {
 		},
 		{
 			execPath:    "",
-			expectedErr: ErrInvalidExecPath,
+			expectedErr: ErrEmptyExecPath,
 		},
 		{
 			execPath:    "invalid",