-
Notifications
You must be signed in to change notification settings - Fork 3.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Get all infra working on Python 3 #8820
Comments
See gh-10128 for wptrunner |
I see that some of these sdists do not have tests included. It will be much easier for packagers to do tests on Python 3 if the sdist for each has the tests included. |
The PyPI packages are in general not actively maintained (although speserve is sometimes updated) and aren't actually used by any wpt infrastructure or by browser vendors who import this stuff (afaik). |
I ran into them because http://pypi.org/project/mozfile is using |
You can tick https://build.opensuse.org/package/show/home:jayvdb:wpt/python-wptmanifest
I note that http://pypi.org/project/manifest/ was pre-alpha, and now redundant, and the author would probably be willing to release the name to wpt project.
|
I spotted the Moving to Python 3 thread on webkit-dev. Key quote:
There was discussion on the thread about running on older versions of macOS that don't have Python 3, and if WPT's Python code is used in any way in WebKit that concern can translate. But in any case, sounds like we could soon start thinking about using Python 3 on macOS as well. Even today we could install Python 3 on Azure Pipelines of course, but the hard question is whether we keep it working with Python 2. @youennf @rniwa is there any impact of WPT's Python code on WebKit? Would it be good for WebKit is WPT supports Python 3, or does it not matter? |
As things stand, we won't be able to run imported tests if WPT tests relied upon Python3 since older versions of macOS on which we still run regression (layout) tests do not support Python3 by default. |
Does that include only wptserve and the "CGI" .py files that are written as part of tests, or do you also rely on other parts of WPT to work using Python 2? |
Alas, it… doesn't actually work. 😳 There's a huge diff between manifests generated by Python 2 and Python 3. |
Interesting! It sounds like there should be some regression tests which compares the generated results to expected results generated from python2. |
@jayvdb Yeah, we should, just never got around to writing one because it's so far from passing (and it never has). Can't regress something that never worked. :) |
CC @ericlaw1979; help is welcome. |
|
This is done, we are now Python 3 only! |
#28776 tracks some additional cleanup. |
We probably want to split these out into individual issues, but given I don't think we're doing this any time soon I'll leave that for when someone starts on this:
Note that even after updating all of this we'll still rely on Python 2 as we have hundreds of helper files for tests.
The text was updated successfully, but these errors were encountered: