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

wolfssl, use master again in CI #11766

Closed
wants to merge 3 commits into from
Closed

Conversation

icing
Copy link
Contributor

@icing icing commented Aug 30, 2023

  • with the shared session update fix landed in master, it is time to use that in our CI again

@github-actions github-actions bot added tests CI Continuous Integration labels Aug 30, 2023
@jay
Copy link
Member

jay commented Aug 30, 2023

rebased on master due to ci fix

@icing icing requested a review from bagder September 1, 2023 06:38
@@ -51,7 +51,7 @@ env:
ngtcp2-version: v0.18.0
nghttp2-version: v1.55.1
gnutls-version: 3.8.0
wolfssl-version: v5.6.3-stable
wolfssl-version: master
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this make the caching weird if we cache on master?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this make the caching weird if we cache on master?

Indeed, need to take that out. Thanks for noticing.

- with the shared session update fix landed in master, it
  is time to use that in our CI again
@bagder bagder closed this in 289cd40 Sep 1, 2023
ptitSeb pushed a commit to wasix-org/curl that referenced this pull request Sep 25, 2023
allow ever changing master for wolfssl

Closes curl#11766
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration tests
Development

Successfully merging this pull request may close these issues.

None yet

3 participants