From: Petr Baudis <pasky@suse.cz>
To: Linus Torvalds <torvalds@osdl.org>
Cc: Junio C Hamano <junkio@cox.net>, Pavel Roskin <proski@gnu.org>,
fonseca@diku.dk, git <git@vger.kernel.org>
Subject: Re: Joining cg-*-id
Date: Tue, 20 Sep 2005 15:57:36 +0200 [thread overview]
Message-ID: <20050920135735.GC1884@pasky.or.cz> (raw)
In-Reply-To: <Pine.LNX.4.58.0509191746130.2553@g5.osdl.org>
Dear diary, on Tue, Sep 20, 2005 at 02:50:21AM CEST, I got a letter
where Linus Torvalds <torvalds@osdl.org> told me that...
> On Tue, 20 Sep 2005, Petr Baudis wrote:
> > But that semantics sucks, and I wouldn't mind changing it at all. I
> > think taking the previous commit works for everything but the start of
> > cg-log range, and it is how CVS behaves.
>
> The thing is, it may have sensible behaviour for CVS, but only because CVS
> doesn't have any notion of concurrent development (even a branch isn't
> concurrent - it's totally separate, and when you merge it, it becomes just
> one big diff at the time of the merge).
>
> So I'd love to have
>
> git diff yesterday..
>
> but the fact is, there's no sensible semantics for it. _which_ yesterday?
> There might be five different points that are "close to 24 hours ago",
> along five different paths backwards in the history.
A well-defined meaning for this from Cogito standpoint would be "the
last commit on our HEAD before the date and all commits committed and
merged to the HEAD". In Cogito, you don't merge two branches _together_,
you merge one branch _into_ another (represented by the parents order),
so this would be sensible.
But we would have to sort the log in merge order for this to be possible
(and probably teach git-rev-list about it anyway). I would absolutely
*love* to have cg-log in merge order instead of date order - longer I
see it the more I believe that it's evil and you should really want
merge order instead. The trouble is that cg-log would be no longer
incremental and would have to load the whole history first, a no-go for
any measurably long history.
By the way, did I understand it right that the only difference between
the results of merge ordering and topo ordering is the order commits
with regard to the order of parents?
--
Petr "Pasky" Baudis
Stuff: http://pasky.or.cz/
VI has two modes: the one in which it beeps and the one in which
it doesn't.
next prev parent reply other threads:[~2005-09-20 13:58 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-09-19 21:40 Joining cg-*-id Pavel Roskin
2005-09-19 21:56 ` Petr Baudis
2005-09-19 22:16 ` Linus Torvalds
2005-09-19 22:54 ` Petr Baudis
2005-09-20 0:50 ` Linus Torvalds
2005-09-20 13:57 ` Petr Baudis [this message]
2005-09-20 14:53 ` Linus Torvalds
2005-09-20 15:07 ` Petr Baudis
2005-09-20 16:54 ` Linus Torvalds
2005-09-20 19:34 ` Linus Torvalds
2005-09-21 10:35 ` Paul Mackerras
2005-09-21 14:51 ` Linus Torvalds
2005-09-21 21:19 ` Paul Mackerras
2005-09-21 8:50 ` Petr Baudis
2005-09-20 15:13 ` Matthias Urlichs
2005-09-20 5:52 ` Junio C Hamano
2005-09-20 10:11 ` Matthias Urlichs
2005-09-20 12:32 ` Paul Mackerras
2005-09-19 22:30 ` Pavel Roskin
2005-09-19 22:58 ` Petr Baudis
2005-09-20 0:19 ` Pavel Roskin
2005-09-20 0:26 ` Petr Baudis
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=20050920135735.GC1884@pasky.or.cz \
--to=pasky@suse.cz \
--cc=fonseca@diku.dk \
--cc=git@vger.kernel.org \
--cc=junkio@cox.net \
--cc=proski@gnu.org \
--cc=torvalds@osdl.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).