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

travis: install libgsasl-dev to add that to the builds #6588

Closed
wants to merge 1 commit into from

Conversation

bagder
Copy link
Member

@bagder bagder commented Feb 9, 2021

No description provided.

@bagder bagder added the CI Continuous Integration label Feb 9, 2021
@bagder
Copy link
Member Author

bagder commented Feb 10, 2021

The travis builds here now reproduce the problems #6587 fixes. I'll merge this after @vszakats has that nailed.

vszakats added a commit that referenced this pull request Feb 10, 2021
- also fix an indentation
- make Curl_auth_gsasl_token() use CURLcode (by Daniel Stenberg)

Ref: #6372 (comment)
Ref: #6588

Reviewed-by: Jay Satiro
Assisted-by: Daniel Stenberg
Reviewed-by: Simon Josefsson
Closes #6587
@vszakats
Copy link
Member

vszakats commented Feb 10, 2021

I've committed both PRs, so, ready to go.

@bagder
Copy link
Member Author

bagder commented Feb 10, 2021

I rebased and force-pushed this to verify that it looks fine before I merge.

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

Successfully merging this pull request may close these issues.

None yet

2 participants