factor out init step for instance creation in updatecommands

Author: umbynos

cli/core/update_index.go

@@ -43,24 +43,7 @@ func initUpdateIndexCommand() *cobra.Command {
 
 func runUpdateIndexCommand(cmd *cobra.Command, args []string) {
 	logrus.Info("Executing `arduino core update-index`")
-	// We don't initialize any CoreInstance when updating indexes since we don't need to.
-	// Also meaningless errors might be returned when calling this command with --additional-urls
-	// since the CLI would be searching for a corresponding file for the additional urls set
-	// as argument but none would be obviously found.
-	inst, status := instance.Create()
-	if status != nil {
-		feedback.Errorf(tr("Error creating instance: %v"), status)
-		os.Exit(errorcodes.ErrGeneric)
-	}
-
-	// In case this is the first time the CLI is run we need to update indexes
-	// to make it work correctly, we must do this explicitly in this command since
-	// we must use instance.Create instead of instance.CreateAndInit for the
-	// reason stated above.
-	if err := instance.FirstUpdate(inst); err != nil {
-		feedback.Errorf(tr("Error updating indexes: %v"), status)
-		os.Exit(errorcodes.ErrGeneric)
-	}
+	inst := instance.CreateInstanceForUpdate()
 
 	_, err := commands.UpdateIndex(context.Background(), &rpc.UpdateIndexRequest{
 		Instance: inst,

cli/instance/instance.go

@@ -140,3 +140,27 @@ func FirstUpdate(instance *rpc.Instance) error {
 
 	return nil
 }
+
+// CreateInstanceForUpdate creates an instance and runs `FirstUpdate`.
+// This is a mandatory to run all `update-index` commands
+func CreateInstanceForUpdate() *rpc.Instance {
+	// We don't initialize any CoreInstance when updating indexes since we don't need to.
+	// Also meaningless errors might be returned when calling this command with --additional-urls
+	// since the CLI would be searching for a corresponding file for the additional urls set
+	// as argument but none would be obviously found.
+	inst, status := Create()
+	if status != nil {
+		feedback.Errorf(tr("Error creating instance: %v"), status)
+		os.Exit(errorcodes.ErrGeneric)
+	}
+
+	// In case this is the first time the CLI is run we need to update indexes
+	// to make it work correctly, we must do this explicitly in this command since
+	// we must use instance.Create instead of instance.CreateAndInit for the
+	// reason stated above.
+	if err := FirstUpdate(inst); err != nil {
+		feedback.Errorf(tr("Error updating indexes: %v"), status)
+		os.Exit(errorcodes.ErrGeneric)
+	}
+	return inst
+}

cli/lib/update_index.go

@@ -25,43 +25,31 @@ import (
 	"github.com/arduino/arduino-cli/cli/output"
 	"github.com/arduino/arduino-cli/commands"
 	rpc "github.com/arduino/arduino-cli/rpc/cc/arduino/cli/commands/v1"
+	"github.com/sirupsen/logrus"
 	"github.com/spf13/cobra"
 )
 
 func initUpdateIndexCommand() *cobra.Command {
-	return &cobra.Command{
+	updateIndexCommand := &cobra.Command{
 		Use:     "update-index",
 		Short:   tr("Updates the libraries index."),
 		Long:    tr("Updates the libraries index to the latest version."),
 		Example: "  " + os.Args[0] + " lib update-index",
 		Args:    cobra.NoArgs,
-		Run: func(cmd *cobra.Command, args []string) {
-			// We don't initialize any CoreInstance when updating indexes since we don't need to.
-			// Also meaningless errors might be returned when calling this command with --additional-urls
-			// since the CLI would be searching for a corresponding file for the additional urls set
-			// as argument but none would be obviously found.
-			inst, status := instance.Create()
-			if status != nil {
-				feedback.Errorf(tr("Error creating instance: %v"), status)
-				os.Exit(errorcodes.ErrGeneric)
-			}
+		Run:     runUpdateIndexCommand,
+	}
+	return updateIndexCommand
+}
 
-			// In case this is the first time the CLI is run we need to update indexes
-			// to make it work correctly, we must do this explicitly in this command since
-			// we must use instance.Create instead of instance.CreateAndInit for the
-			// reason stated above.
-			if err := instance.FirstUpdate(inst); err != nil {
-				feedback.Errorf(tr("Error updating indexes: %v"), status)
-				os.Exit(errorcodes.ErrGeneric)
-			}
+func runUpdateIndexCommand(cmd *cobra.Command, args []string) {
+	logrus.Info("Executing `arduino lib update-index`")
+	inst := instance.CreateInstanceForUpdate()
 
-			err := commands.UpdateLibrariesIndex(context.Background(), &rpc.UpdateLibrariesIndexRequest{
-				Instance: inst,
-			}, output.ProgressBar())
-			if err != nil {
-				feedback.Errorf(tr("Error updating library index: %v"), err)
-				os.Exit(errorcodes.ErrGeneric)
-			}
-		},
+	err := commands.UpdateLibrariesIndex(context.Background(), &rpc.UpdateLibrariesIndexRequest{
+		Instance: inst,
+	}, output.ProgressBar())
+	if err != nil {
+		feedback.Errorf(tr("Error updating library index: %v"), err)
+		os.Exit(errorcodes.ErrGeneric)
 	}
 }

cli/update/update.go

@@ -53,24 +53,7 @@ var updateFlags struct {
 
 func runUpdateCommand(cmd *cobra.Command, args []string) {
 	logrus.Info("Executing `arduino update`")
-	// We don't initialize any CoreInstance when updating indexes since we don't need to.
-	// Also meaningless errors might be returned when calling this command with --additional-urls
-	// since the CLI would be searching for a corresponding file for the additional urls set
-	// as argument but none would be obviously found.
-	inst, status := instance.Create()
-	if status != nil {
-		feedback.Errorf(tr("Error creating instance: %v"), status)
-		os.Exit(errorcodes.ErrGeneric)
-	}
-
-	// In case this is the first time the CLI is run we need to update indexes
-	// to make it work correctly, we must do this explicitly in this command since
-	// we must use instance.Create instead of instance.CreateAndInit for the
-	// reason stated above.
-	if err := instance.FirstUpdate(inst); err != nil {
-		feedback.Errorf(tr("Error updating indexes: %v"), status)
-		os.Exit(errorcodes.ErrGeneric)
-	}
+	inst := instance.CreateInstanceForUpdate()
 
 	err := commands.UpdateCoreLibrariesIndex(context.Background(), &rpc.UpdateCoreLibrariesIndexRequest{
 		Instance: inst,

i18n/data/en.po

@@ -440,7 +440,7 @@ msgstr "Connected to %s! Press CTRL-C to exit."
 msgid "Core"
 msgstr "Core"
 
-#: cli/update/update.go:99
+#: cli/update/update.go:82
 msgid "Core name"
 msgstr "Core name"
 
@@ -639,11 +639,9 @@ msgid "Error copying output file %s"
 msgstr "Error copying output file %s"
 
 #: cli/core/search.go:66
-#: cli/core/update_index.go:52
 #: cli/instance/instance.go:42
+#: cli/instance/instance.go:153
 #: cli/lib/search.go:57
-#: cli/lib/update_index.go:45
-#: cli/update/update.go:62
 msgid "Error creating instance: %v"
 msgstr "Error creating instance: %v"
 
@@ -800,7 +798,7 @@ msgstr "Error in FQBN: %s"
 #: cli/core/search.go:81
 #: cli/instance/instance.go:46
 #: cli/lib/search.go:71
-#: cli/update/update.go:87
+#: cli/update/update.go:70
 msgid "Error initializing instance: %v"
 msgstr "Error initializing instance: %v"
 
@@ -865,7 +863,7 @@ msgid "Error retrieving core list: %v"
 msgstr "Error retrieving core list: %v"
 
 #: cli/outdated/outdated.go:57
-#: cli/update/update.go:94
+#: cli/update/update.go:77
 msgid "Error retrieving outdated cores and libraries: %v"
 msgstr "Error retrieving outdated cores and libraries: %v"
 
@@ -920,23 +918,21 @@ msgstr "Error uninstalling platform %s"
 msgid "Error uninstalling tool %s"
 msgstr "Error uninstalling tool %s"
 
-#: cli/update/update.go:79
+#: cli/update/update.go:62
 msgid "Error updating core and libraries index: %v"
 msgstr "Error updating core and libraries index: %v"
 
 #: cli/core/search.go:75
-#: cli/core/update_index.go:69
+#: cli/core/update_index.go:52
 msgid "Error updating index: %v"
 msgstr "Error updating index: %v"
 
-#: cli/core/update_index.go:61
-#: cli/lib/update_index.go:54
-#: cli/update/update.go:71
+#: cli/instance/instance.go:162
 msgid "Error updating indexes: %v"
 msgstr "Error updating indexes: %v"
 
 #: cli/lib/search.go:66
-#: cli/lib/update_index.go:62
+#: cli/lib/update_index.go:52
 msgid "Error updating library index: %v"
 msgstr "Error updating library index: %v"
 
@@ -1176,8 +1172,8 @@ msgstr "Installed %s"
 
 #: cli/outdated/outdated.go:62
 #: cli/outdated/outdated.go:72
-#: cli/update/update.go:99
-#: cli/update/update.go:109
+#: cli/update/update.go:82
+#: cli/update/update.go:92
 msgid "Installed version"
 msgstr "Installed version"
 
@@ -1349,7 +1345,7 @@ msgid "Library installed"
 msgstr "Library installed"
 
 #: cli/outdated/outdated.go:72
-#: cli/update/update.go:109
+#: cli/update/update.go:92
 msgid "Library name"
 msgstr "Library name"
 
@@ -1497,8 +1493,8 @@ msgstr "Name: \"%s\""
 
 #: cli/outdated/outdated.go:62
 #: cli/outdated/outdated.go:72
-#: cli/update/update.go:99
-#: cli/update/update.go:109
+#: cli/update/update.go:82
+#: cli/update/update.go:92
 msgid "New version"
 msgstr "New version"
 
@@ -2178,11 +2174,11 @@ msgstr "Updates the index of cores to the latest version."
 msgid "Updates the index of cores."
 msgstr "Updates the index of cores."
 
-#: cli/lib/update_index.go:35
+#: cli/lib/update_index.go:36
 msgid "Updates the libraries index to the latest version."
 msgstr "Updates the libraries index to the latest version."
 
-#: cli/lib/update_index.go:34
+#: cli/lib/update_index.go:35
 msgid "Updates the libraries index."
 msgstr "Updates the libraries index."