#397 Allow retirement on epel branches
Merged a year ago by onosek. Opened a year ago by mohanboddu.
mohanboddu/fedpkg fix-epel-retirement  into  master

file modified
+8 -5
@@ -1260,10 +1260,13 @@ 

          Runs the rpkg retire command after check. Check includes reading the state

          of Fedora release.

          """

-         state = get_fedora_release_state(self.config, self.name, self.cmd.branch_merge)

- 

-         if state is None or state == 'pending':

+         if is_epel(self.cmd.branch_merge):

              super(fedpkgClient, self).retire()

          else:

-             self.log.error("Fedora release (%s) is in state '%s' - retire operation "

-                            "is not allowed." % (self.cmd.branch_merge, state))

+             state = get_fedora_release_state(self.config, self.name, self.cmd.branch_merge)

+ 

+             if state is None or state == 'pending':

+                 super(fedpkgClient, self).retire()

+             else:

+                 self.log.error("Fedora release (%s) is in state '%s' - retire operation "

+                                "is not allowed." % (self.cmd.branch_merge, state))

Commit 45f95b6 fixes this pull-request

Pull-Request has been merged by onosek

a year ago

@mohanboddu Thanks for the fix, I will add unittests for this in another commit.