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

[libtorrent] Update to 1.2.11 #14755

Merged
merged 1 commit into from Dec 3, 2020

Conversation

FranciscoPombal
Copy link
Contributor

  • Remove python bindings patch, upstream now has the relevant fixes
  • Remove patch related to finding boost libs, not needed anymore
  • The [python] and [test] features no longer depend on [deprfun]

Previously: #13341 and #13375

The [python] feature is still a no-go on Windows. All else works fine on my end.

Updating to 2.0 will take a while, there is ongoing discussion about that in #14293. In the meantime, this one can be merged to provide the latest fixes.

@NancyLi1013 NancyLi1013 self-assigned this Nov 25, 2020
ports/libtorrent/CONTROL Outdated Show resolved Hide resolved
@NancyLi1013 NancyLi1013 added category:port-update The issue is with a library, which is requesting update new revision requires:author-response labels Nov 25, 2020
@NancyLi1013 NancyLi1013 added the requires:all-feature-testing vcpkg install port[all features supported by that port] needs to be demonstrated to function label Nov 27, 2020
- Remove python bindings patch, upstream now has the relevant fixes
- Remove patch related to finding boost libs, not needed anymore
- The [python] and [test] features no longer depend on [deprfun]
@FranciscoPombal
Copy link
Contributor Author

@NancyLi1013 Sorry for the delay. All comments addressed.

@NancyLi1013
Copy link
Contributor

Thanks for your update. Have you tested these features? If not, I will test them on my local.

@FranciscoPombal
Copy link
Contributor Author

@NancyLi1013

Thanks for your update. Have you tested these features? If not, I will test them on my local.

Yes, on both Windows and Linux (Ubuntu 18.04). But feel free to reproduce on your local setup.

@NancyLi1013 NancyLi1013 removed requires:all-feature-testing vcpkg install port[all features supported by that port] needs to be demonstrated to function requires:author-response labels Dec 2, 2020
@NancyLi1013
Copy link
Contributor

LGTM now, thanks for your PR @FranciscoPombal .

@NancyLi1013 NancyLi1013 added the info:reviewed Pull Request changes follow basic guidelines label Dec 3, 2020
@glassez
Copy link

glassez commented Dec 3, 2020

Excuse my curiosity, is there something else that prevents this PR from being merged?

@BillyONeal BillyONeal merged commit 133051b into microsoft:master Dec 3, 2020
@BillyONeal
Copy link
Member

Thanks for your help!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-update The issue is with a library, which is requesting update new revision info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants