diff --git a/module_build_service/builder/CoprModuleBuilder.py b/module_build_service/builder/CoprModuleBuilder.py index 6237ab8b..7c176187 100644 --- a/module_build_service/builder/CoprModuleBuilder.py +++ b/module_build_service/builder/CoprModuleBuilder.py @@ -125,9 +125,7 @@ class CoprModuleBuilder(GenericBuilder): def _create_module_safe(self): from copr.exceptions import CoprRequestException - modulemd = tempfile.mktemp() - self.module.mmd().dump(modulemd) - + modulemd = self._dump_mmd() kwargs = { "username": self.module.copr_owner or self.owner, "projectname": self.module.copr_project or @@ -144,6 +142,18 @@ class CoprModuleBuilder(GenericBuilder): finally: os.remove(modulemd) + def _dump_mmd(self): + # Write module's name, stream and version into the modulemd file + # so Copr can parse it from there + mmd = self.module.mmd() + mmd.name = str(self.module.name) + mmd.stream = str(self.module.stream) + mmd.version = int(self.module.version) + + modulemd = tempfile.mktemp() + mmd.dump(modulemd) + return modulemd + def buildroot_ready(self, artifacts=None): """ :param artifacts=None : a list of artifacts supposed to be in the buildroot @@ -295,8 +305,7 @@ class CoprModuleBuilder(GenericBuilder): chroots=[self.chroot]) def finalize(self): - modulemd = tempfile.mktemp() - self.module.mmd().dump(modulemd) + modulemd = self._dump_mmd() # Create a module from previous project result = self.client.make_module(username=self.copr.username,