Skip to content

[ libtorrent ] Update to 2.0#14293

Closed
JonLiu1993 wants to merge 6 commits intomicrosoft:masterfrom
JonLiu1993:dev/libtorrent/13702
Closed

[ libtorrent ] Update to 2.0#14293
JonLiu1993 wants to merge 6 commits intomicrosoft:masterfrom
JonLiu1993:dev/libtorrent/13702

Conversation

@JonLiu1993
Copy link
Copy Markdown
Contributor

Fix #13702

Update libtorrent to 2.0

All featureas are tested successfully in the following triplets

  • x86-windows
  • x64-windows
  • x64-windows-static

@JonLiu1993 JonLiu1993 added info:internal category:port-update The issue is with a library, which is requesting update new revision labels Oct 29, 2020
@FranciscoPombal
Copy link
Copy Markdown
Contributor

@JonLiu1993
Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@JonLiu1993
Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Copy Markdown
Contributor

@Convery Convery left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The removal of ports/lua/portfile.cmake is accidental.
Otherwise working as expected on x86 but fails on x64.

Updating to the latest commit fixes it though:

vcpkg_from_github(
    OUT_SOURCE_PATH SOURCE_PATH
    REPO arvidn/libtorrent
    REF master
    SHA512 8205eca746a365a22702d0c3a02aa02438a8558e9ff1f5aa63788cb10233935dda2a3900d21abcdb13111f6d01337752dc0831000e5ded0eb4884f4fa4c736c8
    HEAD_REF RC_2.0
)

@FranciscoPombal
Copy link
Copy Markdown
Contributor

2.0.1 was just released (https://github.com/arvidn/libtorrent/releases/tag/v2.0.1), as well as 1.2.11 (https://github.com/arvidn/libtorrent/releases/tag/v1.2.11). Wouldn't it be better to first update to 1.2.11, and only after that to 2.x?

@Convery
Copy link
Copy Markdown
Contributor

Convery commented Nov 17, 2020

Can always do both. 1.2.x wasn't updated since Sep so I just assumed that 2.0 was the current branch. Can always have two ports, one for libtorrent 1.x and one for 2.x, rather than a single libtorrent.

@JonLiu1993
Copy link
Copy Markdown
Contributor Author

I will upgrade soon, thank you everyone

@glassez
Copy link
Copy Markdown

glassez commented Nov 22, 2020

libtorrent-2.0 shouldn't be considered as update for this port. You should add another port for it (aka libtorrent2) and keep this one for 1.2.x.

@FranciscoPombal
Copy link
Copy Markdown
Contributor

FranciscoPombal commented Nov 23, 2020

libtorrent-2.0 shouldn't be considered as update for this port. You should add another port for it (aka libtorrent2) and keep this one for 1.2.x.

I second this. There are still many projects that benefit from the continued support of the 1.x.x branches, and it is convenient to have them available without having to roll back vcpkg commits.

EDIT: or maybe the new version could be exposed via a feature (much the same way qt5[latest] works)? Anyway, I guess all of this will be easier to manage when proper versioning support lands, but for now, creating a new port seems to be the path of least resistance.

@JonLiu1993
Copy link
Copy Markdown
Contributor Author

Duplicate to #14755

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[libtorrent] update to 2.0

5 participants