Skip to content

[libtorrent] Update to latest 1.2.x commit #17166

Merged
strega-nil-ms merged 2 commits intomicrosoft:masterfrom
FranciscoPombal:update_libtorrent
Apr 30, 2021
Merged

[libtorrent] Update to latest 1.2.x commit #17166
strega-nil-ms merged 2 commits intomicrosoft:masterfrom
FranciscoPombal:update_libtorrent

Conversation

@FranciscoPombal
Copy link
Copy Markdown
Contributor

  • Update to 1.2.13 plus some additional fixes (allows dropping existing
    patches)
  • Build with libiconv by default on all platforms
  • Fix deprecated usage of vcpkg_check_features without the FEATURES
    keyword
  • Migrate to vcpkg.json manifest files

[libtorrent] Update versioning information


Previously: #15620 and #14755.

1.2.14 may take a while to come, and this is a good opportunity to get rid of all patches and migrate to manifest files. Hopefully this will make future work of getting the python feature to work on Windows easier.

Tested onx64-linux, x64-windows, and x64-windows-static.

@JonLiu1993 JonLiu1993 self-assigned this Apr 9, 2021
@JonLiu1993 JonLiu1993 added the category:port-update The issue is with a library, which is requesting update new revision label Apr 9, 2021
- Update to 1.2.13 plus some additional fixes (allows dropping existing
patches)
- Build with libiconv by default on all platforms
- Fix deprecated usage of `vcpkg_check_features` without the `FEATURES`
keyword
- Migrate to vcpkg.json manifest files
@JonLiu1993
Copy link
Copy Markdown
Contributor

@FranciscoPombal ,Thanks for your contribute

@FranciscoPombal
Copy link
Copy Markdown
Contributor Author

FranciscoPombal commented Apr 9, 2021

@JonLiu1993 Looks like we have to use "version-string":

$ (vcpkg.json): 'version' text was not a relaxed version:
Error: String `1.2.13-b799640` must only contain dot-separated numeric values without leading zeroes.

So I've reverted the fixes made in response to your review comments.

@JonLiu1993 JonLiu1993 added info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. and removed requires:author-response labels Apr 12, 2021
@FranciscoPombal
Copy link
Copy Markdown
Contributor Author

@JonLiu1993 Anything preventing this from being merged?

@JonLiu1993
Copy link
Copy Markdown
Contributor

@FranciscoPombal ,Don’t worry, soon this pr will be merged with other prs.

@strega-nil-ms
Copy link
Copy Markdown
Contributor

Thanks @FranciscoPombal :)

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:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants