-
Notifications
You must be signed in to change notification settings - Fork 998
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
Add conda search function #2395
Conversation
@@ -70,7 +70,7 @@ def execute(cmds): | |||
|
|||
Return the standard output if the commands are successful | |||
""" | |||
return _wait(cmds, shell=False) | |||
return __wait(cmds, shell=False, stdin=subprocess.PIPE, stdout=subprocess.PIPE) |
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.
Change back __wait()
to _wait()
? You haven't changed the function name below and double underscore should be used only if you need name mangling (see https://stackoverflow.com/questions/1301346/the-meaning-of-a-single-and-a-double-underscore-before-an-object-name-in-python ).
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.
This was a bad merge, sorry. Will fix now.
👍 |
thanks @bgruening , will try to make use of it! |
@martenson thanks! :) |
This function is useful to check in advance if a package is available.
I hope this can be used by @martenson in #2250 to display cute little anaconda icons during the tool installation.
Something similar could be done for the Docker.