Add external programmers listing to board details commmand

cli/board/details.go

@@ -34,6 +34,7 @@ import (
 var tr = i18n.Tr
 var showFullDetails bool
 var fqbn string
+var listProgrammers bool
 
 func initDetailsCommand() *cobra.Command {
 	var detailsCommand = &cobra.Command{
@@ -47,6 +48,7 @@ func initDetailsCommand() *cobra.Command {
 
 	detailsCommand.Flags().BoolVarP(&showFullDetails, "full", "f", false, tr("Show full board details"))
 	detailsCommand.Flags().StringVarP(&fqbn, "fqbn", "b", "", "Fully Qualified Board Name, e.g.: arduino:avr:uno")
+	detailsCommand.Flags().BoolVarP(&listProgrammers, "list-programmers", "", false, tr("Show list of available programmers"))
 	// detailsCommand.MarkFlagRequired("fqbn") // enable once `board details <fqbn>` is removed
 
 	return detailsCommand
@@ -89,6 +91,16 @@ func (dr detailsResult) Data() interface{} {
 
 func (dr detailsResult) String() string {
 	details := dr.details
+
+	if listProgrammers {
+		t := table.New()
+		t.AddRow(tr("Id"), tr("Programmer name"))
+		for _, programmer := range details.Programmers {
+			t.AddRow(programmer.GetId(), programmer.GetName())
+		}
+		return t.Render()
+	}
+
 	// Table is 4 columns wide:
 	// |               |                             | |                       |
 	// Board name:     Arduino Nano
@@ -180,5 +192,10 @@ func (dr detailsResult) String() string {
 		}
 	}
 
+	t.AddRow(tr("Programmers:"), tr("Id"), tr("Name"))
+	for _, programmer := range details.Programmers {
+		t.AddRow("", programmer.GetId(), programmer.GetName())
+	}
+
 	return t.Render()
 }

cli/upload/upload.go

@@ -68,21 +68,6 @@ func run(command *cobra.Command, args []string) {
 		os.Exit(errorcodes.ErrGeneric)
 	}
 
-	if programmer == "list" {
-		resp, err := upload.ListProgrammersAvailableForUpload(context.Background(), &rpc.ListProgrammersAvailableForUploadReq{
-			Instance: instance,
-			Fqbn:     fqbn,
-		})
-		if err != nil {
-			feedback.Errorf("Error listing programmers: %v", err)
-			os.Exit(errorcodes.ErrGeneric)
-		}
-		feedback.PrintResult(&programmersList{
-			Programmers: resp.GetProgrammers(),
-		})
-		os.Exit(0)
-	}
-
 	var path *paths.Path
 	if len(args) > 0 {
 		path = paths.New(args[0])

commands/board/details.go

@@ -129,5 +129,14 @@ func Details(ctx context.Context, req *rpc.BoardDetailsReq) (*rpc.BoardDetailsRe
 		})
 	}
 
+	details.Programmers = []*rpc.Programmer{}
+	for id, p := range boardPlatform.Programmers {
+		details.Programmers = append(details.Programmers, &rpc.Programmer{
+			Platform: boardPlatform.Platform.Name,
+			Id:       id,
+			Name:     p.Name,
+		})
+	}
+
 	return details, nil
 }

go.mod

@@ -20,7 +20,7 @@ require (
 	github.com/fluxio/iohelpers v0.0.0-20160419043813-3a4dd67a94d2 // indirect
 	github.com/fluxio/multierror v0.0.0-20160419044231-9c68d39025e5 // indirect
 	github.com/gofrs/uuid v3.2.0+incompatible
-	github.com/golang/protobuf v1.4.1
+	github.com/golang/protobuf v1.4.2
 	github.com/h2non/filetype v1.0.8 // indirect
 	github.com/imjasonmiller/godice v0.1.2
 	github.com/juju/loggo v0.0.0-20190526231331-6e530bcce5d8 // indirect

go.sum

@@ -78,13 +78,17 @@ github.com/golang/mock v1.1.1/go.mod h1:oTYuIxOrZwtPieC+H1uAHpcLFnEyAGVDL/k47Jfb
 github.com/golang/protobuf v1.2.0/go.mod h1:6lQm79b+lXiMfvg/cZm0SGofjICqVBUtrP5yJMmIC1U=
 github.com/golang/protobuf v1.3.1/go.mod h1:6lQm79b+lXiMfvg/cZm0SGofjICqVBUtrP5yJMmIC1U=
 github.com/golang/protobuf v1.3.2/go.mod h1:6lQm79b+lXiMfvg/cZm0SGofjICqVBUtrP5yJMmIC1U=
+github.com/golang/protobuf v1.3.3 h1:gyjaxf+svBWX08ZjK86iN9geUJF0H6gp2IRKX6Nf6/I=
+github.com/golang/protobuf v1.3.3/go.mod h1:vzj43D7+SQXF/4pzW/hwtAqwc6iTitCiVSaWz5lYuqw=
 github.com/golang/protobuf v1.4.0-rc.1/go.mod h1:ceaxUfeHdC40wWswd/P6IGgMaK3YpKi5j83Wpe3EHw8=
 github.com/golang/protobuf v1.4.0-rc.1.0.20200221234624-67d41d38c208/go.mod h1:xKAWHe0F5eneWXFV3EuXVDTCmh+JuBKY0li0aMyXATA=
 github.com/golang/protobuf v1.4.0-rc.2/go.mod h1:LlEzMj4AhA7rCAGe4KMBDvJI+AwstrUpVNzEA03Pprs=
 github.com/golang/protobuf v1.4.0-rc.4.0.20200313231945-b860323f09d0/go.mod h1:WU3c8KckQ9AFe+yFwt9sWVRKCVIyN9cPHBJSNnbL67w=
 github.com/golang/protobuf v1.4.0/go.mod h1:jodUvKwWbYaEsadDk5Fwe5c77LiNKVO9IDvqG2KuDX0=
 github.com/golang/protobuf v1.4.1 h1:ZFgWrT+bLgsYPirOnRfKLYJLvssAegOj/hgyMFdJZe0=
 github.com/golang/protobuf v1.4.1/go.mod h1:U8fpvMrcmy5pZrNK1lt4xCsGvpyWQ/VVv6QDs8UjoX8=
+github.com/golang/protobuf v1.4.2 h1:+Z5KGCizgyZCbGh1KZqA0fcLLkwbsjIzS4aV2v7wJX0=
+github.com/golang/protobuf v1.4.2/go.mod h1:oDoupMAO8OvCJWAcko0GGGIgR6R6ocIYbsSw735rRwI=
 github.com/google/btree v1.0.0/go.mod h1:lNA+9X1NB3Zf8V7Ke586lFgjr2dZNuvo3lPJSGZ5JPQ=
 github.com/google/go-cmp v0.2.0 h1:+dTQ8DZQJz0Mb/HjFlkptS1FeQ4cWSnN941F8aEG4SQ=
 github.com/google/go-cmp v0.2.0/go.mod h1:oXzfMopK8JAjlY9xF4vHSVASa0yLyX7SntLO5aqRK0M=
@@ -307,6 +311,8 @@ google.golang.org/appengine v1.1.0/go.mod h1:EbEs0AVv82hx2wNQdGPgUI5lhzA/G0D9Ywl
 google.golang.org/appengine v1.4.0/go.mod h1:xpcJRLb0r/rnEns0DIKYYv+WjYCduHsrkT7/EB5XEv4=
 google.golang.org/genproto v0.0.0-20180817151627-c66870c02cf8/go.mod h1:JiN7NxoALGmiZfu7CAH4rXhgtRTLTxftemlI0sWmxmc=
 google.golang.org/genproto v0.0.0-20190819201941-24fa4b261c55/go.mod h1:DMBHOl98Agz4BDEuKkezgsaosCRResVns1a3J2ZsMNc=
+google.golang.org/genproto v0.0.0-20200204135345-fa8e72b47b90 h1:7THRSvPuzF1bql5kyFzX0JM0vpGhwuhskgJrJsbZ80Y=
+google.golang.org/genproto v0.0.0-20200204135345-fa8e72b47b90/go.mod h1:GmwEX6Z4W5gMy59cAlVYjN9JhxgbQH6Gn+gFDQe2lzA=
 google.golang.org/genproto v0.0.0-20200526211855-cb27e3aa2013 h1:+kGHl1aib/qcwaRi1CbqBZ1rk19r85MNUf8HaBghugY=
 google.golang.org/genproto v0.0.0-20200526211855-cb27e3aa2013/go.mod h1:NbSheEEYHJ7i3ixzK3sjbqSGDJWnxyFXZblF3eUsNvo=
 google.golang.org/grpc v1.19.0/go.mod h1:mqu4LbDTu4XGKhr4mRzUsmM4RtVoemTSY81AxZiDr8c=
@@ -320,6 +326,7 @@ google.golang.org/protobuf v0.0.0-20200228230310-ab0ca4ff8a60/go.mod h1:cfTl7dwQ
 google.golang.org/protobuf v1.20.1-0.20200309200217-e05f789c0967/go.mod h1:A+miEFZTKqfCUM6K7xSMQL9OKL/b6hQv+e19PK+JZNE=
 google.golang.org/protobuf v1.21.0/go.mod h1:47Nbq4nVaFHyn7ilMalzfO3qCViNmqZ2kzikPIcrTAo=
 google.golang.org/protobuf v1.22.0/go.mod h1:EGpADcykh3NcUnDUJcl1+ZksZNG86OlYog2l/sGQquU=
+google.golang.org/protobuf v1.23.0/go.mod h1:EGpADcykh3NcUnDUJcl1+ZksZNG86OlYog2l/sGQquU=
 google.golang.org/protobuf v1.23.1-0.20200526195155-81db48ad09cc/go.mod h1:EGpADcykh3NcUnDUJcl1+ZksZNG86OlYog2l/sGQquU=
 google.golang.org/protobuf v1.25.0 h1:Ejskq+SyPohKW+1uil0JJMtmHCgJPJ/qWTxr8qp+R4c=
 google.golang.org/protobuf v1.25.0/go.mod h1:9JNX74DMeImyA3h4bdi1ymwjUzf21/xIlbajtzgsN7c=

i18n/data/en.po

@@ -13,28 +13,28 @@ msgstr "Aliases:"
 msgid "Available Commands:"
 msgstr "Available Commands:"
 
-#: cli/board/details.go:112
+#: cli/board/details.go:124
 msgid "Board name:"
 msgstr "Board name:"
 
-#: cli/board/details.go:114
+#: cli/board/details.go:126
 msgid "Board version:"
 msgstr "Board version:"
 
-#: cli/board/details.go:157
+#: cli/board/details.go:169
 msgid "Checksum:"
 msgstr "Checksum:"
 
-#: cli/board/details.go:58
-#: cli/board/details.go:73
+#: cli/board/details.go:60
+#: cli/board/details.go:75
 msgid "Error getting board details: %v"
 msgstr "Error getting board details: %v"
 
 #: cli/usage.go:27
 msgid "Examples:"
 msgstr "Examples:"
 
-#: cli/board/details.go:155
+#: cli/board/details.go:167
 msgid "File:"
 msgstr "File:"
 
@@ -46,87 +46,108 @@ msgstr "Flags:"
 msgid "Global Flags:"
 msgstr "Global Flags:"
 
-#: cli/board/details.go:125
+#: cli/board/details.go:97
+#: cli/board/details.go:195
+msgid "Id"
+msgstr "Id"
+
+#: cli/board/details.go:137
 msgid "Identification properties:"
 msgstr "Identification properties:"
 
-#: cli/board/details.go:154
+#: cli/board/details.go:195
+msgid "Name"
+msgstr "Name"
+
+#: cli/board/details.go:166
 msgid "OS:"
 msgstr "OS:"
 
-#: cli/board/details.go:118
+#: cli/board/details.go:130
 msgid "Official Arduino board:"
 msgstr "Official Arduino board:"
 
-#: cli/board/details.go:166
+#: cli/board/details.go:178
 msgid "Option:"
 msgstr "Option:"
 
-#: cli/board/details.go:134
+#: cli/board/details.go:146
 msgid "Package URL:"
 msgstr "Package URL:"
 
-#: cli/board/details.go:133
+#: cli/board/details.go:145
 msgid "Package maintainer:"
 msgstr "Package maintainer:"
 
-#: cli/board/details.go:132
+#: cli/board/details.go:144
 msgid "Package name:"
 msgstr "Package name:"
 
-#: cli/board/details.go:136
+#: cli/board/details.go:148
 msgid "Package online help:"
 msgstr "Package online help:"
 
-#: cli/board/details.go:135
+#: cli/board/details.go:147
 msgid "Package website:"
 msgstr "Package website:"
 
-#: cli/board/details.go:142
+#: cli/board/details.go:154
 msgid "Platform URL:"
 msgstr "Platform URL:"
 
-#: cli/board/details.go:141
+#: cli/board/details.go:153
 msgid "Platform architecture:"
 msgstr "Platform architecture:"
 
-#: cli/board/details.go:140
+#: cli/board/details.go:152
 msgid "Platform category:"
 msgstr "Platform category:"
 
-#: cli/board/details.go:147
+#: cli/board/details.go:159
 msgid "Platform checksum:"
 msgstr "Platform checksum:"
 
-#: cli/board/details.go:143
+#: cli/board/details.go:155
 msgid "Platform file name:"
 msgstr "Platform file name:"
 
-#: cli/board/details.go:139
+#: cli/board/details.go:151
 msgid "Platform name:"
 msgstr "Platform name:"
 
-#: cli/board/details.go:145
+#: cli/board/details.go:157
 msgid "Platform size (bytes):"
 msgstr "Platform size (bytes):"
 
-#: cli/board/details.go:41
+#: cli/board/details.go:42
 msgid "Print details about a board."
 msgstr "Print details about a board."
 
-#: cli/board/details.go:151
+#: cli/board/details.go:97
+msgid "Programmer name"
+msgstr "Programmer name"
+
+#: cli/board/details.go:195
+msgid "Programmers:"
+msgstr "Programmers:"
+
+#: cli/board/details.go:163
 msgid "Required tool:"
 msgstr "Required tool:"
 
-#: cli/board/details.go:48
+#: cli/board/details.go:49
 msgid "Show full board details"
 msgstr "Show full board details"
 
-#: cli/board/details.go:42
+#: cli/board/details.go:43
 msgid "Show information about a board, in particular if the board has options to be specified in the FQBN."
 msgstr "Show information about a board, in particular if the board has options to be specified in the FQBN."
 
-#: cli/board/details.go:156
+#: cli/board/details.go:51
+msgid "Show list of available programmers"
+msgstr "Show list of available programmers"
+
+#: cli/board/details.go:168
 msgid "Size (bytes):"
 msgstr "Size (bytes):"