Reviewers: jarin,

Message:
Committed patchset #1 (id:1) manually as 23664 (presubmit successful).

Description:
Add more output when roll scripts are retrying.

[email protected]

Committed: https://code.google.com/p/v8/source/detail?r=23664

Please review this at https://codereview.chromium.org/529343003/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files (+5, -4 lines):
  M tools/push-to-trunk/common_includes.py


Index: tools/push-to-trunk/common_includes.py
diff --git a/tools/push-to-trunk/common_includes.py b/tools/push-to-trunk/common_includes.py index 7770ce0762cffae84833a2499401d99724d29239..a92998596bc8b946c03e86ea283ae10cac92dc0e 100644
--- a/tools/push-to-trunk/common_includes.py
+++ b/tools/push-to-trunk/common_includes.py
@@ -318,13 +318,14 @@ class Step(GitRecipesMixin):
       got_exception = False
       try:
         result = cb()
-      except NoRetryException, e:
+      except NoRetryException as e:
         raise e
-      except Exception:
-        got_exception = True
+      except Exception as e:
+        got_exception = e
       if got_exception or retry_on(result):
         if not wait_plan:  # pragma: no cover
-          raise Exception("Retried too often. Giving up.")
+          raise Exception("Retried too often. Giving up. Reason: %s" %
+                          str(got_exception))
         wait_time = wait_plan.pop()
         print "Waiting for %f seconds." % wait_time
         self._side_effect_handler.Sleep(wait_time)


--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to