NaCl SDK: Fixed getos.py crashing due to Git migration.

native_client_sdk/src/tools/getos.py

 #!/usr/bin/env python
 # Copyright (c) 2012 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.
 
 """Determine OS and various other system properties.
 
 Determine the name of the platform used and other system properties such as
 the location of Chrome.  This is used, for example, to determine the correct
 Toolchain to invoke.
@@ skipping 62 matching lines @@
       name, value = line.split(':', 1)
       if name == "Version":
         version = value.strip()
       if name == "Chrome Revision":
         revision = value.strip()
 
   if revision == None or version == None:
     raise Error("error parsing SDK README: %s" % readme)
 
   try:
-    revision = int(revision)
     version = int(version)
   except ValueError:
     raise Error("error parsing SDK README: %s" % readme)
 
   return (version, revision)
 
 
 def GetSystemArch(platform):
   if platform == 'win':
     if UseWin64():
@@ skipping 132 matching lines @@
     out = GetNaClArch(platform)
   elif options.chrome:
     out = GetChromePath(platform)
   elif options.sdk_version:
     out = GetSDKVersion()[0]
   elif options.sdk_revision:
     out = GetSDKVersion()[1]
   elif options.check_version:
     required_version = ParseVersion(options.check_version)
     version = GetSDKVersion()
-    if version < required_version:
-      raise Error("SDK version too old (current: %s.%s, required: %s.%s)"
-             % (version[0], version[1],
-                required_version[0], required_version[1]))
+    # We currently ignore the revision and just check the major version number.
+    # Currently, version[1] is just a Git hash, which cannot be compared.
+    # TODO(mgiuca): Compare the minor revision numbers (which should be
+    # Cr-Commit-Position values), when http://crbug.com/406783 is fixed.
+    # Then Cr-Commit-Position should be available: see http://crbug.com/406993.
+    if version[0] < required_version[0]:
+      raise Error("SDK version too old (current: %s, required: %s)"
+             % (version[0], required_version[0]))
     out = None
 
   if out:
     print out
   return 0
 
 
 if __name__ == '__main__':
   try:
     sys.exit(main(sys.argv[1:]))
   except Error as e:
     sys.stderr.write(str(e) + '\n')
     sys.exit(1)