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

Issue 802513003: Use grep -e --force instead of grep "--force" since apparently the latter doesn't work properly in … (Closed)

Created:
6 years ago by Peter Kasting
Modified:
6 years ago
Reviewers:
agable
CC:
chromium-reviews, cmp-cc_chromium.org, Dirk Pranke, iannucci+depot_tools_chromium.org
Target Ref:
refs/heads/master
Project:
tools
Visibility:
Public.

Description

Use grep -e --force instead of grep "--force" since apparently the latter doesn't work properly in all cases. BUG=441072 TEST=No errors from gclient :/ Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=293380

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M update_depot_tools View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 6 (2 generated)
Peter Kasting
6 years ago (2014-12-11 21:11:50 UTC) #2
agable
lgtm
6 years ago (2014-12-12 19:20:40 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/802513003/1
6 years ago (2014-12-12 19:22:13 UTC) #5
commit-bot: I haz the power
6 years ago (2014-12-12 19:26:52 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=293380

Powered by Google App Engine
This is Rietveld 408576698