From 6ae8e51901ee2712962e04875d501b0ca16392f6 Mon Sep 17 00:00:00 2001
From: Cole Robinson <crobinso@redhat.com>
Date: Sat, 1 Sep 2018 07:55:19 -0400
Subject: [PATCH] osdict: Remove unused functions

---
 tests/osdict.py    | 12 ---------
 virtinst/osdict.py | 61 +---------------------------------------------
 2 files changed, 1 insertion(+), 72 deletions(-)

diff --git a/tests/osdict.py b/tests/osdict.py
index 4c299962..2480e6d0 100644
--- a/tests/osdict.py
+++ b/tests/osdict.py
@@ -22,18 +22,6 @@ class TestOSDB(unittest.TestCase):
                 "should never be extended, since it is only for back "
                 "compat with pre-libosinfo osdict.py"))
 
-    def test_osdict_types_ro(self):
-        # 'types' should rarely be altered, this check will make
-        # doubly sure that a new type isn't accidentally added
-        approved_types = OSDB.list_types()
-
-        for osobj in OSDB.list_os():
-            if osobj.get_typename() not in approved_types:
-                raise AssertionError("OS entry '%s' has OS type '%s'.\n"
-                    "The type list should NOT be extended without a lot of "
-                    "thought, please make sure you know what you are doing." %
-                    (osobj.name, osobj.get_typename()))
-
     def test_recommended_resources(self):
         conn = utils.URIs.open_testdefault_cached()
         guest = conn.caps.lookup_virtinst_guest()
diff --git a/virtinst/osdict.py b/virtinst/osdict.py
index 310cb223..41c90211 100644
--- a/virtinst/osdict.py
+++ b/virtinst/osdict.py
@@ -7,7 +7,6 @@
 # This work is licensed under the GNU GPLv2 or later.
 # See the COPYING file in the top-level directory.
 
-import datetime
 import logging
 import re
 import time
@@ -180,11 +179,6 @@ class _OSDB(object):
 
         return osname
 
-    def list_types(self):
-        approved_types = ["linux", "windows", "bsd", "macos",
-            "solaris", "other", "generic"]
-        return approved_types
-
     def list_os(self):
         """
         List all OSes in the DB
@@ -243,7 +237,6 @@ class _OsVariant(object):
 
         self.sortby = self._get_sortby()
         self.urldistro = self._get_urldistro()
-        self._supported = None
 
 
     ########################
@@ -307,29 +300,6 @@ class _OsVariant(object):
 
         return "%s-%s" % (self.distro, version)
 
-    def _get_supported(self):
-        if not self._os:
-            return True
-
-        eol_date = self._os.get_eol_date_string()
-
-        if eol_date:
-            return (datetime.datetime.strptime(eol_date, "%Y-%m-%d") >
-                    datetime.datetime.now())
-
-        if self.name == "fedora-unknown":
-            return False
-
-        # As of libosinfo 2.11, many clearly EOL distros don't have an
-        # EOL date. So assume None == EOL, add some manual work arounds.
-        # We should fix this in a new libosinfo version, and then drop
-        # this hack
-        if self._is_related_to(["fedora24", "rhel7.0", "debian6",
-            "ubuntu13.04", "win8", "win2k12", "mageia5", "centos7.0"],
-            check_clones=False, check_derives=False):
-            return True
-        return False
-
     def _get_urldistro(self):
         if not self._os:
             return None
@@ -350,37 +320,8 @@ class _OsVariant(object):
     # Public APIs #
     ###############
 
-    def get_supported(self):
-        if self._supported is None:
-            self._supported = self._get_supported()
-        return self._supported
-
-    def get_typename(self):
-        """
-        Streamline the family name for use in the virt-manager UI
-        """
-        if not self._os:
-            return "generic"
-
-        if self._family in ['linux']:
-            return "linux"
-
-        if self._family in ['win9x', 'winnt', 'win16']:
-            return "windows"
-
-        if self._family in ['solaris']:
-            return "solaris"
-
-        if self._family in ['openbsd', 'freebsd', 'netbsd']:
-            return "bsd"
-
-        if self._family in ['darwin']:
-            return "macos"
-
-        return "other"
-
     def is_windows(self):
-        return self.get_typename() == "windows"
+        return self._family in ['win9x', 'winnt', 'win16']
 
     def broken_x2apic(self):
         # x2apic breaks networking in solaris10