Update readme + add manifest for ansible files #109
Conversation
setup.cfg
Outdated
| @@ -4,6 +4,7 @@ version = 1.0.0 | |||
| description = Oslo messaging benchmarking orchestrator | |||
| keywords = benchmark, oslo messaging, RPC | |||
| license = GPL-3.0 | |||
| license_file = LICENSE | |||
There was a problem hiding this comment.
I pushed to testpypi and tested and it fails with :
$) pip install ombt-orchestrator -i https://testpypi.python.org/pypi --no-deps
Copying ombt_orchestrator.egg-info to build/bdist.macosx-10.9-x86_64/wheel/ombt_orchestrator-1.0.0-py2.7.egg-info
running install_scripts
error: [Errno 2] No such file or directory: 'LICENSE'
| # maybe there is a better way to do it: | ||
| # https://stackoverflow.com/questions/1612733 | ||
|
|
||
| recursive-include orchestrator/ansible * |
There was a problem hiding this comment.
running python setup.py bdist_wheel looks good on my side now. I've renamed the licence file to fit the spec.
There was a problem hiding this comment.
include LICENSE.txt is necessary for the dist package to work.
Should we publish only wheels ?
There was a problem hiding this comment.
wheels is based on the new spec but if for any reason a system only allows users to use former formats we can also include the licence in the manifest.
| @@ -44,10 +44,8 @@ cd ombt-orchestrator | |||
| * Install the dependencies | |||
There was a problem hiding this comment.
It seems that the README isn't rendered correctly on pypi, that's probably because it's written in md instead of rst.
Should we move to rst README format ?
There was a problem hiding this comment.
I added in f856066 the proper type to the configuration, see: https://packaging.python.org/tutorials/distributing-packages/#description
| @@ -9,6 +9,7 @@ url = https://github.com/msimonin/ombt-orchestrator/ | |||
| author = Matthieu Simonin | |||
| author_email = matthieu.simonin@inria.fr | |||
| long_description = file: README.md | |||
| long_description_content_type = text/markdown | |||
There was a problem hiding this comment.
the PKG-INFO in the tgz file set the value properly: Description-Content-Type: text/markdown
There was a problem hiding this comment.
twine>=1.11.0 and setuptools>=38.6.0 are required for Markdown support according to this: pypi/warehouse#869
|
Great ! |
No description provided.