#9963 distgit-branch-unused: use "rawhide" as the branch name
Closed 3 years ago by zbyszek. Opened 3 years ago by zbyszek.

@@ -8,13 +8,13 @@ 

  Examples:

  

  -\------A---\

-   \----------\----- master

+   \----------\----- rawhide

  

- 'A' has been merged into 'master', so it can be trivially deletected

+ 'A' has been merged into 'rawhide', so it can be trivially deletected

  without checking any builds.

  

    /---/-------\-B"-B'-B

- -/---/---\-----\----master

+ -/---/---\-----\---- rawhide

            \--C

  

  'B' has commits that are not found anywhere else (B, B', and B"), and
@@ -30,7 +30,7 @@ 

  cannot check if builds have been performed, so this script always

  refuses removal.

  

- Removal of the 'master' branch is always refused.

+ Removal of the 'rawhide' branch is always refused.

  

  3. Removal is refused in some additional corner cases:

  - the spec file cannot be parsed
@@ -220,8 +220,8 @@ 

          branch_name = branch.branch_name[l+1:]

          local = False

  

-     if branch_name == 'master':

-         print("Branch 'master' cannot be deleted.")

+     if branch_name in {'master', 'rawhide'}:

+         print(f"Branch '{branch_name}' cannot be deleted.")

          return 1

  

      if bodhi_builds_exist(branch_name, opts.package, opts):

The old "master" name is still checked. In preparation for
https://fedoraproject.org/wiki/Changes/GitRepos-master-to-main.

Signed-off-by: Zbigniew Jędrzejewski-Szmek zbyszek@in.waw.pl

Looks ok to me, can you rebase?

It's not useful any more. The code has already been changed to support rawhide, so the only part of this patch that is left is support for master, which actually I don't think we need.

Pull-Request has been closed by zbyszek

3 years ago
Metadata