-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
[wolfSSL] update version to v5.4.0 AND [wolfTPM] update version to v2.5.0 AND [wolfMQTT] update version to v1.14.0 #25936
Conversation
FYI: @JacobBarthelmeh and @embhorn |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!
PRs must add only one version and must not modify any published versions
When making any changes to a library, the version or port-version in vcpkg.json
or CONTROL
must be modified.
error: checked-in files for wolftpm have changed but the version was not updated
version: 2.5.0
old SHA: d7b76cdee52715ded9066374432428528d3b22e2
new SHA: b18c17a1a5166e5e6d6e3f54babfc480dc87b3f0
Did you remember to update the version or port version?
Use --overwrite-version to bypass this check
***No files were updated***
error: checked-in files for wolfmqtt have changed but the version was not updated
version: 1.13.0
old SHA: bc6b26e4254f8938dc9db7f7b5dc84958a45c877
new SHA: 6b333d32f75662bfc3d331a9148cf47c5bfccd38
Did you remember to update the version or port version?
Use --overwrite-version to bypass this check
***No files were updated***
error: checked-in files for wolfssl have changed but the version was not updated
version: 5.4.0
old SHA: 57aac1c50b45275c7a99eca32ad985998b292dc8
new SHA: 53de158d1ee1ef8f107ba8183b3ad25fc3dd90c2
Did you remember to update the version or port version?
Use --overwrite-version to bypass this check
***No files were updated***
@dgarske Please bump the port version. See documentation. |
afb0afc
to
7744a20
Compare
The failures are due to a wolfSSL CMake bug with v5.4.0 where our SP math all is enabled by default, yet the wolfssl/wolfcrypt/sp_int.h is not being installed. Patch here: wolfSSL/wolfssl#5401 |
f2e73eb
to
46262bb
Compare
Hi @FrankXie05 , having trouble reproducing the latest errors. I have only tried on Mac OS X and Ubuntu 20. Any tips?
|
@dgarske
Regarding this issue, I see that you have fixed it upstream, but the current commit doesn't merge your content, we need a patch until the next release. Log from
The problem is that the file
config-x86-windows-out (3).log |
ba58f85
to
34bea0d
Compare
34bea0d
to
d08c13b
Compare
@FrankXie05 this is good to go now. Thanks for your help. |
Looks great, thanks for the contribution! |
What does your PR fix?
Updated to stable wolfSSL v5.4.0
Updated to stable wolfTPM v2.5.0
Updated to stable wolfMQTT v1.14.0
Fixes #25858
Which triplets are supported/not supported? Have you updated the CI baseline?
all
Does your PR follow the maintainer guide?
yes
If you have added/updated a port: Have you run
./vcpkg x-add-version --all
and committed the result?yes