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

Add missing requirements.txt to fix installation from PyPI. #11

Merged
merged 1 commit into from Nov 19, 2017

Conversation

Projects
None yet
3 participants
@lamby
Contributor

lamby commented Nov 19, 2017

See also #10

@victor-torres victor-torres merged commit fd2ca5d into EvoluxBR:master Nov 19, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@victor-torres

This comment has been minimized.

Show comment
Hide comment
@victor-torres

victor-torres Nov 19, 2017

Contributor

I'll be releasing a new version tomorrow morning. Thanks.

Contributor

victor-torres commented Nov 19, 2017

I'll be releasing a new version tomorrow morning. Thanks.

@lamby lamby changed the title from Add missing requirement.txt to fix installation from PyPI. to Add missing requirements.txt to fix installation from PyPI. Nov 19, 2017

@victor-torres

This comment has been minimized.

Show comment
Hide comment
@victor-torres

victor-torres Nov 20, 2017

Contributor

Done

Contributor

victor-torres commented Nov 20, 2017

Done

@areski

This comment has been minimized.

Show comment
Hide comment
@areski

areski Dec 11, 2017

Contributor

Hi,
I just tried to install it and go an error with missing requirements.txt:
File "/tmp/pip-build-k7PVq_/python-redis-rate-limit/setup.py", line 9, in
with open('requirements.txt') as f:
IOError: [Errno 2] No such file or directory: 'requirements.txt'

Contributor

areski commented Dec 11, 2017

Hi,
I just tried to install it and go an error with missing requirements.txt:
File "/tmp/pip-build-k7PVq_/python-redis-rate-limit/setup.py", line 9, in
with open('requirements.txt') as f:
IOError: [Errno 2] No such file or directory: 'requirements.txt'

@lamby

This comment has been minimized.

Show comment
Hide comment
@lamby

lamby Dec 11, 2017

Contributor

@victor-torres Which version was this fixed in?

@areski Which version are you attempting to install?

:)

Contributor

lamby commented Dec 11, 2017

@victor-torres Which version was this fixed in?

@areski Which version are you attempting to install?

:)

@areski

This comment has been minimized.

Show comment
Hide comment
@areski

areski Dec 11, 2017

Contributor

I'm trying to install the latest version from pypi, v0.0.4
pip install python-redis-rate-limit
Collecting python-redis-rate-limit
Downloading python-redis-rate-limit-0.0.4.tar.gz

v0.0.4 was released on the 30-11-2017, therefore it should be up to date with the latest commits.

strangely it works if I install it directly bypassing pypi "pip install -e git+git://github.com/areski/python-redis-rate-limit.git#egg=python-redis-rate-limit"

Contributor

areski commented Dec 11, 2017

I'm trying to install the latest version from pypi, v0.0.4
pip install python-redis-rate-limit
Collecting python-redis-rate-limit
Downloading python-redis-rate-limit-0.0.4.tar.gz

v0.0.4 was released on the 30-11-2017, therefore it should be up to date with the latest commits.

strangely it works if I install it directly bypassing pypi "pip install -e git+git://github.com/areski/python-redis-rate-limit.git#egg=python-redis-rate-limit"

@lamby

This comment has been minimized.

Show comment
Hide comment
@lamby

lamby Dec 11, 2017

Contributor

0.0.4 on PyPI is missing the requirements.txt file from the tarball so specifying the git... directly will work. @victor-torres, perhaps you uploaded a different branch/tag something? :)

Contributor

lamby commented Dec 11, 2017

0.0.4 on PyPI is missing the requirements.txt file from the tarball so specifying the git... directly will work. @victor-torres, perhaps you uploaded a different branch/tag something? :)

@victor-torres

This comment has been minimized.

Show comment
Hide comment
@victor-torres

victor-torres Dec 11, 2017

Contributor

Prepended lines with include in the MANIFEST.in file and released 0.0.5, it should be working now. Thanks for reporting.

Contributor

victor-torres commented Dec 11, 2017

Prepended lines with include in the MANIFEST.in file and released 0.0.5, it should be working now. Thanks for reporting.

@lamby

This comment has been minimized.

Show comment
Hide comment
@lamby

lamby Dec 11, 2017

Contributor

Prepended lines with include in the MANIFEST.in file

Ahhh!

Contributor

lamby commented Dec 11, 2017

Prepended lines with include in the MANIFEST.in file

Ahhh!

@lamby lamby deleted the lamby:setup-py-requirements branch Dec 11, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment