Chad has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/403155 )

Change subject: Fix branch.py
......................................................................

Fix branch.py

- Support bundles other than base
- Protect core branching with a flag
- Require explicit "really branch everything" syntax
- Remove unused options (they'll be back)
- Make manual branch points actually work

Change-Id: I0e0cbfd99e0247d35ee3a3065bf3527313def059
---
M make-release/branch.py
1 file changed, 16 insertions(+), 19 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/tools/release 
refs/changes/55/403155/1

diff --git a/make-release/branch.py b/make-release/branch.py
index bc76e74..6e2bf3e 100755
--- a/make-release/branch.py
+++ b/make-release/branch.py
@@ -24,9 +24,14 @@
         auth=HTTPDigestAuth(CONFIG['username'], CONFIG['password']))
 
 
-def create_branch(repository, branch, revision='HEAD'):
+def create_branch(repository, branch, revision):
     """Create a branch for a given repo."""
     try:
+        try:
+            revision = CONFIG['manual_branch_points'][branch][repository]
+        except KeyError:
+            pass
+
         _get_client().put(
             '/projects/%s/branches/%s' % (
                 repository.replace('/', '%2F'),
@@ -40,22 +45,12 @@
             raise
 
 
-def branch_core(branch, branch_point='HEAD'):
-    """Just branch core."""
-    create_branch('core', branch, branch_point)
-
-
-def branch_everything(branch, branch_point='HEAD', bundle=None):
+def branch_everything(branch, branch_point, bundle=None):
     """Branch stuff."""
-    if not bundle:
+    if bundle == '*':
         repos_to_branch = get_star_bundle()
 
     for repo in repos_to_branch:
-        try:
-            branch_point = CONFIG['manual_branch_points'][branch][repo]
-        except KeyError:
-            pass
-
         print('Branching %s to %s from %s' % (
             repo, branch, branch_point))
         create_branch(repo, branch, branch_point)
@@ -83,10 +78,10 @@
     parser.add_argument('branch', nargs='?', help='Branch we want to make')
     parser.add_argument('--branchpoint', dest='branch_point', default='HEAD',
                         help='Where to branch from')
-    parser.add_argument('--submodules', dest='submodules', action='store_true',
-                        help='Add the newly branched repos as submodules?')
-    parser.add_argument('--set-version', dest='set_version',
-                        help='Update $wgVersion?')
+    parser.add_argument('--core', dest='core', action='store_true',
+                        help='If we branch core or not')
+    parser.add_argument('--bundle', dest='bundle', default=None,
+                        help='What bundle of extensions & skins to branch')
 
     return parser.parse_args()
 
@@ -95,5 +90,7 @@
     OPTIONS = parse_args()
     logging.basicConfig(level=logging.INFO, stream=sys.stderr)
 
-    branch_core(OPTIONS.branch, OPTIONS.branch_point)
-    branch_everything(OPTIONS.branch, OPTIONS.branch_point)
+    if OPTIONS.core:
+        create_branch('core', OPTIONS.branch, OPTIONS.branch_point)
+    if OPTIONS.bundle:
+        branch_everything(OPTIONS.branch, OPTIONS.branch_point, OPTIONS.bundle)

-- 
To view, visit https://gerrit.wikimedia.org/r/403155
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I0e0cbfd99e0247d35ee3a3065bf3527313def059
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/tools/release
Gerrit-Branch: master
Gerrit-Owner: Chad <ch...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to