cmake: Consistent expansion of include path

Accept both space and semicolon separated paths and defines from cmake.
The add_custom_command really wants to replace the semicolons with
spaces, whereas the 'file' command doesn't.  To make this work, have the
build.rs split on either space or semicolon.

This fixes a problem where `cargo check` and `rust-analyzer` fail to run
correctly when the Zephyr build is configured for release, and they are
trying to do a debug build.  With this fix, the debug build from cargo
will work, allowing the analysis or verification.

Signed-off-by: David Brown <[email protected]>

Author: d3zd3z

CMakeLists.txt

@@ -57,7 +57,7 @@ endfunction()
 function(get_include_dirs target dirs)
   get_target_property(include_dirs ${target} INTERFACE_INCLUDE_DIRECTORIES)
   if(include_dirs)
-    set(${dirs} ${include_dirs} PARENT_SCOPE)
+    set(${dirs} "${include_dirs}" PARENT_SCOPE)
   else()
     set(${dirs} "" PARENT_SCOPE)
   endif()

zephyr-sys/build.rs

@@ -108,7 +108,9 @@ fn main() -> Result<()> {
 fn define_args(bindings: Builder, prefix: &str, var_name: &str) -> Builder {
     let text = env::var(var_name).unwrap();
     let mut bindings = bindings;
-    for entry in text.split(" ") {
+    // Split on either spaces or semicolons, to allow some flexibility in what cmake might generate
+    // for us.
+    for entry in text.split(&[' ', ';']) {
         if entry.is_empty() {
             continue;
         }