Conversation
|
Hi @rth, this is great news, thanks for the initial work on this! I hope you'll get the emscripten update fightened during the sprint. I'll take a look at the minor problems in a few days. What do you think about making a separate branch for python38 to merge further pull requests, especially for package contributions? We then might first concentrate on Pyodides core to run with Python 3.8 and then partially extend to the packages. |
I think this PR is not that far from being mergable. There are just 8 failing test modules in the CPython test suite (though I also still need to update the list of CPython tests). The rest of the packages appear to build fine (aside from cytoolz which I updated). Generally packages that worked with Python 3.7 will likely also work with 3.8 aside for some warnings. We can update packages and likely emscripten warning in a follow up PR. Don't hesitate to either push fixes directly to this branch (you should have access being a maintainer) or open PRs to this branch on my fork. I'll also try to look into failing tests this week. |
|
I think CPython tests should now pass. I have skipped a few using the async module or subprocess, and patched others. The only blocker IMO is the which seems to indicate that something in hiwire is not working as it should and might need updating for Python 3.8 Edit: probably need to read through https://docs.python.org/3.8/whatsnew/3.8.html#changes-in-the-c-api in mode detail. Edit2: actually it was an issue with the test server. |
| make -C zlib clean | ||
| echo "The Emsdk, CPython and CLAPACK are not cleaned. cd into those directories to do so." | ||
|
|
||
| clean-all: clean |
|
@phorward This should now be ready for a review. The one lxml failure on chrome which happens from time to time is a known unrelated issue (#544 (comment)) everything else passes. If you merge, don't forget to use the "Squash and Merge" UI button :) |

An attempt to update to Python 3.8.2
Supersedes and closes #594
Partially addresses #635
Currently the build of CPython works and a number of tests pass. Required patches can be found as commits in https://github.com/rth/cpython/commits/pyodide-py38
TODO
emcccall,some of the packages included in pyodide might also need updating to be compatible with python 3.8Can be done in follow up PRs.circleci/python:3.8.2-busterimage include chromium 83, while tests fail for some reason with that version (see MNT Update to firefox 70.0 / geckodriver 0.26 in CI #622). So for the time being we might need to force installation of chromium 80 (which is currently used in CI).cc @phorward