[skip-changelog] fixed verbose '1200-bps touch' notice (arduino#726)

cmaglie · web-flow · commit f5b41257d7e0 · 2020-05-27T13:04:18.000+02:00
Previously, if there were many serial port connected, multiple Performing 1200-bps touch reset on serial port /dev/ttyACMxx were printed due to misplacement of the verbose print.
diff --git a/commands/upload/upload.go b/commands/upload/upload.go
@@ -172,11 +172,11 @@ func Upload(ctx context.Context, req *rpc.UploadReq, outStream io.Writer, errStr
 			return nil, fmt.Errorf("cannot get serial port list: %s", err)
 		}
 		for _, p := range ports {
-			if req.GetVerbose() {
-				outStream.Write([]byte(fmt.Sprintf("Performing 1200-bps touch reset on serial port %s", p)))
-				outStream.Write([]byte(fmt.Sprintln()))
-			}
 			if p == port {
+				if req.GetVerbose() {
+					outStream.Write([]byte(fmt.Sprintf("Performing 1200-bps touch reset on serial port %s", p)))
+					outStream.Write([]byte(fmt.Sprintln()))
+				}
 				if err := touchSerialPortAt1200bps(p); err != nil {
 					return nil, fmt.Errorf("cannot perform reset: %s", err)
 				}
