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

Backport 2.16: Check if feature macro is defined before define it #4502

Merged
merged 2 commits into from May 17, 2021

Conversation

d3zd3z
Copy link
Contributor

@d3zd3z d3zd3z commented May 12, 2021

Backport of #4470

@gilles-peskine-arm gilles-peskine-arm added component-platform Portability layer and build scripts enhancement help-wanted This issue is not being actively worked on, but PRs welcome. single-reviewer This PR qualifies for having only one reviewer needs-ci Needs to pass CI tests needs-work and removed help-wanted This issue is not being actively worked on, but PRs welcome. labels May 12, 2021
ceolin and others added 2 commits May 13, 2021 11:18
Zephyr's native posix port define _POSIX_C_SOURCE with a higher value
during the build, so when mbedTLS defines it with a different value
breaks the build.

As Zephyr is already defining a higher value is guaranteed that mbedTLS
required features will be available. So, just define it in case it was
not defined before.

[taken from Zephyr mbedtls module:
zephyrproject-rtos/mbedtls@76dcd6e]

Signed-off-by: Flavio Ceolin <flavio.ceolin@intel.com>
Signed-off-by: David Brown <david.brown@linaro.org>
Signed-off-by: David Brown <david.brown@linaro.org>
@gilles-peskine-arm gilles-peskine-arm added approved Design and code approved - may be waiting for CI or backports and removed needs-ci Needs to pass CI tests needs-work labels May 17, 2021
@gilles-peskine-arm gilles-peskine-arm merged commit 6c0f91e into Mbed-TLS:mbedtls-2.16 May 17, 2021
@d3zd3z d3zd3z deleted the bp216-posix-define branch May 17, 2021 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Design and code approved - may be waiting for CI or backports component-platform Portability layer and build scripts enhancement single-reviewer This PR qualifies for having only one reviewer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants