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

Loosen versioned dependencies. #159

Merged
merged 1 commit into from Apr 28, 2016

Conversation

ghisvail
Copy link
Contributor

We should not require such strict versioned dependencies for setuptools and wxpython. Changing to minimum versions rather than strict ones.

@@ -89,7 +89,7 @@ def txt_read(*paths):
# requirements
setup_args['setup_requires'] =\
[
"setuptools==19.2",
"setuptools>=19.2",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The only reason of the '==' is in case of freezing with PyInstaller, otherwise the version can be dropped:
pyinstaller/pyinstaller#1773
Although the PyInstaller ticket is closed, I had to revert to such a version to freeze on OS X.

@jreadey : have you been able to freeze on OS X? Did you need to downgrade to setuptools 19.2?
However, since this bit of info is now in the wiki, I would actually propose to drop the version requirement.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ghisvail Have you been able to freeze HDF Compass on your favorite platform?

@ghisvail
Copy link
Contributor Author

Shall I hold off my packaging effort then? Seems that there is still some uncertainties regarding the dependencies?

@giumas
Copy link
Collaborator

giumas commented Apr 27, 2016

I am not sure that I understood. Does the Debian packaging work with the 2 suggested modifications?

@ghisvail
Copy link
Contributor Author

I can build with the modifications, indeed. Whether they are loosen or dropped does not matter much on my end.

@giumas
Copy link
Collaborator

giumas commented Apr 27, 2016

Can you modify the PR to drop the setuptools version dependency? Leaving just setuptools

@ghisvail
Copy link
Contributor Author

so leaving wxpython as is (strict versioning) and dropping setuptools?

@giumas
Copy link
Collaborator

giumas commented Apr 27, 2016

@ghisvail For 'setuptools', I would leave it just as dependency, without a specific release. For 'wxPython', since '3.0.2' seems to be the last official release, it does not currently matter. I would be curious to know what is the wxPython release that is available in Debian.

@jreadey With such a modification, this PR is fine. Do you need to contact @ghisvail as a contributor?

@jreadey
Copy link
Member

jreadey commented Apr 27, 2016

@ghisvail - this is a pretty trivial change, but if you anticipate making more contributions going forward let me know (and send me your email address), and I'll send you a contributors form to fill out.

@ghisvail
Copy link
Contributor Author

For 'setuptools', I would leave it just as dependency, without a specific release.

Done

this is a pretty trivial change

Indeed, this should not require me signing a CLA.

@giumas giumas merged commit b1dda4d into HDFGroup:develop Apr 28, 2016
@giumas
Copy link
Collaborator

giumas commented Apr 28, 2016

Should we tag a 6.0.0b4?

@ghisvail
Copy link
Contributor Author

That would be helpful, thanks.

@ghisvail ghisvail mentioned this pull request Apr 28, 2016
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