[Freeipa-devel] [freeipa PR#203][comment] Add sdist_list plugin to all setup.py

pspacek freeipa-github-notification at redhat.com
Tue Nov 1 13:13:04 UTC 2016


  URL: https://github.com/freeipa/freeipa/pull/203
Title: #203: Add sdist_list plugin to all setup.py

pspacek commented:
"""
Thanks for the propsal. It has several problems:
- It prints extra text at the beginning of output. This makes the output unsuitable for automated processing.
~~~
$ python setup.py sdist_list --quiet
running sdist_list
running egg_info
writing freeipa.egg-info/PKG-INFO
writing top-level names to freeipa.egg-info/top_level.txt
writing dependency_links to freeipa.egg-info/dependency_links.txt
reading manifest file 'freeipa.egg-info/SOURCES.txt'
writing manifest file 'freeipa.egg-info/SOURCES.txt'
warning: sdist_list: standard file not found: should have one of README, README.rst, README.txt

running check
/home/pspacek/pkg/ipa/git/ipaserver/__init__.py
~~~
- pylint is failing

I wonder if the printed list is always the same as *.egg-info/SOURCES.txt or not. If it is the same we may very well re-use that file. It seems that egg-info is generated during `sdist_list` processing anyway ...
"""

See the full comment at https://github.com/freeipa/freeipa/pull/203#issuecomment-257562968


More information about the Freeipa-devel mailing list