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

Issue 1777043002: Don't use 'git remote get-url', which is a new-ish feature. (Closed)

Created:
4 years, 9 months ago by szager1
Modified:
4 years, 9 months ago
Reviewers:
Dirk Pranke, agable, wkorman
CC:
chromium-reviews, dpranke+depot_tools_chromium.org, iannucci+depot_tools_chromium.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Don't use 'git remote get-url', which is a new-ish feature. Apparently most Mac's have git 2.6.2, which doesn't have this feature. This only affects end developers, so there's no urgency to push this out to the bots. TBR=agable@chromium.org,wkorman@chromium.org BUG= NOTRY=true Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=299185

Patch Set 1 #

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

Messages

Total messages: 19 (8 generated)
szager1
4 years, 9 months ago (2016-03-09 20:15:07 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1777043002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1777043002/1
4 years, 9 months ago (2016-03-09 20:16:36 UTC) #3
commit-bot: I haz the power
No L-G-T-M from a valid reviewer yet. CQ run can only be started by full ...
4 years, 9 months ago (2016-03-09 20:16:38 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1777043002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1777043002/1
4 years, 9 months ago (2016-03-09 20:17:26 UTC) #8
commit-bot: I haz the power
No L-G-T-M from a valid reviewer yet. CQ run can only be started by full ...
4 years, 9 months ago (2016-03-09 20:17:27 UTC) #10
szager1
lgtm
4 years, 9 months ago (2016-03-09 20:17:45 UTC) #11
szager1
On 2016/03/09 20:17:45, szager1 wrote: > lgtm Oh, hmm, I guess I'm not in project-infra-committers. ...
4 years, 9 months ago (2016-03-09 20:18:31 UTC) #12
Dirk Pranke
lgtm
4 years, 9 months ago (2016-03-09 20:21:14 UTC) #14
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1777043002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1777043002/1
4 years, 9 months ago (2016-03-09 20:25:26 UTC) #16
commit-bot: I haz the power
Committed patchset #1 (id:1) as http://src.chromium.org/viewvc/chrome?view=rev&revision=299185
4 years, 9 months ago (2016-03-09 20:27:59 UTC) #18
wkorman
4 years, 9 months ago (2016-03-09 20:33:58 UTC) #19
Message was sent while issue was closed.
lgtm

Powered by Google App Engine
This is Rietveld 408576698