263119 2023-10-13 05:44:28 -0700 [AutoInstall] prefer wheels whenever possible 2023-11-01 09:25:55 -0700 1 1 1 Unclassified WebKit Tools / Tests WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=263392 InRadar P2 Normal --- 1 gsnedders jbedard bfan2 jbedard webkit-bug-importer oldest_to_newest 1984859 0 gsnedders 2023-10-13 05:44:28 -0700 There's very little reason _not_ to prefer wheels in general; we might not always be able to use them (e.g., we may be on a platform where wheels are rarely provided, or even where packaging doesn't currently have any concept of the platform such as iOS), but we should use them when possible. AIUI, we rely on some things (in tests?) which aren't in the Buildbot wheel (c.f. https://github.com/buildbot/buildbot/commit/2d351c2270a22302149eecd11265ea2ce3ec6617, https://github.com/buildbot/buildbot/commit/d34e1ccd3e62748b6d0c83965486774cdfb1dacc, https://github.com/buildbot/buildbot/commit/7d5c1b1ccb7ff2fc01b44b064851cea8fc7af74c), but maybe the 2021 change (shipped in 3.5.0) gives us enough of what we care about in the wheel? It sounds like prior to that they didn't really consider the testing API to be public, which meant there was little-to-no way to test Buildbot configurations. 1984860 1 webkit-bug-importer 2023-10-13 05:44:37 -0700 <rdar://problem/116915892> 1984861 2 gsnedders 2023-10-13 05:45:10 -0700 (I have a change locally for this, but need to step out for a bit; will upload PR once I'm back.) 1984869 3 gsnedders 2023-10-13 06:51:35 -0700 Pull request: https://github.com/WebKit/WebKit/pull/19045 1987758 4 ews-feeder 2023-10-26 09:42:23 -0700 Committed 269817@main (dba0adf16439): <https://commits.webkit.org/269817@main> Reviewed commits have been landed. Closing PR #19045 and removing active labels. 1989138 5 jbedard 2023-11-01 07:59:15 -0700 Re-opening for pull request https://github.com/WebKit/WebKit/pull/19835 1989163 6 ews-feeder 2023-11-01 09:25:50 -0700 Committed 270059@main (219eb0bb7b43): <https://commits.webkit.org/270059@main> Reviewed commits have been landed. Closing PR #19835 and removing active labels.