tools: fix make host_info flag parsing and config string escaping

Fix incorrect flag handling and string escaping in the `make host_info`
diagnostic target.

Previously, CFLAGS, CXXFLAGS, and LDFLAGS were passed in a form that caused
improper splitting and quoting, which resulted in malformed output and
incorrectly escaped configuration values such as CONFIG_APPS_DIR.

This change ensures that:
- Compilation flags are passed as proper shell strings
- Flags are split correctly using shlex
- Configuration values are escaped exactly once when generating sysinfo.h
- Parsed output matches the contents of the .config file

This change affects diagnostic output only and does not modify the NuttX
build process or generated binaries.

Signed-off-by: Adwait Godbole <adwaitngodbole@gmail.com>

Author: adwait-godbole

include/.gitignore

@@ -14,3 +14,4 @@
 /metal
 /etl
 /minmea
+/sysinfo.h

tools/Unix.mk

@@ -633,9 +633,9 @@ SYSINFO_PARSE_FLAGS += "-finclude/sysinfo.h"
 
 SYSINFO_FLAGS = "-c"
 SYSINFO_FLAGS += "-p"
-SYSINFO_FLAGS += -f \""$(shell echo '$(CFLAGS)' | sed 's/"/\\\\\\"/g')"\"
-SYSINFO_FLAGS += \""$(shell echo '$(CXXFLAGS)' | sed 's/"/\\\\\\"/g')"\"
-SYSINFO_FLAGS += \""$(shell echo '$(LDFLAGS)' | sed 's/"/\\\\\\"/g')"\"
+SYSINFO_FLAGS += -f "$(CFLAGS)"
+SYSINFO_FLAGS += "$(CXXFLAGS)"
+SYSINFO_FLAGS += "$(LDFLAGS)"
 SYSINFO_FLAGS += "--target_info"
 
 # host_info: Parse nxdiag example output file (sysinfo.h) and print

tools/host_info_dump.py

@@ -23,6 +23,7 @@
 import os
 import platform
 import re
+import shlex
 import subprocess
 import sys
 
@@ -297,6 +298,11 @@ def get_os_version():
     return sys_id
 
 
+FLAGS_WITH_ARGS = {
+    "-isystem",
+}
+
+
 def get_compilation_flags(flags):
     """
     Gets the compilation flags used to compile the NuttX source code and splits
@@ -310,13 +316,23 @@ def get_compilation_flags(flags):
     """
 
     if not flags:
-        return [""]
+        return []
+
+    tokens = shlex.split(flags)
+    merged = []
+    i = 0
 
-    flag_list = flags.split(" -")
-    flag_list[0] = flag_list[0][1:]
-    flag_list = ["-" + flag for flag in flag_list]
+    while i < len(tokens):
+        tok = tokens[i]
 
-    return flag_list
+        if tok in FLAGS_WITH_ARGS and i + 1 < len(tokens):
+            merged.append(f"{tok} {tokens[i + 1]}")
+            i += 2
+        else:
+            merged.append(tok)
+            i += 1
+
+    return merged
 
 
 def generate_header(args):
@@ -355,13 +371,6 @@ def generate_header(args):
     if args.flags:
         cflags, cxxflags, ldflags = args.flags
 
-        if cflags:
-            cflags = cflags[1:-1]
-        if cxxflags:
-            cxxflags = cxxflags[1:-1]
-        if ldflags:
-            ldflags = ldflags[1:-1]
-
         info["NUTTX_CFLAGS"] = get_compilation_flags(cflags)
         info["NUTTX_CXXFLAGS"] = get_compilation_flags(cxxflags)
         info["NUTTX_LDFLAGS"] = get_compilation_flags(ldflags)
@@ -370,24 +379,27 @@ def generate_header(args):
             len(info["NUTTX_CFLAGS"])
         )
         output += "static const char *NUTTX_CFLAGS[NUTTX_CFLAGS_ARRAY_SIZE] =\n{\n"
-        for i in range(len(info["NUTTX_CFLAGS"])):
-            output += '  "' + info["NUTTX_CFLAGS"][i] + '",\n'
+        for flag in info["NUTTX_CFLAGS"]:
+            flag = flag.replace('"', '\\"')
+            output += '  "' + flag + '",\n'
         output += "};\n\n"
 
         output += "#define NUTTX_CXXFLAGS_ARRAY_SIZE {}\n".format(
             len(info["NUTTX_CXXFLAGS"])
         )
         output += "static const char *NUTTX_CXXFLAGS[NUTTX_CXXFLAGS_ARRAY_SIZE] =\n{\n"
-        for i in range(len(info["NUTTX_CXXFLAGS"])):
-            output += '  "' + info["NUTTX_CXXFLAGS"][i] + '",\n'
+        for flag in info["NUTTX_CXXFLAGS"]:
+            flag = flag.replace('"', '\\"')
+            output += '  "' + flag + '",\n'
         output += "};\n\n"
 
         output += "#define NUTTX_LDFLAGS_ARRAY_SIZE {}\n".format(
             len(info["NUTTX_LDFLAGS"])
         )
         output += "static const char *NUTTX_LDFLAGS[NUTTX_LDFLAGS_ARRAY_SIZE] =\n{\n"
-        for i in range(len(info["NUTTX_LDFLAGS"])):
-            output += '  "' + info["NUTTX_LDFLAGS"][i] + '",\n'
+        for flag in info["NUTTX_LDFLAGS"]:
+            flag = flag.replace('"', '\\"')
+            output += '  "' + flag + '",\n'
         output += "};\n\n"
 
     # NuttX Configuration
@@ -411,7 +423,7 @@ def generate_header(args):
         )
         output += "static const char *NUTTX_CONFIG[NUTTX_CONFIG_ARRAY_SIZE] =\n{\n"
         for i in range(len(info["NUTTX_CONFIG"])):
-            output += '  "' + info["NUTTX_CONFIG"][i].replace('"', '\\"') + '",\n'
+            output += "  " + info["NUTTX_CONFIG"][i] + ",\n"
         output += "};\n\n"
 
     # OS Version

tools/host_info_parse.py

@@ -1,5 +1,5 @@
 #!/usr/bin/env python3
-# tools/parse_sysinfo.py
+# tools/host_info_parse.py
 #
 # SPDX-License-Identifier: Apache-2.0
 #
@@ -38,7 +38,7 @@ def parse_information_from_header(file_path):
     """
 
     VARIABLE_NAMES_REGEX = r"static\s+const\s+char\s+\**([A-Za-z0-9_]+)\s*"
-    VARIABLE_VALUES_REGEX = r'"([^"]*)"|{([^}]+)};'
+    VARIABLE_VALUES_REGEX = r'\{([^}]+)\}|=\s*"([^"]*)"'
     result = {}
     var_name_to_print_dict = {
         "NUTTX_CFLAGS": "NuttX CFLAGS",
@@ -75,18 +75,24 @@ def parse_information_from_header(file_path):
 
         # Process values to print it prettier
 
-        for i in range(len(values_array)):
-            tmp_list = []
-            for y in range(len(values_array[i])):
-                tmp_str = values_array[i][y]
-                tmp_str = tmp_str.replace('"', "")
-                tmp_str = tmp_str.replace("\n  ", "", 1)
-                tmp_str = tmp_str.replace(",", "")
-
-                if tmp_str != "":
-                    tmp_list.append(tmp_str)
-
-            values_array[i] = tuple(tmp_list)
+        processed_values = []
+
+        for array_block, single_value in values_array:
+            if array_block:
+                items = []
+                for line in array_block.splitlines():
+                    line = line.strip()
+                    if not line or line == "{":
+                        continue
+                    line = line.rstrip(",")
+                    if line.startswith('"') and line.endswith('"'):
+                        line = line[1:-1]
+                    items.append(line)
+                processed_values.append(tuple(items))
+            else:
+                processed_values.append((single_value,))
+
+            values_array = processed_values
 
         keys_values_to_return = [var_name_to_print_dict[x] for x in keys_array]
         result = dict(zip(keys_values_to_return, values_array))