-
Notifications
You must be signed in to change notification settings - Fork 202
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
Use requests instead of urllib2 for 403 errors. #2695
Use requests instead of urllib2 for 403 errors. #2695
Conversation
…3 error. requests comes to the rescue there too.
@bartoldeman Can we cover this in the tests? See |
@bartoldeman Now blocked because of a merge conflict, please sync with current Let me know if you need help by covering this in the tests. |
@bartoldeman Ping on making this ready to go, maybe include in EasyBuild v3.9.1 (window is closing...) |
@bartoldeman Pinging again... |
ok, will have another look. |
Conflicts: easybuild/tools/filetools.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.
lgtm
NASM has this issue. requests comes to the rescue there too.