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

Issue 18083033: Get rid of place-holder for commit-msg hook. (Closed)

Created:
7 years, 5 months ago by szager1
Modified:
7 years, 5 months ago
CC:
chromium-reviews, Dirk Pranke, cmp-cc_chromium.org, M-A Ruel, iannucci+depot_tools_chromium.org
Visibility:
Public.

Description

Get rid of place-holder for commit-msg hook. This caused problems for repo-based projects that need to install the standard Change-Id commit-msg hook. With the commit-msg hook gone, we can also reinstate the --template argument to 'git clone'. BUG=235215 R=iannucci@chromium.org, ilevy@chromium.org, maruel@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=211464

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -5 lines) Patch
M gclient_scm.py View 1 1 chunk +4 lines, -1 line 0 comments Download
D git-templates/hooks/commit-msg View 1 chunk +0 lines, -4 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
szager1
7 years, 5 months ago (2013-07-03 22:04:01 UTC) #1
iannucci
code lgtm, though I'm not very familiar with the requirements of repo
7 years, 5 months ago (2013-07-03 22:07:21 UTC) #2
M-A Ruel
rubberstamp lgtm
7 years, 5 months ago (2013-07-08 18:02:34 UTC) #3
Isaac (away)
lgtm, should be OK. Note however this didn't affect just repo projects, but any project ...
7 years, 5 months ago (2013-07-08 18:41:47 UTC) #4
szager1
7 years, 5 months ago (2013-07-12 21:57:35 UTC) #5
Message was sent while issue was closed.
Committed patchset #2 manually as r211464.

Powered by Google App Engine
This is Rietveld 408576698