Command 'debug info' choose between "script" and "scripts" configs exclusively

Author: cmaglie

commands/debug/debug_info.go

@@ -159,8 +159,8 @@ func getDebugProperties(req *rpc.GetDebugConfigRequest, pme *packagemanager.Expl
 	case "openocd":
 		openocdProperties := debugProperties.SubTree("server." + server)
 		scripts := openocdProperties.ExtractSubIndexLists("scripts")
-		if s := openocdProperties.Get("script"); s != "" {
-			// backward compatibility
+		if s := openocdProperties.Get("script"); s != "" && len(scripts) == 0 {
+			// backward compatibility: use "script" property if there are no "scipts.N"
 			scripts = append(scripts, s)
 		}
 		openocdConf := &rpc.DebugOpenOCDServerConfiguration{

internal/integrationtest/debug/debug_test.go

@@ -106,13 +106,12 @@ func testAllDebugInformation(t *testing.T, env *integrationtest.Environment, cli
 	require.NoError(t, err)
 
 	// Build sketch
-	fqbn := "my:samd:my"
-	_, _, err = cli.Run("compile", "-b", fqbn, sketchPath.String(), "--format", "json")
+	_, _, err = cli.Run("compile", "-b", "my:samd:my", sketchPath.String(), "--format", "json")
 	require.NoError(t, err)
 
 	{
 		// Starts debugger
-		jsonDebugOut, _, err := cli.Run("debug", "-b", fqbn, "-P", "atmel_ice", sketchPath.String(), "--info", "--format", "json")
+		jsonDebugOut, _, err := cli.Run("debug", "-b", "my:samd:my", "-P", "atmel_ice", sketchPath.String(), "--info", "--format", "json")
 		require.NoError(t, err)
 		debugOut := requirejson.Parse(t, jsonDebugOut)
 		debugOut.MustContain(`
@@ -154,7 +153,7 @@ func testAllDebugInformation(t *testing.T, env *integrationtest.Environment, cli
 
 	// Starts debugger with another programmer
 	{
-		jsonDebugOut, _, err := cli.Run("debug", "-b", fqbn, "-P", "my_cold_ice", sketchPath.String(), "--info", "--format", "json")
+		jsonDebugOut, _, err := cli.Run("debug", "-b", "my:samd:my", "-P", "my_cold_ice", sketchPath.String(), "--info", "--format", "json")
 		require.NoError(t, err)
 		debugOut := requirejson.Parse(t, jsonDebugOut)
 		debugOut.MustContain(`
@@ -192,5 +191,29 @@ func testAllDebugInformation(t *testing.T, env *integrationtest.Environment, cli
 				]
 			}
 		}`)
+
+		{
+			// Starts debugger with an old-style openocd script definition
+			jsonDebugOut, _, err := cli.Run("debug", "-b", "my:samd:my2", "-P", "atmel_ice", sketchPath.String(), "--info", "--format", "json")
+			require.NoError(t, err)
+			debugOut := requirejson.Parse(t, jsonDebugOut)
+			debugOut.MustContain(`
+			{
+				"toolchain": "gcc",
+				"toolchain_path": "gcc-path",
+				"toolchain_prefix": "gcc-prefix",
+				"server": "openocd",
+				"server_path": "openocd-path",
+				"server_configuration": {
+					"path": "openocd-path",
+					"scripts_dir": "openocd-scripts-dir",
+					"scripts": [
+						"single-script"
+					]
+				},
+				"svd_file": "svd-file"
+			}`)
+		}
+
 	}
 }

internal/integrationtest/debug/testdata/hardware/my/samd/boards.txt

@@ -45,3 +45,37 @@ my.debug.cortex-debug.custom.overrideRestartCommands.2=thb setup
 my.debug.cortex-debug.custom.overrideRestartCommands.3=c
 my.debug.cortex-debug.custom.anotherStringParamer=hellooo
 my.debug.svd_file=svd-file
+
+my2.name=My Cool Board
+my2.vid.0=0x2341
+my2.pid.0=0x804e
+my2.upload_port.0.vid=0x2341
+my2.upload_port.0.pid=0x804e
+my2.upload.tool=bossac
+my2.upload.tool.default=bossac
+my2.upload.tool.network=arduino_ota
+my2.upload.protocol=sam-ba
+my2.upload.maximum_size=262144
+my2.upload.maximum_data_size=32768
+my2.upload.use_1200bps_touch=true
+my2.upload.wait_for_upload_port=true
+my2.upload.native_usb=true
+my2.build.mcu=cortex-m0plus
+my2.build.f_cpu=48000000L
+my2.build.usb_product="Arduino MKR1000"
+my2.build.usb_manufacturer="Arduino LLC"
+my2.build.board=SAMD_MY
+my2.build.core=arduino:arduino
+my2.build.extra_flags=-DUSE_ARDUINO_MKR_PIN_LAYOUT -D__SAMD21G18A__ {build.usb_flags}
+my2.build.ldscript=linker_scripts/gcc/flash_with_bootloader.ld
+my2.build.openocdscript=openocd_scripts/arduino_zero.cfg
+my2.build.variant=arduino:mkr1000
+my2.build.vid=0x2341
+my2.build.pid=0x804e
+
+my2.debug.toolchain.path=gcc-path
+my2.debug.toolchain.prefix=gcc-prefix
+my2.debug.server.openocd.path=openocd-path
+my2.debug.server.openocd.scripts_dir=openocd-scripts-dir
+my2.debug.server.openocd.script=single-script
+my2.debug.svd_file=svd-file