From 3b217f537f8fc567981de075cffca6eefc5e0bf0 Mon Sep 17 00:00:00 2001 From: "maruel@chromium.org" Date: Mon, 1 Jun 2009 17:54:20 +0000 Subject: [PATCH] Remove the tree open check since it's now a presubmit script. Review URL: http://codereview.chromium.org/115996 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@17330 0039d316-1c4b-4281-b951-d872f2087c98 --- gcl.py | 17 +---------------- 1 file changed, 1 insertion(+), 16 deletions(-) diff --git a/gcl.py b/gcl.py index 3a4503d9c8..e281baaec4 100755 --- a/gcl.py +++ b/gcl.py @@ -187,15 +187,6 @@ def GetCodeReviewSetting(key): return CODEREVIEW_SETTINGS.get(key, "") -def IsTreeOpen(): - """Fetches the tree status and returns either True or False.""" - url = GetCodeReviewSetting('STATUS') - status = "" - if url: - status = urllib2.urlopen(url).read() - return status.find('0') == -1 - - def Warn(msg): ErrorExit(msg, exit=False) @@ -607,7 +598,7 @@ Basic commands: [--send_mail] [--no_try] [--no_presubmit] Uploads the changelist to the server for review. - gcl commit change_name [--no_presubmit] [--force] + gcl commit change_name [--no_presubmit] Commits the changelist to the repository. gcl lint change_name @@ -873,12 +864,6 @@ def Commit(change_info, args): else: args.remove("--no_presubmit") - no_tree_status_check = ("--force" in args or "-f" in args) - if not no_tree_status_check and not IsTreeOpen(): - print ("Error: The tree is closed. Try again later or use --force to force" - " the commit. May the --force be with you.") - return - # We face a problem with svn here: Let's say change 'bleh' modifies # svn:ignore on dir1\. but another unrelated change 'pouet' modifies # dir1\foo.cc. When the user `gcl commit bleh`, foo.cc is *also committed*.