Commit 76d0e6ac authored by Josip Sokcevic's avatar Josip Sokcevic Committed by LUCI CQ

[bot_update] Resolve revision only if patching

If bot_update is not patching, it shouldn't look into gitiles patch
information.

R=ehmaldonado@google.com

Bug: 1147625
Recipe-Nontrivial-Roll: build
Change-Id: I111f0dc1c36d588341c02930bf1cda1f5b8cc2e7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2644199Reviewed-by: 's avatarEdward Lesmes <ehmaldonado@chromium.org>
Commit-Queue: Josip Sokcevic <sokcevic@google.com>
parent d686b1d1
...@@ -240,8 +240,8 @@ class BotUpdateApi(recipe_api.RecipeApi): ...@@ -240,8 +240,8 @@ class BotUpdateApi(recipe_api.RecipeApi):
fixed_revision = self.m.gclient.resolve_revision(revision) fixed_revision = self.m.gclient.resolve_revision(revision)
if fixed_revision: if fixed_revision:
fixed_revisions[name] = fixed_revision fixed_revisions[name] = fixed_revision
if fixed_revision.upper() == 'HEAD': if fixed_revision.upper() == 'HEAD' and patch:
# Sync to correct destination ref if HEAD was specified. # Sync to correct destination ref
fixed_revision = self._destination_ref(cfg, name) fixed_revision = self._destination_ref(cfg, name)
# If we're syncing to a ref, we want to make sure it exists before # If we're syncing to a ref, we want to make sure it exists before
# trying to check it out. # trying to check it out.
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment