From: Junio C Hamano <gitster@pobox.com>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Thomas Rast <trast@student.ethz.ch>, git@vger.kernel.org
Subject: Re: [RFC PATCH] Documentation: rev-list-options: clarify history simplification with paths
Date: Sun, 10 Aug 2008 14:58:05 -0700 [thread overview]
Message-ID: <7v63q8353m.fsf@gitster.siamese.dyndns.org> (raw)
In-Reply-To: <alpine.LFD.1.10.0808101226130.3462@nehalem.linux-foundation.org> (Linus Torvalds's message of "Sun, 10 Aug 2008 12:27:58 -0700 (PDT)")
Linus Torvalds <torvalds@linux-foundation.org> writes:
> On Sun, 10 Aug 2008, Junio C Hamano wrote:
>>
>> Personally I do not think --full-history without --parents is of much
>> usefulness (I'd let Linus or somebody else defend this usage, or make it
>> imply revs.rewrite_parents otherwise). If you remove that case from your
>> set of experiments in the equation, do the rest of the results make sense?
>
> Oh, it's _very_ useful.
>
> The most common case is "git whatchanged". It's useful to find a commit
> that did some change _without_ any graphical front-end.
>
> And then the merges and parenthood are totally pointless - no human can
> try to tie things together in their head _anyway_, so why show them? You
> just want to find the change.
Oh, I was not talking about revs.print_parents part, but about
revs.rewrite_parents part. What got Thomas puzzled about was exactly how
the set of commits _shown_ are different with and without --parents, which
sets both of these internal flags. Your "pointless" argument applies to
"print_parents" part, but "rewrite_parents" affects the resulting set.
next prev parent reply other threads:[~2008-08-10 21:59 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-08-10 13:44 [RFC PATCH] Documentation: rev-list-options: clarify history simplification with paths Thomas Rast
2008-08-10 19:19 ` Junio C Hamano
2008-08-10 19:27 ` Linus Torvalds
2008-08-10 21:58 ` Junio C Hamano [this message]
2008-08-10 22:16 ` Linus Torvalds
2008-08-10 21:31 ` Thomas Rast
2008-08-11 20:39 ` Junio C Hamano
2008-08-11 20:34 ` Junio C Hamano
2008-08-11 23:51 ` Thomas Rast
2008-08-11 23:55 ` [PATCH 1/3] Documentation: rev-list-options: Fix a typo Thomas Rast
2008-08-11 23:55 ` [PATCH 2/3] Documentation: rev-list-options: Rewrite simplification descriptions for clarity Thomas Rast
2008-08-11 23:55 ` [PATCH 3/3] Documentation: rev-list-options: move --simplify-merges documentation Thomas Rast
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=7v63q8353m.fsf@gitster.siamese.dyndns.org \
--to=gitster@pobox.com \
--cc=git@vger.kernel.org \
--cc=torvalds@linux-foundation.org \
--cc=trast@student.ethz.ch \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.