From: Sam Vilain <sam@vilain.net>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Junio C Hamano <gitster@pobox.com>, git@vger.kernel.org
Subject: Re: Unresolved issues
Date: Thu, 20 Mar 2008 11:48:19 +1300 [thread overview]
Message-ID: <47E19833.60402@vilain.net> (raw)
In-Reply-To: <alpine.LFD.1.00.0803171853180.3020@woody.linux-foundation.org>
Linus Torvalds wrote:
>> When you run "git rev-list A..B C", and there is a commit in the chain
>> between A and B whose timestamp is much older than its parent, sometimes
>> we fail to mark C as reachable from A (hence not interesting) even when
>> it actualy is. This is very expensive to solve in general, and we are
>> not going to introduce "generation number" field to the commit objects,
>> so we may have to settle with a heuristic.
> Here is the already posted heuristic that fixes both t/t6009 and the
> real-world case that triggered the whole discussion.
> It's certainly not perfect, but I think it's likely an improvement on what
> we have now, and it should be robust in the face of the _occasional_ wrong
> date.
> Now, if there are consistently totally bogus dates, the SLOP thing won't
> help, but ...
Ouch - I had always supposed that topology was king, and that the commit
dates were purely informational. In particular the Perl history that I
produced in general takes a position of blatant and wanton disregard to
such consistency.
I can't find the other thread you refer to - is there a good summary of
the issues somewhere? The test script is not very descriptive.
If timewise out-of-order commits are bad, perhaps git-filter-branch
should warn when it is creating histories whose topology disagrees with
their chronology... and also the user manual should probably describe this.
Sam
next prev parent reply other threads:[~2008-03-19 22:46 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-03-18 1:12 Unresolved issues Junio C Hamano
2008-03-18 1:26 ` Jeff King
2008-03-18 1:56 ` Linus Torvalds
2008-03-19 22:48 ` Sam Vilain [this message]
2008-03-19 0:27 ` [PATCH] remote show: do not show symbolic refs Johannes Schindelin
-- strict thread matches above, loose matches on Subject: below --
2008-04-19 8:19 Unresolved issues Junio C Hamano
2007-02-20 7:28 Junio C Hamano
[not found] ` <Pine.LNX.4.64.07022009 34270.20368@woody.linux-foundation.org>
2007-02-20 8:57 ` Andy Parkins
2007-02-20 17:41 ` Linus Torvalds
2007-02-20 21:43 ` Junio C Hamano
2007-02-21 0:21 ` Linus Torvalds
2007-02-21 0:25 ` Junio C Hamano
2007-02-21 0:39 ` Johannes Schindelin
2007-02-21 0:56 ` Linus Torvalds
2007-02-21 0:51 ` David Lang
2007-02-21 1:12 ` Johannes Schindelin
2007-02-21 1:51 ` Nicolas Pitre
2007-02-21 2:03 ` Linus Torvalds
2007-02-21 16:32 ` Robin Rosenberg
2007-02-21 1:49 ` Theodore Tso
2007-02-21 10:42 ` Martin Waitz
2007-02-21 12:55 ` Johannes Schindelin
2007-02-21 16:57 ` Brian Gernhardt
2007-02-21 17:05 ` Shawn O. Pearce
2007-02-26 1:33 ` Julian Phillips
2007-02-26 3:39 ` Junio C Hamano
2007-02-26 5:10 ` Julian Phillips
2007-02-26 5:33 ` Junio C Hamano
2007-02-27 20:10 ` Johannes Schindelin
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=47E19833.60402@vilain.net \
--to=sam@vilain.net \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=torvalds@linux-foundation.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).