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

chore(zephyr): global CSTD property is deprecated #78

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fouge
Copy link

@fouge fouge commented Nov 22, 2024

select required C standard and GNU extensions for compact logs directly in KConfig.
Use global CSTD if config REQUIRES_STD_C11 & CONFIG_GNU_C_EXTENSIONS
are not selected, for Zephyr versions older than 3.7.0.

This removes the warning during compilation while still ensuring that C11 + GNU extensions
are still enabled.

Please make sure it's not a breaking change. 🙏

select required C standard and GNU extensions for compact logs directly
in KConfig.
Use global CSTD if config `REQUIRES_STD_C11` & `CONFIG_GNU_C_EXTENSIONS`
 are not selected, for Zephyr versions older than 3.7.0.

Signed-off-by: Cyril Fougeray <[email protected]>
@noahp
Copy link
Contributor

noahp commented Nov 26, 2024

Thank you @fouge for the fix! we'll include it in our next SDK release (scheduled for next week). Let me know if you have an urgent need for an official release and we will prioritize it!

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.

2 participants