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

setup.py's install_requires is overly strict #45

Open
crccheck opened this issue Dec 19, 2014 · 1 comment
Open

setup.py's install_requires is overly strict #45

crccheck opened this issue Dec 19, 2014 · 1 comment

Comments

@crccheck
Copy link

it's currently:

 install_requires = [
'PyYAML==3.11',
'SQLAlchemy==0.9.6',
'humanize==0.5',
'pytest==2.5.2',
'schema==0.3.1',
'click==3.1',
'SQLAlchemy-Utils==0.26.11',
'psutil==2.1.1',
]

The problems I see are:

  1. pinned to specific versions. This is a major barrier to actively using this because installing stellar would change my installed versions, when my already installed versions maybe would have worked. Possible solutions: use a range of versions, or use no versions and let developers figure it out
  2. installing things that aren't even used. For example, pytest isn't required to use stellar.
@Teemu
Copy link
Contributor

Teemu commented Jan 5, 2015

pinned to specific versions. This is a major barrier to actively using this because installing stellar would > change my installed versions, when my already installed versions maybe would have worked. Possible > solutions: use a range of versions, or use no versions and let developers figure it out

Try installing stellar on it's own isolated virtualenv so it doesn't affect your projects. Unfortunately pip doesn't exactly make this easy so I can see this is a practical problem.

installing things that aren't even used. For example, pytest isn't required to use stellar.
Good point.

@Teemu Teemu closed this as completed Mar 17, 2015
@Teemu Teemu reopened this Mar 17, 2015
tvuotila added a commit that referenced this issue Oct 20, 2017
tvuotila added a commit that referenced this issue Oct 20, 2017
tvuotila added a commit that referenced this issue Oct 24, 2017
tvuotila added a commit that referenced this issue Oct 24, 2017
quantus added a commit that referenced this issue Nov 13, 2017
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

No branches or pull requests

2 participants