From: Torsten Krah <krah.tm@gmail.com>
To: Jeff King <peff@peff.net>
Cc: "git@vger.kernel.org" <git@vger.kernel.org>
Subject: Re: Broken branch after git commit - tracked files in staging area can't be removed with restore --staged, or commit or stash
Date: Wed, 08 Jan 2020 13:42:14 +0100 [thread overview]
Message-ID: <654cc7a58cf6947f91478411dd6a3f7f3473bc67.camel@gmail.com> (raw)
In-Reply-To: <20200108104008.GA2207365@coredump.intra.peff.net>
Am Mittwoch, den 08.01.2020, 05:40 -0500 schrieb Jeff King:
> So there seem to be at least two bugs:
>
> - git-restore doesn't properly invalidate the cache-tree
>
> - the index-reading code is not careful enough about bogus cache-
> trees,
> and may segfault
>
> -Peff
--
Hi Peff,
thanks for confirming those bugs and taking a look at my report itself
even before I've put the small reproducer recipe.
I don't need to take any other actions or create tickets somewhere else
now to get those fixed, you're driving this from here, right?
Another question - how can I fix my broken original branch where this
bug did hit me which is in that "bogus" state now to get back to the
"original" commit made so I can remove my unwanted files with "git
reset HEAD <files>" instead of git-restore in the meantime (tried your
rm .git/index variant but after that I had files which were already in
the branch shown as unversioned after that, so that did not work very
well for me)?
kind regards
Torsten
next prev parent reply other threads:[~2020-01-08 12:42 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-01-07 12:55 Broken branch after git commit - tracked files in staging area can't be removed with restore --staged, or commit or stash Torsten Krah
2020-01-07 13:43 ` Torsten Krah
2020-01-07 15:28 ` Torsten Krah
2020-01-08 9:11 ` Jeff King
2020-01-08 10:02 ` Torsten Krah
2020-01-08 10:31 ` Torsten Krah
2020-01-08 10:40 ` Jeff King
2020-01-08 11:43 ` [PATCH] restore: invalidate cache-tree when removing entries with --staged Jeff King
2020-01-08 15:41 ` Junio C Hamano
2020-02-05 20:24 ` Dennis Kaarsemaker
2020-01-08 12:42 ` Torsten Krah [this message]
2020-01-09 9:39 ` Broken branch after git commit - tracked files in staging area can't be removed with restore --staged, or commit or stash Jeff King
2020-01-09 10:49 ` Torsten Krah
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=654cc7a58cf6947f91478411dd6a3f7f3473bc67.camel@gmail.com \
--to=krah.tm@gmail.com \
--cc=git@vger.kernel.org \
--cc=peff@peff.net \
/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).