Bug 263299

Summary:	REGRESSION(268723@main): Fix auto-installed versions of twisted and PyOpenSSL for the bots (try 2)
Product:	WebKit	Reporter:	Carlos Alberto Lopez Perez <clopez>
Component:	Tools / Tests	Assignee:	Carlos Alberto Lopez Perez <clopez>
Status:	RESOLVED FIXED
Severity:	Normal	CC:	gsnedders, jbedard, webkit-bug-importer
Priority:	P2	Keywords:	InRadar
Version:	WebKit Nightly Build
Hardware:	Unspecified
OS:	Unspecified
See Also:	https://bugs.webkit.org/show_bug.cgi?id=261082 https://bugs.webkit.org/show_bug.cgi?id=262369 https://bugs.webkit.org/show_bug.cgi?id=262512

Description Carlos Alberto Lopez Perez 2023-10-17 19:47:15 PDT

At 268723@main the versions added for twisted and pyOpenSSL caused several issues:


- twisted version
  I added added a version of twisted that can't be installed because it requires PEP 517 support (Bug 261082).
  Not sure how I ended validating this. I guess I was playing with different versions and at one point I got it building correctly then tried the newer one and I overlooked it was reusing maybe a previous cached version.. no idea.
  But it is clear this version  23.x can't be used until Bug 261082 gets fixed...

@Sam pointed that here https://bugs.webkit.org/show_bug.cgi?id=261082#c2 Thanks!


- PyOpenSSL version
  The version added requires cryptography>=38.0.0,<42,!=40.0.0,!=40.0.1 but we were installing 36.0.0.
  ups! sorry! I didn't noticed this because on ARM32 bots we are skipping to install this cryptography module since 265835@main and it was picking the system one (python3-cryptography 39.9)



So, let's see if this time I can get this right 🤞

Comment 1 Carlos Alberto Lopez Perez 2023-10-17 20:02:56 PDT

See also 268757@main fixing issues caused by 268723@main

Comment 2 Carlos Alberto Lopez Perez 2023-10-17 20:04:29 PDT

Pull request: https://github.com/WebKit/WebKit/pull/19208

Comment 3 Radar WebKit Bug Importer 2023-10-24 19:48:14 PDT

<rdar://problem/117449370>

Comment 4 EWS 2023-12-12 16:16:02 PST

Committed 271958@main (dcdaa7641d3b): <https://commits.webkit.org/271958@main>

Reviewed commits have been landed. Closing PR #19208 and removing active labels.