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

Added FileNotFoundError to custom_build_ext.run(). #162

Merged
merged 2 commits into from
Apr 25, 2019

Conversation

acksmaggart
Copy link
Contributor

This pull request adds FileNotFoundError to the except list in setup.py after verifying that it fixed the installation problems on the test machine, which was Windows Server 2012 R2.

Fixes issue #161.

@tobgu
Copy link
Owner

tobgu commented Apr 25, 2019

Did adding this help? If so, please also add it to line 51 in the same file. It's more or less a duplicate used for testing. Common code could potentially be factored out.

@acksmaggart
Copy link
Contributor Author

It did fix it, luckily.

I just pushed an update adding it to line 51. Let me know if there is anything else. (I'm still planning on finishing the wheel CI stuff, by the way)

@coveralls
Copy link

coveralls commented Apr 25, 2019

Coverage Status

Coverage remained the same at 95.676% when pulling 75e52bc on MaxTaggart:WindowsInstallFix into c79eb03 on tobgu:master.

@tobgu tobgu merged commit 4be1c70 into tobgu:master Apr 25, 2019
@tobgu
Copy link
Owner

tobgu commented Apr 25, 2019

Great, thanks!

@codeimpossible
Copy link

This seems to be throwing a global name 'FileNotFoundError' is not defined when installing pyrsistent on python 2.7...

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.

4 participants