Skip to content

Fix cmake warnings of zephyr builds. - #165

Merged
boaks merged 1 commit into
eclipse-tinydtls:mainfrom
boaks:fix_cmake_warnings_with_zephyr
Sep 28, 2022
Merged

Fix cmake warnings of zephyr builds.#165
boaks merged 1 commit into
eclipse-tinydtls:mainfrom
boaks:fix_cmake_warnings_with_zephyr

Conversation

@boaks

@boaks boaks commented Sep 20, 2022

Copy link
Copy Markdown
Contributor

Exclude cmake options for zephyr builds, if these are set by zephyr.

Signed-off-by: Achim Kraus achim.kraus@cloudcoap.net

@boaks
boaks force-pushed the fix_cmake_warnings_with_zephyr branch from 87d4281 to f00da21 Compare September 27, 2022 13:09
@boaks

boaks commented Sep 27, 2022

Copy link
Copy Markdown
Contributor Author

@obgm

If you could have a short look.
it affects only the zephyr build, replacing the cmake options by variables set from kconfig.
The fixes some cmake warnings.

@boaks
boaks force-pushed the fix_cmake_warnings_with_zephyr branch from f00da21 to e14529a Compare September 27, 2022 13:57
@boaks

boaks commented Sep 27, 2022

Copy link
Copy Markdown
Contributor Author

I turns out, that the file(CONFIGURE ...) requires a newer cmake version.
This PR therefore only applies this, if cmake has the required version.

Exclude cmake options for zephyr builds, if these are set by zephyr.
Exclude file(CONFIGURE ..), if cmake is older than 3.18.

Signed-off-by: Achim Kraus <achim.kraus@cloudcoap.net>
@boaks
boaks force-pushed the fix_cmake_warnings_with_zephyr branch from e14529a to c64069d Compare September 28, 2022 11:59
@boaks
boaks merged commit f903906 into eclipse-tinydtls:main Sep 28, 2022
@boaks
boaks deleted the fix_cmake_warnings_with_zephyr branch January 21, 2023 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant