-
Notifications
You must be signed in to change notification settings - Fork 60
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
Does not run under py 2.7 #42
Comments
Ah sorry for missing this previously. Is python 2.7 support important for you? |
No but it caught me by surprise as |
This is the Dockerfile https://github.com/lsst-sqre/sandbox-cmirror/blob/master/cmirror/Dockerfile (if anyone else wants to package it up this way). |
Oh. Yeah. That's a very good point. I'll make it clear that this project is py3 only. I really don't like supporting mixed py2/py3 projects. |
I just discovered this package but I'm unable to to get it run on stock el7 python with either
0.6.5
published on pypi or from master. This appears to be because of the usage oftempfile.TemporaryDirectory
, which is a py >= 3.2 construct. It seems like a py 2.7 compatible tempdir wrapper needs to be used or the import from future can be removed.The text was updated successfully, but these errors were encountered: