Tiny updates to trychange_git.

trychange_git.py

 #!/usr/bin/env python
 # Copyright (c) 2013 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Client-side script to send local git changes to a tryserver.
 
-It pushes the local feature branch to a private ref on googlesource
+It pushes the local feature branch to a private try-ref on the central repo
 and posts a description of the job to an appengine instance, where it will get
 picked up by the buildbot tryserver itself.
 """
 
 from __future__ import print_function
 
 import json
 import os
 import subprocess
 import sys
@@ skipping 43 matching lines @@
         continue
       k, v = line.split(':', 1)
       settings[k.strip()] = v.strip()
   return settings
 
 
 def PushBranch():
   """Pushes the current local branch to a ref in the try repo.
 
   The try repo is either the remote called 'try', or 'origin' otherwise.
-  The ref is '/refs/try/<username>/<local branch>-<short hash>.
+  The ref is '/refs/tryjobs/<username>/<local branch>-<short hash>.
 
   Returns the ref to which the local branch was pushed."""
   username = RunGit('config', '--get', 'user.email').split('@', 1)[0]
   branch = RunGit('symbolic-ref', '--short', '-q', 'HEAD')
   commit = RunGit('rev-parse', branch)[:8]

[iannucci, 2013/09/21 00:49:42]

I would use the whole hash, I don't think the shortname is buying us much.

[agable, 2013/09/21 00:54:51]

I'll think on it some more. There's probably a better way to do this
all-together where the ref is totally human-readable and the bot uses a raw
hash.

   remotes = RunGit('remote').splitlines()
   if not all((username, branch, commit, remotes)):
     DieWithError('Unable to get necessary git configuration.')
 
   remote = 'try' if 'try' in remotes else 'origin'
-  ref = 'refs/try/%s/%s-%s' % (username, branch, commit)
+  ref = 'refs/tryjobs/%s/%s-%s' % (username, branch, commit)
 
   RunGit('push', remote, '%s:%s' % (branch, ref))
   return ref
 
 
 def MakeJob(project, jobname, ref):
   """Creates a job description blob."""
   email = RunGit('config', '--get', 'user.email')
   repository = RunGit('config', '--get', 'remote.origin.url')
   job = {
@@ skipping 21 matching lines @@
     conn = urllib.urlopen(url, data)
   except IOError as e:
     DieWithError(e)
   response = conn.getcode()
   if response != 200:
     DieWithError('Failed to POST. Got: %d' % response)
 
 
 def Main(_argv):
   """Main entry point."""
+  # Sanity check.
   EnsureInGitRepo()
 
+  # Get some settings.
   settings = GetCodeReviewSettings()
   server = settings.get('TRYSERVER_HTTP_HOST')
   project = settings.get('TRYSERVER_PROJECT')
   jobnames = settings.get('TRYSERVER_JOB_NAME')
   if not all((server, project, jobnames)):
     DieWithError('Missing configuration in codereview.settings.')
 
+  # Do the heavy lifting.
   ref = PushBranch()
   for jobname in jobnames.split(','):
     job = MakeJob(project, jobname, ref)
     PostJob(server, project, job)
 
 
 if __name__ == '__main__':
   sys.exit(Main(sys.argv))