From: "Björn Steinbrink" <B.Steinbrink@gmx.de>
To: Junio C Hamano <gitster@pobox.com>
Cc: "Michael P. Soulier" <msoulier@digitaltorque.ca>, git@vger.kernel.org
Subject: Re: git confused by rename
Date: Fri, 10 Oct 2008 18:19:22 +0200 [thread overview]
Message-ID: <20081010161922.GH3671@atjola.homenet> (raw)
In-Reply-To: <7v3aj4momt.fsf@gitster.siamese.dyndns.org>
On 2008.10.10 08:54:50 -0700, Junio C Hamano wrote:
> Junio C Hamano <gitster@pobox.com> writes:
> > Because git does not care how you created B (IOW, it does not matter
> > if B was typed from scratch with copying and pasting, or created by
> > copying and editing), the "renamed: A -> B" entry itself is not
> > surprising nor look like a bug at all,...
>
> The above observation of mine is correct, but I forgot that "git status"
> (and the comment in the commit template from "git commit") is generated
> internally with "diff-index -B -M". So if
>
> (0) had A but not B in the HEAD commit;
> (1) you created B that is very similar to the original A; and
> (2) you modified A beyond recognition;
>
> then git will say "A was modified and B was created by renaming A".
>
> > ... but its presense at the same time as "modified: A" does feel very
> > fishy.
>
> So this is not fishy anymore.
>
> This however makes me wonder if "diff-index -B -M" should say B is copied
> (instead of being renamed) from A and A is modified in such a case. I do
> not think we would want to make such a change without thinking things,
> through.
Oh, heh, my patch doesn't help in that case. It's still modify + rename
with it, of course, as -B still breaks the stuff apart. Me being
confused about -M reporting copies in some cases remains though, I had
really expected that only -C would detect copies. Of course, this
confusion is purely from a user's point of view, I have zero knowledge
about the involved code...
Björn
next prev parent reply other threads:[~2008-10-10 16:20 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-10-10 14:41 git confused by rename Michael P. Soulier
2008-10-10 15:44 ` Junio C Hamano
2008-10-10 15:54 ` Junio C Hamano
2008-10-10 16:19 ` Björn Steinbrink [this message]
2008-10-10 22:08 ` Michael P. Soulier
2008-10-10 16:08 ` [PATCH] git-status: Detect copies as well as renames Björn Steinbrink
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=20081010161922.GH3671@atjola.homenet \
--to=b.steinbrink@gmx.de \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=msoulier@digitaltorque.ca \
/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