Check Exception object's argument length in trychange.py.

trychange.py

 #!/usr/bin/python
 # Copyright (c) 2009 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 a try job to the try server. It communicates to
 the try server by either writting to a svn repository or by directly connecting
 to the server by HTTP.
 """
 
 
@@ skipping 300 matching lines @@
   proxies = None
   if options.proxy:
     if options.proxy.lower() == 'none':
       # Effectively disable HTTP_PROXY or Internet settings proxy setup.
       proxies = {}
     else:
       proxies = {'http': options.proxy, 'https': options.proxy}
   try:
     connection = urllib.urlopen(url, urllib.urlencode(values), proxies=proxies)
   except IOError, e:
-    # TODO(thestig) this probably isn't quite right.
-    if values.get('bot') and e[2] == 'got a bad status line':
+    if (values.get('bot') and len(e.args) > 2 and
+        e.args[2] == 'got a bad status line'):
       raise NoTryServerAccess('%s is unaccessible. Bad --bot argument?' % url)
     else:
-      raise NoTryServerAccess('%s is unaccessible.' % url)
+      raise NoTryServerAccess('%s is unaccessible. Reason: %s' % (url,
+                                                                  str(e.args)))
   if not connection:
     raise NoTryServerAccess('%s is unaccessible.' % url)
   if connection.read() != 'OK':
     raise NoTryServerAccess('%s is unaccessible.' % url)
   return options.name
 
 
 def _SendChangeSVN(options):
   """Send a change to the try server by committing a diff file on a subversion
   server."""
@@ skipping 249 matching lines @@
     if patch_name == 'Unnamed':
       print "Note: use --name NAME to change the try's name."
   except (InvalidScript, NoTryServerAccess), e:
     if swallow_exception:
       return
     print e
 
 
 if __name__ == "__main__":
   TryChange(None, None, False)