From: Jonathan Nieder <jrnieder@gmail.com>
To: Jakub Narebski <jnareb@gmail.com>
Cc: Mike <xandrani@gmail.com>,
git@vger.kernel.org, "Ben Walton" <bwalton@artsci.utoronto.ca>,
"Ramkumar Ramachandra" <artagnon@gmail.com>,
"Nguyễn Thái Ngọc Duy" <pclouds@gmail.com>
Subject: Re: Undo last commit?
Date: Sat, 18 Jun 2011 19:37:18 -0500 [thread overview]
Message-ID: <20110619003718.GA5628@elie> (raw)
In-Reply-To: <m31uyrutx7.fsf@localhost.localdomain>
Hi,
Jakub Narebski wrote:
> Mike <xandrani@gmail.com> writes:
>> % git reset --hard HEAD~1
>
> Errr... here you screwed up. This reset state of you working area to
> the state at last commit, removing all your changes to tracked files.
Or rather, here we screwed up. Jakub and others gave some useful
advice about how to recover, so let's consider how the UI or
documentation could be improved to prevent it from happening again.
* In this example if I understand correctly then the index contained
some useful information, perhaps about a larger commit intended for
later. To preserve that, you could have used
git reset --soft HEAD~1
which would _just_ undo the effect of "git commit", leaving the index
and worktree alone.
* Another situation that comes up from time to time is making a change
that just turned out to be a bad idea. After commiting it, you might
want to discard the erroneous change, like so:
git reset --keep HEAD~1
The "--keep" option uses some safeguards to make sure that only the
committed change gets discarded, instead of clobbering local changes
at the same time.
* In the early days of git, the "--keep" option did not exist. So a lot
of old documentation recommends to do
git reset --hard HEAD~1
which is the same if you don't have any local changes.
It would be useful to fix such documentation by adding a few words
about local changes. Recently Duy wrote a patch to improve "reset -h"
output in that vein, but discussion drifted off:
http://thread.gmane.org/gmane.comp.version-control.git/170266
I also sent a couple of documentation patches and then dropped the
ball:
http://thread.gmane.org/gmane.comp.version-control.git/165358
http://thread.gmane.org/gmane.comp.version-control.git/160319
If someone wants to pick any of these up and run with it, I wouldn't
mind (hey, I'd be happy).
Thanks for a useful example.
Jonathan
next prev parent reply other threads:[~2011-06-19 0:41 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-06-18 13:15 Undo last commit? Mike
2011-06-18 13:43 ` Ben Walton
2011-06-18 13:48 ` Ramkumar Ramachandra
2011-06-18 13:54 ` Jakub Narebski
2011-06-19 0:37 ` Jonathan Nieder [this message]
2011-06-19 10:37 ` Jakub Narebski
2011-06-20 12:08 ` Massimo Manca
2011-06-28 13:57 ` Holger Hellmuth
2011-06-28 14:31 ` Mike
2011-06-30 4:50 ` Ramkumar Ramachandra
2011-06-30 18:38 ` Jonathan Nieder
2011-06-30 19:48 ` Mike
2011-06-30 17:29 ` Junio C Hamano
2012-02-11 18:19 ` Neal Kreitzinger
[not found] ` <CAHK-92oMc62O0S8Bxt6+uxobE+kg5wOeRDoOsHWvvenXaXmZGQ@mail.gmail.com>
2012-02-11 22:07 ` Jakub Narebski
2012-02-11 22:29 ` Jonathan Nieder
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20110619003718.GA5628@elie \
--to=jrnieder@gmail.com \
--cc=artagnon@gmail.com \
--cc=bwalton@artsci.utoronto.ca \
--cc=git@vger.kernel.org \
--cc=jnareb@gmail.com \
--cc=pclouds@gmail.com \
--cc=xandrani@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).