Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix installation from PyPI sources #1

Merged
merged 2 commits into from
Jul 4, 2018
Merged

Fix installation from PyPI sources #1

merged 2 commits into from
Jul 4, 2018

Conversation

althonos
Copy link
Contributor

@althonos althonos commented Jul 4, 2018

Trying to install DnaCauldron from PyPI currently fails:

$ pip install --user -U dnacauldron
Collecting dnacauldron
  Using cached https://files.pythonhosted.org/packages/cd/b1/85b1f3a9d42a63daa3efc20dc8b8a0f124d91320208824845d6329f9af6f/dnacauldron-0.1.4.tar.gz
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-install-e_j8hp_r/dnacauldron/setup.py", line 13, in <module>
        long_description=open('pypi-readme.rst').read(),
    FileNotFoundError: [Errno 2] No such file or directory: 'pypi-readme.rst'

I edited the MANIFEST.in file so that pypi-readme.rst and ez_setup.py are added to the source distribution.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 76.432% when pulling 227adcf on althonos:master into 1e27397 on Edinburgh-Genome-Foundry:master.

@Zulko Zulko merged commit 8cd52ea into Edinburgh-Genome-Foundry:master Jul 4, 2018
@Zulko
Copy link
Member

Zulko commented Jul 4, 2018

Thanks for catching this.

@Zulko
Copy link
Member

Zulko commented Jul 4, 2018

I just updated the version on PyPI so pip install should work just fine now.

@althonos
Copy link
Contributor Author

althonos commented Jul 4, 2018

Just tested and it installs fine 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants