This is an automated email from the ASF dual-hosted git repository. hapylestat pushed a commit to branch trunk in repository https://gitbox.apache.org/repos/asf/ambari.git
commit 737dfc7781d06df9072be3f1559229b21c38a1c8 Author: Dmytro Grinenko <[email protected]> AuthorDate: Wed May 2 09:38:54 2018 +0300 Revert "initial" This reverts commit fb178d2fa3cdf6f2586648264a1888357dd95a3f. --- .../src/main/python/ambari_commons/repo_manager/apt_manager.py | 3 --- .../src/main/python/ambari_commons/repo_manager/yum_manager.py | 3 --- .../src/main/python/ambari_commons/repo_manager/zypper_manager.py | 3 --- 3 files changed, 9 deletions(-) diff --git a/ambari-common/src/main/python/ambari_commons/repo_manager/apt_manager.py b/ambari-common/src/main/python/ambari_commons/repo_manager/apt_manager.py index 76d31f1..38cf602 100644 --- a/ambari-common/src/main/python/ambari_commons/repo_manager/apt_manager.py +++ b/ambari-common/src/main/python/ambari_commons/repo_manager/apt_manager.py @@ -304,8 +304,5 @@ class AptManager(GenericManager): apt-get in inconsistant state (locked, used, having invalid repo). Once packages are installed we should not rely on that. """ - if not name: - raise ValueError("Package name can't be empty") - r = shell.subprocess_executor(self.properties.check_cmd % name) return not bool(r.code) diff --git a/ambari-common/src/main/python/ambari_commons/repo_manager/yum_manager.py b/ambari-common/src/main/python/ambari_commons/repo_manager/yum_manager.py index 6a2b629..5e404b1 100644 --- a/ambari-common/src/main/python/ambari_commons/repo_manager/yum_manager.py +++ b/ambari-common/src/main/python/ambari_commons/repo_manager/yum_manager.py @@ -263,9 +263,6 @@ class YumManager(GenericManager): yum in inconsistant state (locked, used, having invalid repo). Once packages are installed we should not rely on that. """ - if not name: - raise ValueError("Package name can't be empty") - if os.geteuid() == 0: return self.yum_check_package_available(name) else: diff --git a/ambari-common/src/main/python/ambari_commons/repo_manager/zypper_manager.py b/ambari-common/src/main/python/ambari_commons/repo_manager/zypper_manager.py index 9f6f09a..8617ef9 100644 --- a/ambari-common/src/main/python/ambari_commons/repo_manager/zypper_manager.py +++ b/ambari-common/src/main/python/ambari_commons/repo_manager/zypper_manager.py @@ -282,7 +282,4 @@ class ZypperManager(GenericManager): zypper in inconsistant state (locked, used, having invalid repo). Once packages are installed we should not rely on that. """ - if not name: - raise ValueError("Package name can't be empty") - return self.rpm_check_package_available(name) -- To stop receiving notification emails like this one, please contact [email protected].
