From 8d2b7729bd58b3d180982c1ee82479c2fba83d04 Mon Sep 17 00:00:00 2001
From: Ralph Bean <rbean@redhat.com>
Date: Mar 02 2017 16:23:04 +0000
Subject: Restore the original functionality, even if it is weird.


---

diff --git a/module_build_service/pdc.py b/module_build_service/pdc.py
index 3f338ca..1d30669 100644
--- a/module_build_service/pdc.py
+++ b/module_build_service/pdc.py
@@ -301,8 +301,11 @@ def module_depsolving_wrapper(session, modules, strict=True):
 
             # Take note of the tag of this module, but only if it is a dep and
             # not in the original list.
-            if module_dict not in original_modules:
-                module_tags.add(info['koji_tag'])
+            #if module_dict not in original_modules:
+            #    module_tags.add(info['koji_tag'])
+            # XXX - But, for now go ahead and include it because that's how this
+            # code used to work.
+            module_tags.add(info['koji_tag'])
 
             # Queue up the next tier of deps that we should look at..
             for pdc_dep in info['build_deps']:
diff --git a/tests/test_pdc.py b/tests/test_pdc.py
index 84facbf..256aa6a 100644
--- a/tests/test_pdc.py
+++ b/tests/test_pdc.py
@@ -57,7 +57,8 @@ class TestPDCModule(unittest.TestCase):
         result = mbs_pdc.module_depsolving_wrapper(self.pdc, query)
         expected = [
             u'module-bootstrap-master-1',
-            # The list of deps should not include the original tag.
-            #u'module-testmodule-master-20170228215102',
+            # Should the list of deps should not include the original tag?
+            # Probably not.
+            u'module-testmodule-master-20170228215102',
         ]
         self.assertEqual(result, expected)