From: Linus Torvalds <torvalds@osdl.org>
To: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Cc: git@vger.kernel.org
Subject: Re: path limiting broken
Date: Sun, 16 Apr 2006 11:27:15 -0700 (PDT) [thread overview]
Message-ID: <Pine.LNX.4.64.0604161117360.3701@g5.osdl.org> (raw)
In-Reply-To: <Pine.LNX.4.63.0604162006050.19560@wbgn013.biozentrum.uni-wuerzburg.de>
On Sun, 16 Apr 2006, Johannes Schindelin wrote:
>
> There's another reason it does not show it. ATM, you have to add "-p" to
> the command line, or "-c" will not show *any* patch, let alone a combined
> one.
Oh, it will show the "raw" patch, which is often very useful. I've grown
quite fond of it, because it shows what happened on a bigger level,
without showing the details within a file ("--name-status" makes it more
readable, but I'm too lazy to type it, so I often just look at the raw
git patch).
> Thanks for all your help, but in this case it was not irrelevant. Because
> I *had* the function in my working copy. And I had changed it. So I had to
> find out where to move the change.
Right, but it was irrelevant as far as "top-of-head" was concerned (which
is all that "git log" shows you - it doesn't care about your working
tree).
The fact that it _had_ been relevant in the state you used to be at is not
something "git log" and friends know or care about.
Now, I'm not disputing that we might want to make it easier to see what
_had_ been relevant at some earlier time. But you'd have to specify that
earlier time somehow. I assume you had tried to do a "git rebase", and the
problem with that is that git rebase really doesn't help you at all when
things go wrong, exactly because "rebase" - by design - screws up the
history and loses that information for you.
If your problem state had been as a result of a "git merge", you'd
actually have had much better tools available to you, exactly because
merge doesn't screw up history, so you've got both sides of the merge you
can look at (HEAD and MERGE_HEAD, and "git diff --ours" and "--theirs").
That said, even "rebase" will help somewhat. You've got "ORIG_HEAD" to
use, and that should help at least a _bit_. In particular, you can do
gitk ORIG_HEAD..
to see all the changes you rebased across. But right now you'd have to
fall back on the "-m -p" thing if you wanted to see it all..
Linus
next prev parent reply other threads:[~2006-04-16 18:27 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-04-16 12:26 path limiting broken Johannes Schindelin
2006-04-16 16:06 ` Linus Torvalds
2006-04-16 16:45 ` Johannes Schindelin
2006-04-16 17:07 ` Linus Torvalds
2006-04-16 17:27 ` path limiting broken (NOT) Johannes Schindelin
2006-04-16 17:39 ` path limiting broken Johannes Schindelin
2006-04-16 17:58 ` Linus Torvalds
2006-04-16 18:09 ` Johannes Schindelin
2006-04-16 18:27 ` Linus Torvalds [this message]
2006-04-16 23:49 ` Johannes Schindelin
2006-04-17 2:48 ` Linus Torvalds
2006-04-16 17:05 ` Johannes Schindelin
2006-04-16 17:51 ` Linus Torvalds
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.0604161117360.3701@g5.osdl.org \
--to=torvalds@osdl.org \
--cc=Johannes.Schindelin@gmx.de \
--cc=git@vger.kernel.org \
/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).