Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(179)

Issue 2326473003: Revert of Add a DEPS gclient hook to prune corrupt mockito remote. (Closed)

Created:
4 years, 3 months ago by kjellander_webrtc
Modified:
4 years, 3 months ago
Reviewers:
ehmaldonado_webrtc
CC:
webrtc-reviews_webrtc.org, tterriberry_mozilla.com, qiang.lu, niklas.enbom, yujie_mao (webrtc), peah-webrtc
Target Ref:
refs/pending/heads/master
Project:
webrtc
Visibility:
Public.

Description

Revert of Add a DEPS gclient hook to prune corrupt mockito remote. (patchset #1 id:1 of https://codereview.webrtc.org/2326523002/ ) Reason for revert: Unfortunately this didn't help, so I'm reverting this to clean things up for now. Original issue's description: > Add a DEPS gclient hook to prune corrupt mockito remote. > > A third_party remote for third_party/mockito/src is corrupt. Run > `git remote prune origin` in the checkout directory before syncing > Chromium. > > Thanks to slan@ for providing this fix. > > TBR=kjellander@webrtc.org > BUG=chromium:644722 > NOTRY=True > > Committed: https://crrev.com/3115b06957d5b8cba5d45cebc301795e94a926ec > Cr-Commit-Position: refs/heads/master@{#14138} TBR=ehmaldonado@webrtc.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:644722 Committed: https://crrev.com/432950cab292beb8efb88e37024088e9d7297504 Cr-Commit-Position: refs/heads/master@{#14143}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -45 lines) Patch
M DEPS View 1 chunk +0 lines, -11 lines 0 comments Download
M sync_chromium.py View 1 chunk +1 line, -1 line 0 comments Download
D webrtc/build/prune_third_party_mockito_origin.py View 1 chunk +0 lines, -33 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
kjellander_webrtc
Created Revert of Add a DEPS gclient hook to prune corrupt mockito remote.
4 years, 3 months ago (2016-09-08 17:55:30 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/2326473003/1
4 years, 3 months ago (2016-09-08 17:55:38 UTC) #3
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 3 months ago (2016-09-08 17:55:45 UTC) #4
commit-bot: I haz the power
4 years, 3 months ago (2016-09-08 17:55:54 UTC) #6
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/432950cab292beb8efb88e37024088e9d7297504
Cr-Commit-Position: refs/heads/master@{#14143}

Powered by Google App Engine
This is Rietveld 408576698