From ddd371d76cd32edcdb62f5f1e3991718c44a7c65 Mon Sep 17 00:00:00 2001 From: Jakub Kadlcik Date: Mon, 24 Jul 2017 23:33:43 +0200 Subject: [PATCH] Check whether the buildroot is preped for all build methods --- module_build_service/builder/CoprModuleBuilder.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/module_build_service/builder/CoprModuleBuilder.py b/module_build_service/builder/CoprModuleBuilder.py index 901a5cff..92ff1fde 100644 --- a/module_build_service/builder/CoprModuleBuilder.py +++ b/module_build_service/builder/CoprModuleBuilder.py @@ -236,6 +236,9 @@ class CoprModuleBuilder(GenericBuilder): """ log.info("Copr build") + if not self.__prep: + raise RuntimeError("Buildroot is not prep-ed") + # TODO: If we are sure that this method is thread-safe, we can just # remove _build_lock locking. with CoprModuleBuilder._build_lock: @@ -250,9 +253,6 @@ class CoprModuleBuilder(GenericBuilder): return response.data["ids"][0], koji.BUILD_STATES["BUILDING"], response.message, None def build_srpm(self, artifact_name, source, build_id=None): - if not self.__prep: - raise RuntimeError("Buildroot is not prep-ed") - # Build package from `source` return self.client.create_new_build(self.copr.projectname, [source], username=self.copr.username)