michaelh/zephyr
1
0
Fork 0
Code Releases Activity

cmake: dts: getting realpath for system includes for dts.

Browse source 
Fixes: #28893

Now using get_filename_component(REALPATH) to ensure full path with
symlinks and `..` resolved.

Having `..` in path causes the gcc generated dependency file to contain
`\` in path on windows. This confuses ninja.
The result is that ninja consider the file missing, and thus invokes a
CMake run to create the file, resulting in endless loop.

By ensuring that full path (containing `/` which is already ensured)
when calling gcc, then all paths will be posix style, making ninja
happy.

Signed-off-by: Torsten Rasmussen <Torsten.Rasmussen@nordicsemi.no>
This commit is contained in:
Torsten Rasmussen 2020-10-27 13:30:36 +01:00 • committed by Carles Cufí
commit ad78766139
1 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
cmake/dts.cmake
View file

@ -95,7 +95,7 @@ if(SUPPORTS_DTS)
dts/${ARCH}
dts
)
set(full_path ${dts_root}/${dts_root_path})
get_filename_component(full_path ${dts_root}/${dts_root_path} REALPATH)
if(EXISTS ${full_path})
list(APPEND
DTS_ROOT_SYSTEM_INCLUDE_DIRS