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

Issue 166273024: More fixes for hardcoded git directory paths. (Closed)

Created:
6 years, 10 months ago by Michael Moss
Modified:
6 years, 10 months ago
Reviewers:
iannucci, M-A Ruel
CC:
chromium-reviews, Dirk Pranke, cmp-cc_chromium.org, M-A Ruel, iannucci+depot_tools_chromium.org, ilevy-cc_chromium.org
Visibility:
Public.

Description

More fixes for hardcoded git directory paths. The git path is based on the git version, so it might change whenever git is updated. Run git.bat instead, since that's updated with new git installs and should always point to the right path. The mingw logic is based on similar handling of python.bat in git-cl. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=252584

Patch Set 1 #

Patch Set 2 : Use eval to execute GIT var #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+19 lines, -16 lines) Patch
M update_depot_tools View 1 7 chunks +19 lines, -16 lines 1 comment Download

Messages

Total messages: 8 (0 generated)
Michael Moss
6 years, 10 months ago (2014-02-20 20:36:41 UTC) #1
M-A Ruel
somewhat rubberstamp lgtm
6 years, 10 months ago (2014-02-21 02:40:24 UTC) #2
Michael Moss
On 2014/02/21 02:40:24, M-A Ruel wrote: > somewhat rubberstamp lgtm Is there anyone who's actually ...
6 years, 10 months ago (2014-02-21 04:39:13 UTC) #3
iannucci
/me has no idea :( https://codereview.chromium.org/166273024/diff/30001/update_depot_tools File update_depot_tools (right): https://codereview.chromium.org/166273024/diff/30001/update_depot_tools#newcode20 update_depot_tools:20: base_dir="${0%\\*}" TIL: bash variable ...
6 years, 10 months ago (2014-02-21 08:25:07 UTC) #4
Michael Moss
On 2014/02/21 08:25:07, iannucci wrote: > /me has no idea :( > > https://codereview.chromium.org/166273024/diff/30001/update_depot_tools > ...
6 years, 10 months ago (2014-02-21 17:18:57 UTC) #5
Michael Moss
The CQ bit was checked by mmoss@chromium.org
6 years, 10 months ago (2014-02-21 17:19:13 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/mmoss@chromium.org/166273024/30001
6 years, 10 months ago (2014-02-21 17:19:19 UTC) #7
commit-bot: I haz the power
6 years, 10 months ago (2014-02-21 17:21:00 UTC) #8
Message was sent while issue was closed.
Change committed as 252584

Powered by Google App Engine
This is Rietveld 408576698