commit 2fef5f6b70b987a839e4a907e9858c37a56f5100 Author: Damian Johnson atagar@torproject.org Date: Mon Nov 18 16:30:50 2019 -0800
Fix is_available() ternary
This was simply returning the result of shutil. The proper ternary is...
return True if shutil.which(command) else False
That said, this really doesn't matter. Usage within conditionals implicitly converts this to a boolean anyway. --- nyx/__init__.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/nyx/__init__.py b/nyx/__init__.py index 0418567..0efdbed 100644 --- a/nyx/__init__.py +++ b/nyx/__init__.py @@ -51,8 +51,10 @@ import threading import time import shutil
+ def is_available(command): - return shutil.which(command) if True else False + return bool(shutil.which(command)) +
class PkgManagers: ARCHLINUX = 'pacman' @@ -93,7 +95,7 @@ except ImportError as exc: else: pkg_manager_not_found = True pkg_install_procedure = ', you can find it at https://stem.torproject.org/download.html' - + require_message = 'nyx requires python-stem' if pkg_manager_not_found: final_message = require_message + pkg_install_procedure