[NaCl SDK] Set "recommended" tag to "yes" automatically for stable pepper bundle.

native_client_sdk/src/build_tools/update_nacl_manifest.py

Index: native_client_sdk/src/build_tools/update_nacl_manifest.py
diff --git a/native_client_sdk/src/build_tools/update_nacl_manifest.py b/native_client_sdk/src/build_tools/update_nacl_manifest.py
index 9f7af059498ba94cd17af97f1665c2461e868231..8147077e8009b2a03e434179bffcddc798375dd6 100755
--- a/native_client_sdk/src/build_tools/update_nacl_manifest.py
+++ b/native_client_sdk/src/build_tools/update_nacl_manifest.py
@@ -467,7 +467,6 @@ class Updater(object):
     for bundle_name, version, channel, archives in self.versions_to_update:
       self.delegate.Print('Updating %s to %s...' % (bundle_name, version))
       bundle = manifest.GetBundle(bundle_name)
-      bundle_recommended = bundle.recommended
       for archive in archives:
         platform_bundle = self._GetPlatformArchiveBundle(archive)
         # Normally the manifest snippet's bundle name matches our bundle name.
@@ -476,7 +475,11 @@ class Updater(object):
         platform_bundle.name = bundle_name
         bundle.MergeWithBundle(platform_bundle)
       bundle.stability = channel
-      bundle.recommended = bundle_recommended
+      # We always recommend the stable version.
+      if channel == 'stable':
+        bundle.recommended = 'yes'
+      else:
+        bundle.recommended = 'no'

[noelallen1, 2012/07/09 21:58:39]

Does the no mean we don't recommend anything but?  I suppose that's reasonable. 
Why recommend post stable...

[binji, 2012/07/09 22:06:29]

That's what I figured... I've pretty much been updating the
naclsdk_manifest2.json file to make stable recommended and everything else not.
It's a pain now, because I have to ssh into the master and "svn up" for any
changes like that.

Personally, I think recommended should go away, but I'd have to change the
manifest format to do it... :-/

       manifest.MergeBundle(bundle)
     self._UploadManifest(manifest)
     self.delegate.Print('Done.')