Make wait-for-SVN-server.py accept empty string as revision
authorossy@webkit.org <ossy@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 8 Nov 2012 07:29:06 +0000 (07:29 +0000)
committerossy@webkit.org <ossy@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 8 Nov 2012 07:29:06 +0000 (07:29 +0000)
https://bugs.webkit.org/show_bug.cgi?id=98014

Reviewed by Dirk Pranke.

* BuildSlaveSupport/wait-for-SVN-server.py:
(waitForSVNRevision):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@133854 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/BuildSlaveSupport/wait-for-SVN-server.py
Tools/ChangeLog

index da90ab2df5ae20e877457e512b356e116d27e3eb..f093024a10886f148ca1631c0041f7ecb985842c 100755 (executable)
@@ -44,11 +44,12 @@ def getLatestSVNRevision(SVNServer):
 
 
 def waitForSVNRevision(SVNServer, revision):
-    if not revision:
+    if not revision or not revision.isdigit():
         latestRevision = int(getLatestSVNRevision(SVNServer))
-        print "Latest SVN revision on %s is r%d. Don't wait, because revision argument is empty." % (SVNServer, latestRevision)
+        print "Latest SVN revision on %s is r%d. Don't wait, because revision argument isn't a valid SVN revision." % (SVNServer, latestRevision)
         return
 
+    revision = int(revision)
     while True:
         latestRevision = int(getLatestSVNRevision(SVNServer))
         if latestRevision < revision:
@@ -61,7 +62,7 @@ def waitForSVNRevision(SVNServer, revision):
 
 if __name__ == '__main__':
     parser = OptionParser()
-    parser.add_option("-r", "--revision", dest="revision", type="int", help="SVN revision number")
+    parser.add_option("-r", "--revision", dest="revision", help="SVN revision number")
     parser.add_option("-s", "--svn-server", dest="SVNServer", help="SVN server")
     options, args = parser.parse_args()
     waitForSVNRevision(options.SVNServer, options.revision)
index c304d05844f1d0ed479878362f3c041f2deeb6ea..c30511344c90b99157df2ff3902d0d51b424558d 100644 (file)
@@ -1,3 +1,13 @@
+2012-11-07  Csaba Osztrogon√°c  <ossy@webkit.org>
+
+        Make wait-for-SVN-server.py accept empty string as revision
+        https://bugs.webkit.org/show_bug.cgi?id=98014
+
+        Reviewed by Dirk Pranke.
+
+        * BuildSlaveSupport/wait-for-SVN-server.py:
+        (waitForSVNRevision):
+
 2012-11-07  Dirk Pranke  <dpranke@chromium.org>
 
         garden-o-matic should handle concurrent requests