Code review comment for ~lgp171188/lazr.restful:add-launchpad-yaml

Revision history for this message
Jürgen Gmach (jugmac00) wrote :

I started debugging yesterday.

While I have not finished, I have some clues.

a) we need to add a `passenv = HOME` to the linter env in tox.ini.
That is a known issue, see this thread: https://chat.canonical.com/canonical/pl/qh4qniaehbfcbbuua1kdtyca4c

b) as Debian and thus Ubuntu splits Python into several packages, we need to also install `python3.x-distutils`

c) then the next error probably happens as the existing pyproject.yaml file implicitly triggers pip to do a PEP517 type build... I'd need to investigate what needs to be done to fix this, maybe updating some packages, or forcing pip to not do a PEP517 build ( python3 -m pip install --no-use-pep517 - that would require to update tox' install command)

« Back to merge proposal