[devtools] Fix dbgconf handling when not present in PDSC

Author: brondani

libs/rtemodel/src/RteProject.cpp

@@ -1405,7 +1405,9 @@ void RteProject::CollectSettings(const string& targetName)
   t->AddDeviceProperties(d, processorName);
   // add dbgconf file
   auto debugVars = t->GetDeviceDebugVars();
-  AddFileInstance(nullptr, debugVars, 0, t);
+  if (debugVars && debugVars->HasAttribute("configfile")) {
+    AddFileInstance(nullptr, debugVars, 0, t);
+  }
 
   // collect copied files and sources
   for (auto [_,fi] : m_files) {

test/packs/ARM/RteTest_DFP/0.1.1/ARM.RteTest_DFP.pdsc

@@ -27,7 +27,7 @@
 	 RteTest ARM M Device Family
       </description>
       <algorithm name="Device/ARM/Flash/FAMILY.FLM" start="0x00000000" size="0x00040000"  default="1"/>
-      <debugvars configfile="Device/ARM/Debug/ARMCM.dbgconf">
+      <debugvars>
         __var DbgMCU_CR      = 0x00000007;   // DBGMCU_CR: DBG_SLEEP, DBG_STOP, DBG_STANDBY
         __var TraceClk_Pin   = 0x00040002;   // PE2
         __var TraceD0_Pin    = 0x00040003;   // PE3

tools/projmgr/test/data/TestRunDebug/no-dbgconf.csolution.yml

@@ -0,0 +1,19 @@
+# yaml-language-server: $schema=https://raw.githubusercontent.com/Open-CMSIS-Pack/devtools/main/tools/projmgr/schemas/csolution.schema.json
+
+solution:
+
+  compiler: AC6
+
+  target-types:
+    - type: ARMCM3
+      device: RteTest_ARMCM3
+      target-set:
+        - set:
+          images:
+            - project-context: run-debug
+
+  projects:
+    - project: run-debug.cproject.yml
+
+  packs:
+    - pack: ARM::RteTest_DFP@0.1.1

tools/projmgr/test/src/ProjMgrUnitTests.cpp

@@ -6721,6 +6721,23 @@ TEST_F(ProjMgrUnitTests, TestRunDebug) {
     testinput_folder + "/TestRunDebug/ref/run-debug+TestHW2.cbuild.yml");
 }
 
+TEST_F(ProjMgrUnitTests, TestNoDbgconf) {
+  char* argv[7];
+  const string& csolution = testinput_folder + "/TestRunDebug/no-dbgconf.csolution.yml";
+  argv[1] = (char*)"convert";
+  argv[2] = (char*)csolution.c_str();
+  argv[3] = (char*)"-o";
+  argv[4] = (char*)testoutput_folder.c_str();
+  argv[5] = (char*)"--active";
+  argv[6] = (char*)"ARMCM3";
+  EXPECT_EQ(0, RunProjMgr(7, argv, m_envp));
+
+  const YAML::Node& cbuild = YAML::LoadFile(testoutput_folder + "/run-debug+ARMCM3.cbuild.yml");
+  EXPECT_FALSE(cbuild["build"]["dbgconf"].IsDefined());
+  const YAML::Node& cbuildrun = YAML::LoadFile(testoutput_folder + "/no-dbgconf+ARMCM3.cbuild-run.yml");
+  EXPECT_FALSE(cbuildrun["cbuild-run"]["debugger"]["dbgconf"].IsDefined());
+}
+
 TEST_F(ProjMgrUnitTests, TestRunDebugMulticore) {
   char* argv[7];
   const string& csolution = testinput_folder + "/TestRunDebug/run-debug.csolution.yml";