Skip to content

Win32: check for bcrypt.lib#6

Closed
Coeur wants to merge 3 commits intotransmission:masterfrom
Coeur:coeur/BCryptGenRandom
Closed

Win32: check for bcrypt.lib#6
Coeur wants to merge 3 commits intotransmission:masterfrom
Coeur:coeur/BCryptGenRandom

Conversation

@Coeur
Copy link
Copy Markdown

@Coeur Coeur commented Nov 9, 2025

First of all, Transmission's fork of libevent needs a post-2.2.0-transmission branch:
https://github.com/transmission/libevent/branches/all

Once created, we can retarget this PR to the new 2.2 branch.

Then, regarding the scope of this PR, it's the same as libevent#1813 and is needed for Transmission to migrate from libevent 2.1 to libevent 2.2 (transmission/transmission#7765).

@Neustradamus
Copy link
Copy Markdown

@Coeur: Thanks for your PR!

@Coeur Coeur force-pushed the coeur/BCryptGenRandom branch from 6fa8ccc to 97a86bf Compare November 19, 2025 10:25
@ckerr
Copy link
Copy Markdown
Member

ckerr commented Feb 12, 2026

@Coeur IDK if/when we're going to create a post-2.2.0-transmission branch here, for all the reasons we've discussed in the past:

  • Upstream hasn't released in 6 years
  • There are 6 years of untested commits built up since 2.1.12, so we shouldn't rush into that
  • We're trying to reduce/remove our libevent dependency; so the less work we put into libevent triage/upkeep, the better

So I'm going to decline this PR as-is with it targeting master.

What I would be OK with is cherry-picking a specific bugfix into post-2.1.12-transmission, similar to what was done with #4 and #7.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants