From 70cc6bb182245d20c54afba79a265ef40f1ed080 Mon Sep 17 00:00:00 2001
From: "Chris St. Pierre" <chris.a.st.pierre@gmail.com>
Date: Mon, 25 Mar 2013 11:39:50 -0400
Subject: Revert "Packages: properly implemented deepcopy() for PackagesSources
 objects"

This reverts commit 432f448983ff27452d82d62314d91c942f31bce5.
---
 src/lib/Bcfg2/Server/Plugins/Packages/PackagesSources.py | 12 ++----------
 src/lib/Bcfg2/Server/Plugins/Packages/Source.py          |  7 +------
 src/lib/Bcfg2/Server/Plugins/Packages/__init__.py        | 11 +----------
 3 files changed, 4 insertions(+), 26 deletions(-)

(limited to 'src/lib/Bcfg2/Server/Plugins')

diff --git a/src/lib/Bcfg2/Server/Plugins/Packages/PackagesSources.py b/src/lib/Bcfg2/Server/Plugins/Packages/PackagesSources.py
index 3069e4068..2735e389a 100644
--- a/src/lib/Bcfg2/Server/Plugins/Packages/PackagesSources.py
+++ b/src/lib/Bcfg2/Server/Plugins/Packages/PackagesSources.py
@@ -8,17 +8,13 @@ from Bcfg2.Server.Plugins.Packages.Source import SourceInitError
 
 
 class PackagesSources(Bcfg2.Server.Plugin.StructFile,
-                      Bcfg2.Server.Plugin.Debuggable,
-                      Bcfg2.Server.Plugin.DeepcopyMixin):
+                      Bcfg2.Server.Plugin.Debuggable):
     """ PackagesSources handles parsing of the
     :mod:`Bcfg2.Server.Plugins.Packages` ``sources.xml`` file, and the
     creation of the appropriate
     :class:`Bcfg2.Server.Plugins.Packages.Source.Source` object for
     each ``Source`` tag. """
 
-    _deepcopy_exclude = Bcfg2.Server.Plugin.DeepcopyMixin._deepcopy_exclude + \
-        ['pkg_obj']
-
     __identifier__ = None
 
     def __init__(self, filename, cachepath, fam, packages, setup):
@@ -43,7 +39,6 @@ class PackagesSources(Bcfg2.Server.Plugin.StructFile,
                  If ``sources.xml`` cannot be read
         """
         Bcfg2.Server.Plugin.Debuggable.__init__(self)
-        Bcfg2.Server.Plugin.DeepcopyMixin.__init__(self)
         try:
             Bcfg2.Server.Plugin.StructFile.__init__(self, filename, fam=fam,
                                                     should_monitor=True)
@@ -134,7 +129,7 @@ class PackagesSources(Bcfg2.Server.Plugin.StructFile,
         """ Create a
         :class:`Bcfg2.Server.Plugins.Packages.Source.Source` subclass
         object from XML representation of a source in ``sources.xml``.
-        ``source_from_xml`` determines the appropriate subclass of
+        ``source_from-xml`` determines the appropriate subclass of
         ``Source`` to instantiate according to the ``type`` attribute
         of the ``Source`` tag.
 
@@ -181,6 +176,3 @@ class PackagesSources(Bcfg2.Server.Plugin.StructFile,
 
     def __len__(self):
         return len(self.entries)
-
-    def _deepcopy_constructor_args(self):
-        return (self.name, self.cachepath, None, None, dict(self.setup))
diff --git a/src/lib/Bcfg2/Server/Plugins/Packages/Source.py b/src/lib/Bcfg2/Server/Plugins/Packages/Source.py
index 40c1f6676..985405e65 100644
--- a/src/lib/Bcfg2/Server/Plugins/Packages/Source.py
+++ b/src/lib/Bcfg2/Server/Plugins/Packages/Source.py
@@ -92,8 +92,7 @@ class SourceInitError(Exception):
 REPO_RE = re.compile(r'(?:pulp/repos/|/RPMS\.|/)([^/]+)/?$')
 
 
-class Source(Bcfg2.Server.Plugin.Debuggable,  # pylint: disable=R0902
-             Bcfg2.Server.Plugin.DeepcopyMixin):
+class Source(Bcfg2.Server.Plugin.Debuggable):  # pylint: disable=R0902
     """ ``Source`` objects represent a single <Source> tag in
     ``sources.xml``.  Note that a single Source tag can itself
     describe multiple repositories (if it uses the "url" attribute
@@ -129,7 +128,6 @@ class Source(Bcfg2.Server.Plugin.Debuggable,  # pylint: disable=R0902
         :raises: :class:`Bcfg2.Server.Plugins.Packages.Source.SourceInitError`
         """
         Bcfg2.Server.Plugin.Debuggable.__init__(self)
-        Bcfg2.Server.Plugin.DeepcopyMixin.__init__(self)
 
         #: The base filesystem path under which cache data for this
         #: source should be stored
@@ -740,6 +738,3 @@ class Source(Bcfg2.Server.Plugin.Debuggable,  # pylint: disable=R0902
                 if group in metadata.groups:
                     return True
             return False
-
-    def _deepcopy_constructor_args(self):
-        return (self.basepath, self.xsource, dict(self.setup))
diff --git a/src/lib/Bcfg2/Server/Plugins/Packages/__init__.py b/src/lib/Bcfg2/Server/Plugins/Packages/__init__.py
index cc78497f1..308a0efc4 100644
--- a/src/lib/Bcfg2/Server/Plugins/Packages/__init__.py
+++ b/src/lib/Bcfg2/Server/Plugins/Packages/__init__.py
@@ -185,14 +185,6 @@ class Packages(Bcfg2.Server.Plugin.Plugin,
         for (key, value) in list(attrib.items()):
             entry.attrib.__setitem__(key, value)
 
-    def get_config(self, metadata):
-        """ Get yum/apt config, as a string, for the specified client.
-
-        :param metadata: The client to create the config for.
-        :type metadata: Bcfg2.Server.Plugins.Metadata.ClientMetadata
-        """
-        return self.get_collection(metadata).get_config()
-
     def HandleEntry(self, entry, metadata):
         """ Bind configuration entries.  ``HandleEntry`` handles
         entries two different ways:
@@ -546,8 +538,7 @@ class Packages(Bcfg2.Server.Plugin.Plugin,
         """
         collection = self.get_collection(metadata)
         return dict(sources=collection.get_additional_data(),
-                    allsources=copy.deepcopy(self.sources),
-                    get_config=self.get_config)
+                    allsources=copy.deepcopy(self.sources))
 
     def end_client_run(self, metadata):
         """ Hook to clear the cache for this client in
-- 
cgit v1.2.3-1-g7c22