-
-
Notifications
You must be signed in to change notification settings - Fork 147
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
distutils import is problematic (causes error with frozen tests) #29
Comments
Today I thought with myself "I think I will test packages with devpi after this release"... well, what do you know! 😅 I'm working on a fix as well. |
Release |
Sorry 😄. I actually had it like this before – that's why there is this test: 773f8d9#diff-7cc07d0976b2f8543c7956925eb81df9R334 But then I really disliked doing something this basic ourselves and found the distutils function... ^^. |
For the record, I think it'd make a lot of sense having this utility function added to the pytest core for plugins to use. Want to make a PR by any chance? 😆 |
distutils
is a somewhat weird biest, and it's used in #27. I thought a simple util method would be okay while reviewing - turns out I was wrong.I get this when trying to run the tests frozen via cx_Freeze:
I guess it'd be better to just do
{'false': False, 'true': True}[val.lower()]
or so.cc @Chronial
The text was updated successfully, but these errors were encountered: