Comment 23 for bug 1880749

Revision history for this message
Rick Elrod (relrod) wrote :

> What is the impact of this? It seems to happily install PEP 517 packages as-is.

@Stefano sorry for the delay here. Installing PEP517 packages is fine without the patch, but building them (building a wheel) is where the issue comes in.

If you need a concrete example, download the .tar.gz of any project that is normally capable of producing wheels, such as 'sampleproject' from pypi (https://pypi.org/project/sampleproject/#files).
Activate a fresh virtualenv, untar it the .tar.gz, cd into the project, and run:

  python -m pep517.build --binary --out-dir dist .

Before the patch mentioned above, it will fail with: ModuleNotFoundError: No module named 'toml'

The .deb's from the SRU above get us a step closer but seem to still have an issue/traceback (see attachment which includes the commands to reproduce). It looks to be due to the vendored pytoml being listed in `lib/python3.8/site-packages/pip/_vendor/__init__.py` in the virtualenv.

Whereas doing similar steps in a ubuntu:20.10 container (and installing pep517 explicitly in the virtualenv) produces a successful result.