[NaCl SDK] Fix bug in "naclsdk update", updating recommended bundles.

native_client_sdk/src/build_tools/sdk_tools/command/update.py

 # 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.
 
 import copy
 import logging
 import os
 import sys
 import urlparse
 import urllib2
@@ skipping 194 matching lines @@
 def _GetRequestedBundleNamesFromArgs(remote_manifest, requested_bundles):
   requested_bundles = set(requested_bundles)
   if RECOMMENDED in requested_bundles:
     requested_bundles.discard(RECOMMENDED)
     requested_bundles |= set(_GetRecommendedBundleNames(remote_manifest))
 
   return requested_bundles
 
 
 def _GetRecommendedBundleNames(remote_manifest):
-  return [bundle.name for bundle in remote_manifest.GetBundles() if
-      bundle.recommended]
+  result = []
+  for bundle in remote_manifest.GetBundles():
+    if bundle.recommended == 'yes' and bundle.name != SDK_TOOLS:
+      result.append(bundle.name)
+  return result
 
 
 def _BundleNeedsUpdate(delegate, local_manifest, bundle):
   # Always update the bundle if the directory doesn't exist;
   # the user may have deleted it.
   if not delegate.BundleDirectoryExists(bundle.name):
     return True
 
   return local_manifest.BundleNeedsUpdate(bundle)
 
@@ skipping 48 matching lines @@
   return path.split('/')[-1]
 
 
 def _ValidateArchive(archive, actual_sha1, actual_size):
   if actual_sha1 != archive.GetChecksum():
     raise Error('SHA1 checksum mismatch on "%s".  Expected %s but got %s' % (
         archive.name, archive.GetChecksum(), actual_sha1))
   if actual_size != archive.size:
     raise Error('Size mismatch on "%s".  Expected %s but got %s bytes' % (
         archive.name, archive.size, actual_size))