Restore the original functionality, even if it is weird.

This commit is contained in:
Ralph Bean
2017-03-02 11:23:04 -05:00
parent 724599fb77
commit 8d2b7729bd
2 changed files with 8 additions and 4 deletions

View File

@@ -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']:

View File

@@ -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)