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

Verify and fix different options: disable MBEDTLS_SSL_USE_MPS, enable MBEDTLS_PROTO_* , disable TLS1_3 and nbio=0 #298

Open
yuhaoth opened this issue Jul 20, 2021 · 0 comments
Labels

Comments

@yuhaoth
Copy link
Collaborator

yuhaoth commented Jul 20, 2021

Description

This is part of #15.
To introduce TLS1_2 only case.

#undef MBEDTLS_SSL_USE_MPS
#define MBEDTLS_SSL_PROTO_TLS1_2
#define MBEDTLS_TMP_BLOCK_IO
#undef MBEDTLS_SSL_PROTO_TLS1_3_EXPERIMENTAL
yuhaoth added a commit to yuhaoth/mbedtls1.3 that referenced this issue Jul 21, 2021
According to hannestschofenig#15, different options report
fail. To fix the issue we should not break
passed options. The script is to make sure
current status.

If all relative issues are resolved, This
patch should be removed or re-considered.

issues: hannestschofenig#15, hannestschofenig#297,hannestschofenig#238,hannestschofenig#298

Change-Id: Iaebbdaa5861802f2a48e6bca238a94672ddfaf70
CustomizedGitHooks: yes
Signed-off-by: Jerry Yu <[email protected]>
yuhaoth added a commit to yuhaoth/mbedtls1.3 that referenced this issue Jul 21, 2021
According to hannestschofenig#15, different options report
fail. To fix the issue we should not break
passed options. The script is to make sure
current status.

If all relative issues are resolved, This
patch should be removed or re-considered.

issues: hannestschofenig#15, hannestschofenig#297,hannestschofenig#238,hannestschofenig#298,hannestschofenig#301

Change-Id: Iaebbdaa5861802f2a48e6bca238a94672ddfaf70
CustomizedGitHooks: yes
Signed-off-by: Jerry Yu <[email protected]>
@yuhaoth yuhaoth added the bug label Jul 21, 2021
yuhaoth added a commit to yuhaoth/mbedtls1.3 that referenced this issue Jul 22, 2021
According to hannestschofenig#15, different options report
fail. To fix the issue we should not break
passed options. The script is to make sure
current status.

If all relative issues are resolved, This
patch should be removed or re-considered.

issues: hannestschofenig#15, hannestschofenig#297,hannestschofenig#238,hannestschofenig#298,hannestschofenig#301

Change-Id: Iaebbdaa5861802f2a48e6bca238a94672ddfaf70
CustomizedGitHooks: yes
Signed-off-by: Jerry Yu <[email protected]>
yuhaoth added a commit to yuhaoth/mbedtls1.3 that referenced this issue Jul 23, 2021
According to hannestschofenig#15, different options report
fail. To fix the issue we should not break
passed options. The script is to make sure
current status.

If all relative issues are resolved, This
patch should be removed or re-considered.

issues: hannestschofenig#15, hannestschofenig#297,hannestschofenig#238,hannestschofenig#298,hannestschofenig#301

Change-Id: Iaebbdaa5861802f2a48e6bca238a94672ddfaf70
CustomizedGitHooks: yes
Signed-off-by: Jerry Yu <[email protected]>
yuhaoth pushed a commit to yuhaoth/mbedtls1.3 that referenced this issue Aug 1, 2021
This change enables automatic detection and consumption of Mbed TLS
library targets from within other CMake projects. By generating an
`MbedTLSConfig.cmake` file, consuming projects receive a more complete
view of these targets, allowing them to be used as dependencies which
properly inherit the transitive dependencies of the libraries.

This is fairly fragile, as it seems Mbed TLS's libraries do not appear
to properly model their dependencies on other targets, including
third-party dependencies. It is, however, sufficient for building and
linking the compiled Mbed TLS libraries when there are no third-party
dependencies involved. Further work is needed for more complex
use-cases, but this will likely meet the needs of most projects.

Resolves hannestschofenig#298. Probably useful for Mbed-TLS#2857.

Signed-off-by: Chris Kay <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant