Skip to content

wolfssl, use master again in CI#11766

Closed
icing wants to merge 3 commits intocurl:masterfrom
icing:ci-wolfssl-master-again
Closed

wolfssl, use master again in CI#11766
icing wants to merge 3 commits intocurl:masterfrom
icing:ci-wolfssl-master-again

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 jay force-pushed the ci-wolfssl-master-again branch from b2aa2c0 to 74b5c1a Compare August 30, 2023 16:14
@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
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
@icing icing force-pushed the ci-wolfssl-master-again branch from 74b5c1a to 1c97814 Compare September 1, 2023 09:44
@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.

3 participants