Bug 263299 - REGRESSION(268723@main): Fix auto-installed versions of twisted and PyOpenSSL for the bots (try 2)
Summary: REGRESSION(268723@main): Fix auto-installed versions of twisted and PyOpenSSL...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Carlos Alberto Lopez Perez
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2023-10-17 19:47 PDT by Carlos Alberto Lopez Perez
Modified: 2023-12-12 16:16 PST (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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.