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 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.