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

Issue 199993005: Fix branch for resetting change log in push-to-trunk. (Closed)

Created:
6 years, 9 months ago by Michael Achenbach
Modified:
6 years, 9 months ago
Reviewers:
Jakob Kummerow
CC:
v8-dev, Jarin
Visibility:
Public.

Description

Fix branch for resetting change log in push-to-trunk. The patch is applied to ChangeLog with the "--index" option, which already updates the index of the file. For resetting the ChangeLog to the trunk revision, the trunk branch needs to be stated explicitly in the checkout, otherwise it checks out the already patched HEAD. TEST=tools/push-to-trunk/script_test.py R=jkummerow@chromium.org Committed: https://code.google.com/p/v8/source/detail?r=19996

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -4 lines) Patch
M tools/push-to-trunk/git_recipes.py View 1 chunk +3 lines, -2 lines 0 comments Download
M tools/push-to-trunk/push_to_trunk.py View 1 chunk +1 line, -1 line 0 comments Download
M tools/push-to-trunk/test_scripts.py View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
Michael Achenbach
PTAL
6 years, 9 months ago (2014-03-17 09:51:32 UTC) #1
Jakob Kummerow
lgtm
6 years, 9 months ago (2014-03-17 13:20:43 UTC) #2
Michael Achenbach
6 years, 9 months ago (2014-03-17 13:52:47 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as r19996 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698