-
Notifications
You must be signed in to change notification settings - Fork 51
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
Cleanup #257
Cleanup #257
Conversation
# (see http://stackoverflow.com/questions/4748344/whats-the-reverse-of-shlex-split for alternatives if needed) | ||
return subprocess.list2cmdline([str(x)]) | ||
"""Add quotes so it can be passed to shell""" | ||
return quote(str(x)) |
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.
Can you check whether this implies that we can remove the monkey patching we do in EasyBuild?
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.
yep, looks like this should fix this.
I was already wondering why nobody complained about this, seems like it's because you had a workaround :)
@boegel i'm going to merge this, we should not forget to push to pypi |
No description provided.