Repository navigation
Conversation
|
Seems good to me. If anything we're more familiar with mbedTLS and it works well for Godot, so it's a better pick IMO. As discussed with @Calinou, we'll first make a release with the new OpenSSL PRs merged (and Godot 4.2 baseline), and then merge this PR and make another release. So there will be a recent release with OpenSSL and Godot 4.2 to compare with in case there are any new issues related to mbedTLS. |
b0cab0c to
c9eb393
Compare
da656d6 to
a8cb0c0
Compare
|
@Faless Can you look into rebasing this PR? The SSL certificate issue is still being run into regularly with 3.2.1, so I'd like to merge this, tag 3.3.0 and hopefully fix the issue once and for all. |
c8d21c3 to
ad534a4
Compare
|
@Faless Out of curiosity, is this ready for review now? |
87d239e to
026a5ef
Compare
543c604 to
5c04ea5
Compare
I'm not really sure how I can debug this case, could you maybe try a few more times and see if you actually get this error again? Because it seems to work for me. |

Improve CI, add x86_32, arm32, arm64 and android builds
Original Description!
Build dependencies using cmake.I was nerd-sniped by the merge of #191 and #199 to try and switch to mbedTLS as the crypto library.
This should make it much easier to get ARM32/ARM64 (#260) builds on both windows and linux (I've confirmed cross-builds for linux arm32/arm64, for windows, see godotengine/webrtc-native#174 ).
I can add the CI bits as a separate commit to this PR if desired.