Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Zephyr: Fix CMake error with CONFIG_MCUMGR_CMD_LOG_MGMT=y #87

Closed

Conversation

NZSmartie
Copy link

@NZSmartie NZSmartie commented Jun 13, 2020

Fixes a CMake error due to missing parentheses on an endif and corrects source file paths.

See #86 for more details

Fixes #86

Part of apache#86, enabling
this option highlights the failed CMake configuration due to lack of
log_mgmnt command from being compiled.

Signed-off-by: Roman Vaughan <nzsmartie@gmail.com>
Fixes a CMake error due to missing parentheses on an `endif` and
corrects source file paths.

Fixes apache#86 (apache#86)

Signed-off-by: Roman Vaughan <nzsmartie@gmail.com>
@de-nordic
Copy link
Contributor

@NZSmartie do you have Zephyr west.yaml update referencing this change?
(example here: zephyrproject-rtos/zephyr#25711), so that it could be checked with zephyr CI?

@NZSmartie
Copy link
Author

I'm closing this as the Zephyr port of log management command is completely broken and needs a re-write. I've attempted progress on this, but will require more effort

@NZSmartie NZSmartie closed this Jul 20, 2020
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.

Zephyr: CMake error when Log Management is enabled.
2 participants