-@gerrit:opendev.org- Roman Kuznecov proposed: [zuul/zuul-jobs] 901072: tox: Do not concat stdout and stderr in getting siblings https://review.opendev.org/c/zuul/zuul-jobs/+/901072 | 06:32 | |
@cidlik:matrix.org | Hello. | 06:45 |
---|---|---|
I found, IMHO, the issue in tox job. In the determination of sibling packages names stderr concated with stdout. Due to this package names maybe incorrect, if some warnings were printed. Example: | ||
``` | ||
$ python setup.py --name <PACKAGE_NAME> | ||
... | ||
_DeprecatedInstaller: setuptools.installer and fetch_build_eggs are deprecated. | ||
!! | ||
******************************************************************************** | ||
Requirements should be satisfied by a PEP 517 installer. | ||
If you are using pip, you can try `pip install --use-pep517`. | ||
******************************************************************************** | ||
!! | ||
dist.fetch_build_eggs(dist.setup_requires) | ||
WARNING setuptools_scm.pyproject_reading toml section missing 'pyproject.toml does not contain a tool.setuptools_scm section' | ||
<PACKAGE_NAME> | ||
``` | ||
So tox cannot reinstall sibling packages. I posted change ([link](https://review.opendev.org/c/zuul/zuul-jobs/+/901072)) into zuul/zuul-jobs project. Could someone review this change? | ||
@cidlik:matrix.org | * Hello. | 06:45 |
I found, IMHO, the issue in tox job. In the determination of sibling packages names stderr concated with stdout. Due to this package names maybe incorrect, if some warnings were printed. Example: | ||
``` | ||
$ python setup.py --name <PACKAGE_NAME> | ||
... | ||
_DeprecatedInstaller: setuptools.installer and fetch_build_eggs are deprecated. | ||
!! | ||
******************************************************************************** | ||
Requirements should be satisfied by a PEP 517 installer. | ||
If you are using pip, you can try `pip install --use-pep517`. | ||
******************************************************************************** | ||
!! | ||
dist.fetch_build_eggs(dist.setup_requires) | ||
WARNING setuptools_scm.pyproject_reading toml section missing 'pyproject.toml does not contain a tool.setuptools_scm section' | ||
<PACKAGE_NAME> | ||
``` | ||
So tox cannot reinstall sibling packages. I posted change ([link](https://review.opendev.org/c/zuul/zuul-jobs/+/901072)) into zuul/zuul-jobs project. Can someone review this change? | ||
-@gerrit:opendev.org- Zuul merged on behalf of Matthieu Huin https://matrix.to/#/@mhuin:matrix.org: [zuul/zuul] 901673: zuul-web: ensure HTTPError is invoked with 4xx status code https://review.opendev.org/c/zuul/zuul/+/901673 | 08:13 | |
-@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 901829: Set remote head refs in workspace repo clone https://review.opendev.org/c/zuul/zuul/+/901829 | 13:04 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!