-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Revert 981 only setup py #1020
Revert 981 only setup py #1020
Conversation
Only use `setup.py`
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we still want to go the setup.py route, we need to make sure it:
- installs local module requirements, in a fresh copy (
make clean
or fresh git clone) - doesn't move the requirements to the full test_libs folder. That's just a hack to avoid using local dependencies. I prefer to separate debug/util packages from the pyspec module.
- don't trust CI for an upgrade, it caches things.
@djrtwo @protolambda
Do you mean in CI? It seems another issue, not related to
Do you mean my latest comment: Continue the discussion on #981, I'm still pro-single If we using one single deps = {
'config_helpers': [
"ruamel.yaml==0.15.87",
],
}
deps += {
'pyspec': [
"eth-utils>=1.3.0,<2",
"eth-typing>=2.1.0,<3.0.0",
"pycryptodome==3.7.3",
"py_ecc>=1.6.0",
],
'pyspec-test': [
"pytest>=3.6,<3.7",
] + deps['config_helpers'],
...
}
deps += {
'pyspec-dev': deps['pyspec'] + deps['pyspec-test']
} And install it with I see there's more value in organizing all dependencies in one file:
Right, that is a nightmare. Next time we can probably update the version: https://circleci.com/docs/2.0/caching/#clearing-cache when updating without changing checksum file. |
No description provided.