diff --git a/module_build_service/builder/KojiModuleBuilder.py b/module_build_service/builder/KojiModuleBuilder.py index 2f82b7ce..6c7c2835 100644 --- a/module_build_service/builder/KojiModuleBuilder.py +++ b/module_build_service/builder/KojiModuleBuilder.py @@ -531,7 +531,7 @@ class KojiModuleBuilder(GenericBuilder): self.tag_name + "-build", self.arches, perm="admin") buildopts = self.mmd.get_buildopts() - if buildopts: + if buildopts and buildopts.get_rpm_whitelist(): rpm_whitelist = buildopts.get_rpm_whitelist() else: rpm_whitelist = self.components diff --git a/tests/test_builder/test_koji.py b/tests/test_builder/test_koji.py index 2f650304..9cca97c2 100644 --- a/tests/test_builder/test_koji.py +++ b/tests/test_builder/test_koji.py @@ -487,6 +487,14 @@ class TestKojiBuilder: opts.add_rpm_to_whitelist("custom2") mmd.set_buildopts(opts) self.module.modulemd = mmd_to_str(mmd) + else: + # Set some irrelevant buildopts options to test that KojiModuleBuilder + # is not confused by this. + mmd = self.module.mmd() + opts = Modulemd.Buildopts() + opts.set_rpm_macros("%my_macro 1") + mmd.set_buildopts(opts) + self.module.modulemd = mmd_to_str(mmd) if repo_include_all is False: mmd = self.module.mmd()