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 |
||
Carlos Alberto Lopez Perez
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 🤞
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Carlos Alberto Lopez Perez
See also 268757@main fixing issues caused by 268723@main
Carlos Alberto Lopez Perez
Pull request: https://github.com/WebKit/WebKit/pull/19208
Radar WebKit Bug Importer
<rdar://problem/117449370>
EWS
Committed 271958@main (dcdaa7641d3b): <https://commits.webkit.org/271958@main>
Reviewed commits have been landed. Closing PR #19208 and removing active labels.