cmake: change SDP ASM developer mode to not replace ASM files

Change SDP ASM developer mode to not replace ASM files in source
directory. Instead build includes the temporary files from build
directory.
This change is needed so that CI would pass after adding test
that enables assertions.

Signed-off-by: Magdalena Pastula <[email protected]>

Author: magp-nordic

cmake/sdp.cmake

@@ -20,19 +20,18 @@ function(sdp_assembly_install target hrt_srcs)
   sdp_assembly_prepare_install(${CONFIG_SOC} "${hrt_srcs}")
   sdp_assembly_target_sources(${CONFIG_SOC} ${target} "${hrt_srcs}")
 
-  if(CONFIG_SDP_DEVELOPER_MODE)
-    add_dependencies(asm_install asm_check)
-    add_dependencies(${target} asm_install)
-  else()
-    add_dependencies(${target} asm_check)
-  endif()
+  add_dependencies(${target} asm_check)
 endfunction()
 
 function(sdp_assembly_target_sources soc target hrt_srcs)
   foreach(hrt_src ${hrt_srcs})
     get_filename_component(hrt_dir ${hrt_src} DIRECTORY)  # directory
     get_filename_component(hrt_src_file ${hrt_src} NAME_WE)  # filename without extension
-    set(hrt_s_file "${hrt_dir}/${hrt_src_file}-${soc}.s")
+    if(CONFIG_SDP_DEVELOPER_MODE)
+      set(hrt_s_file "${hrt_src_file}-${soc}-temp.s")
+    else()
+      set(hrt_s_file "${hrt_dir}/${hrt_src_file}-${soc}.s")
+    endif()
     target_sources(${target} PRIVATE ${hrt_s_file})
   endforeach()
 endfunction()

cmake/sdp_asm_check.cmake

@@ -32,13 +32,15 @@ function(asm_check)
     elseif( compare_result EQUAL 1)
       if(dev_mode)
         message(WARNING "${asm_filename}-${soc}.s ASM file content has changed.\
-                It will be updated and included in build.")
+                The new version will be included in build, but it will not be moved to the source \
+                directory. If you want to update ASM file in source directory with the new one \
+                run `ninja asm_install` in FLPR build directory.")
       else()
         message(FATAL_ERROR "${asm_filename}-${soc}.s ASM file content has changed.\
                 If you want to include the new ASM in build, \
                 please run `ninja asm_install` in FLPR build directory and build again. \
                 If you want to disable this error and include new ASM in build every time,\
-                enable SB_CONFIG_DEVELOPER_MODE option.")
+                enable SB_CONFIG_SDP_DEVELOPER_MODE option.")
       endif()
     else()
       message("Something went wrong when comparing ${asm_filename}-${soc}.s and ${asm_filename}-${soc}-temp.s")