So we can move install_what_provides there and simplify the SoftwareManager class.
Signed-off-by: Lucas Meneghel Rodrigues <l...@redhat.com> --- client/shared/software_manager.py | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/client/shared/software_manager.py b/client/shared/software_manager.py index 2712e68..c27262d 100755 --- a/client/shared/software_manager.py +++ b/client/shared/software_manager.py @@ -130,12 +130,13 @@ class SoftwareManager(object): def __getattr__(self, name): - if name == "install_what_provides": - return self.install_what_provides(file) - else: - return self.backend.__getattribute__(self, name) + return self.backend.__getattribute__(name) +class BaseBackend(object): + """ + This class implements all common methods among backends. + """ def install_what_provides(self, file): """ Installs package that provides [file]. @@ -149,7 +150,7 @@ class SoftwareManager(object): logging.warning('No package seems to provide %s', file) -class RpmBackend(object): +class RpmBackend(BaseBackend): """ This class implements operations executed with the rpm package manager. @@ -234,7 +235,7 @@ class RpmBackend(object): return [] -class DpkgBackend(object): +class DpkgBackend(BaseBackend): """ This class implements operations executed with the dpkg package manager. -- 1.7.10.4 _______________________________________________ Autotest mailing list Autotest@test.kernel.org http://test.kernel.org/cgi-bin/mailman/listinfo/autotest