Teach bot_update to support patching deps in solutions other than the patch_root

scripts/slave/bot_update.py

 #!/usr/bin/env python
 # Copyright 2014 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.
 
 # TODO(hinoka): Use logging.
 
 import cStringIO
 import codecs
 import collections
@@ skipping 1239 matching lines @@
   # Get a checkout of each solution, without DEPS or hooks.
   # Calling git directly because there is no way to run Gclient without
   # invoking DEPS.
   print 'Fetching Git checkout'
   git_ref = git_checkout(solutions, revisions, shallow, with_branch_heads)
 
   patches = None
   if patch_url:
     patches = get_svn_patch(patch_url)
 
   for solution in solutions:

[kjellander_chromium, 2014/09/08 14:14:23]

Please add a blacklist to be used when patching the rest below. Something like:

# A blacklist used to exclude the DEPS patch entries when applying the other
# parts of the patch further below.
blacklist = []

[agable, 2014/09/08 18:23:24]

Acknowledged.

-    # At first, only patch top-level DEPS.
-    if patch_root == solution['name']:
+    if (patch_root == solution['name'] or
+        solution['name'].startswith(patch_root + '/')):
+      relative_root = solution['name'][len(patch_root) + 1:]
+      target = '/'.join(relative_root, 'DEPS').lstrip('/')

[kjellander_chromium, 2014/09/08 14:14:23]

This needs to be
target = '/'.join([relative_root, 'DEPS']).lstrip('/')

[agable, 2014/09/08 18:23:24]

Done.

       if patches:

[kjellander_chromium, 2014/09/08 14:14:23]

Add
blacklist.append(target)
somewhere around here.

[agable, 2014/09/08 18:23:24]

Acknowledged.

-        apply_svn_patch(patch_root, patches, whitelist=['DEPS'])
+        apply_svn_patch(patch_root, patches, whitelist=[target])
       elif issue:
         apply_rietveld_issue(issue, patchset, patch_root, rietveld_server,
-                            revision_mapping, git_ref, whitelist=['DEPS'])
-      break
+                            revision_mapping, git_ref, whitelist=[target])
 
   if buildspec:
     buildspecs2git(first_sln, buildspec)
   else:
     # Run deps2git if there is a DEPS change after the last .DEPS.git commit.
     for solution in solutions:
       ensure_deps2git(solution, shallow)
 
   # Ensure our build/ directory is set up with the correct .gclient file.
   gclient_configure(solutions, target_os, target_os_only)
@@ skipping 12 matching lines @@
     # TODO(hinoka): Remove this when the official builders run their own
     #               runhooks step.
     gclient_runhooks(gyp_env)
 
   # Finally, ensure that all DEPS are pinned to the correct revision.
   dir_names = [sln['name'] for sln in solutions]
   ensure_deps_revisions(gclient_output.get('solutions', {}),
                         dir_names, revisions)
   # Apply the rest of the patch here (sans DEPS)
   if patches:
     apply_svn_patch(patch_root, patches, blacklist=['DEPS'])

[kjellander_chromium, 2014/09/08 14:14:23]

blacklist=blacklist
here and below.

[agable, 2014/09/08 18:23:24]

Done.

   elif issue:
     apply_rietveld_issue(issue, patchset, patch_root, rietveld_server,
                          revision_mapping, git_ref, blacklist=['DEPS'])
 
   # Reset the deps_file point in the solutions so that hooks get run properly.
   for sln in solutions:
     sln['deps_file'] = sln.get('deps_file', 'DEPS').replace('.DEPS.git', 'DEPS')
   gclient_configure(solutions, target_os, target_os_only)
 
   return gclient_output
@@ skipping 324 matching lines @@
   except Exception:
     # Unexpected failure.
     emit_flag(options.flag_file)
     raise
   else:
     emit_flag(options.flag_file)
 
 
 if __name__ == '__main__':
   sys.exit(main())