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

[ixwebsocket] update to 7.6.3 #9397

Merged
merged 5 commits into from
Jan 10, 2020

Conversation

bsergean
Copy link
Contributor

Update to the latest version, which improve SSL support on Windows.

(cf machinezone/IXWebSocket#135)

@ras0219-msft
Copy link
Contributor

/azp run

ports/ixwebsocket/portfile.cmake Show resolved Hide resolved
ports/ixwebsocket/portfile.cmake Show resolved Hide resolved
@NancyLi1013
Copy link
Contributor

NancyLi1013 commented Dec 24, 2019

Hi @bsergean thanks for this PR.
Please update the changes.
Note: all features tested pass with the following triplets:

  • x86-windows
  • x64-windows
  • x64-linux

@NancyLi1013
Copy link
Contributor

/azp run

@bsergean
Copy link
Contributor Author

bsergean commented Jan 7, 2020

I updated the version and added the homepage.

@NancyLi1013
Copy link
Contributor

Thanks for your instant update.
It seems that the version: 7.6.3 in CONTROL file is not different from that v7.9.2 in portfile.cmake.
Could you check it again?

@bsergean
Copy link
Contributor Author

bsergean commented Jan 7, 2020 via email

@NancyLi1013
Copy link
Contributor

/azp run

@NancyLi1013 NancyLi1013 added the info:reviewed Pull Request changes follow basic guidelines label Jan 10, 2020
@dan-shaw dan-shaw merged commit e8c662d into microsoft:master Jan 10, 2020
@bsergean
Copy link
Contributor Author

Thank you !

Someone recently added the ability to use OpenSSL on Windows for TLS, I need to learn a bit more about the way things can be configured dynamically ; but it took me such a long time to get a portfile that work that I'm very afraid to change it now :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants