Sfoglia il codice sorgente

Change readme updating to be on by default for roll_dep.py

Change-Id: Id6d5a365e55ad0a79e512af24e72f141f8b9a60a
Bug: b/390067679
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/6521682
Reviewed-by: Rachael Newitt <renewitt@google.com>
Reviewed-by: Andy Perelson <ajp@google.com>
Commit-Queue: Jordan Brown <rop@google.com>
Jordan Brown 3 mesi fa
parent
commit
977c37458f
2 ha cambiato i file con 5 aggiunte e 5 eliminazioni
  1. 4 4
      roll_dep.py
  2. 1 1
      tests/roll_dep_test.py

+ 4 - 4
roll_dep.py

@@ -232,7 +232,7 @@ def finalize(args, commit_msg, current_dir, rolls):
         check_call(['git', 'checkout', '--quiet', roll_to], cwd=full_dir)
 
         # Attempt to update README.chromium.
-        if args.update_readme:
+        if not args.no_update_readme:
             update_readme_chromium(dependency, roll_to, current_dir)
 
         # This adds the submodule revision update to the commit.
@@ -349,9 +349,9 @@ def main():
                         default=[],
                         help='Regex(es) for dependency in DEPS file')
     parser.add_argument('dep_path', nargs='+', help='Path(s) to dependency')
-    parser.add_argument('--update-readme',
+    parser.add_argument('--no-update-readme',
                        action='store_true',
-                       help='Update Revision in README.chromium if it exists')
+                       help='Do not try to update Revision in README.chromium')
     args = parser.parse_args()
 
     if len(args.dep_path) > 1:
@@ -380,7 +380,7 @@ def main():
         d.replace('\\', '/').rstrip('/') for d in args.dep_path)
     cmdline = 'roll-dep ' + ' '.join(dependencies) + ''.join(' --key ' + k
                                                              for k in args.key)
-    if args.update_readme:
+    if not args.no_update_readme:
         cmdline += ' --update-readme'
     try:
         if not args.ignore_dirty_tree and not is_pristine(current_dir):

+ 1 - 1
tests/roll_dep_test.py

@@ -217,7 +217,7 @@ class RollDepTest(fake_repos.FakeReposTestBase):
         if not self.enabled:
             return
         stdout, stderr, returncode = self.call(
-                [ROLL_DEP, '--update-readme']+self.all_repos
+                [ROLL_DEP]+self.all_repos
         )
         latest_revision = self.githash('repo_2', 3)