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

Issue 1983963002: Revert "Gclient: Don't check if repository is clean if --force is passed in" (Closed)

Created:
4 years, 7 months ago by smut
Modified:
4 years, 7 months ago
Reviewers:
dnj, hinoka
CC:
chromium-reviews, dpranke+depot_tools_chromium.org, iannucci+depot_tools_chromium.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Revert "Gclient: Don't check if repository is clean if --force is passed in" This reverts commit 3f0dacf095df6f73a2c54a8a9fb35c1c441a2fe1. Reason for revert: Is causing staged changes to be reverted. BUG=607913 TBR=hinoka,dnj Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=300598

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -4 lines) Patch
M gclient_scm.py View 2 chunks +2 lines, -4 lines 0 comments Download

Messages

Total messages: 8 (3 generated)
smut
4 years, 7 months ago (2016-05-16 19:00:51 UTC) #2
smut
Committed patchset #1 (id:1) manually as 300598.
4 years, 7 months ago (2016-05-16 19:06:10 UTC) #5
hinoka
What is this breaking? And who's expecting staged changes to not be reverted if --force ...
4 years, 7 months ago (2016-05-16 19:40:37 UTC) #6
hinoka
Oh i see the bug On Mon, May 16, 2016 at 12:40 PM, <hinoka@chromium.org> wrote: ...
4 years, 7 months ago (2016-05-16 19:41:58 UTC) #7
smut
4 years, 7 months ago (2016-05-16 19:45:08 UTC) #8
Message was sent while issue was closed.
On 2016/05/16 19:40:37, hinoka wrote:
> What is this breaking? And who's expecting staged changes to not be reverted
if
> --force is passed in?

See #2 in the bug. Note that you were reverting staged changes even when --force
was not passed in.

Powered by Google App Engine
This is Rietveld 408576698