From: Johannes Schindelin <Johannes.Schindelin@gmx.de>
To: Junio C Hamano <gitster@pobox.com>
Cc: "Kristian Høgsberg" <krh@redhat.com>, git@vger.kernel.org
Subject: Re: *[PATCH] builtin-commit: Refresh cache after adding files.
Date: Sat, 10 Nov 2007 02:02:31 +0000 (GMT) [thread overview]
Message-ID: <Pine.LNX.4.64.0711100201310.4362@racer.site> (raw)
In-Reply-To: <7v640ari76.fsf@gitster.siamese.dyndns.org>
Hi,
On Fri, 9 Nov 2007, Junio C Hamano wrote:
> Junio C Hamano <gitster@pobox.com> writes:
>
> > This discussion exposes a problem with add_files_to_cache()
> > function.
> > ...
> > And add_file_to_cache(), which calls add_file_to_index() on
> > the_index, does call the fill_stat_cache_info() to sync the stat
> > information by itself, as it should be. I am still puzzled with
> > this.
>
> Blech. I think it is 0781b8a9b2fe760fc4ed519a3a26e4b9bd6ccffe
> (add_file_to_index: skip rehashing if the cached stat already matches)
> that broke "git add". I get the same problem not just with "git add -u"
> but also with an explicit "git add Makefile".
But I think that we still need to refresh the index. It outputs which
files names were new, and which ones were removed. Otherwise git commit
would be eerily silent on those.
Ciao,
Dscho
next prev parent reply other threads:[~2007-11-10 2:02 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-11-09 16:40 [PATCH] builtin-commit: Refresh cache after adding files Kristian Høgsberg
2007-11-09 17:05 ` Johannes Schindelin
2007-11-09 17:13 ` Kristian Høgsberg
2007-11-09 17:24 ` Johannes Schindelin
2007-11-09 17:38 ` Kristian Høgsberg
2007-11-09 18:24 ` Junio C Hamano
2007-11-09 18:39 ` Kristian Høgsberg
2007-11-10 1:40 ` *[PATCH] " Junio C Hamano
2007-11-10 2:02 ` Johannes Schindelin [this message]
2007-11-10 2:26 ` Junio C Hamano
2007-11-10 2:22 ` [PATCH] git-add: make the entry stat-clean after re-adding the same contents Junio C Hamano
2007-11-10 2:43 ` Linus Torvalds
2007-11-10 9:01 ` [PATCH 1/2] ce_match_stat, run_diff_files: use symbolic constants for readability Junio C Hamano
2007-11-10 9:02 ` [PATCH 2/2] git-add: make the entry stat-clean after re-adding the same contents Junio C Hamano
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=Pine.LNX.4.64.0711100201310.4362@racer.site \
--to=johannes.schindelin@gmx.de \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=krh@redhat.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).