From: Linus Torvalds <torvalds@linux-foundation.org>
To: "Santi Béjar" <santi@agolina.net>
Cc: Brian Foster <brian.foster@innova-card.com>,
Junio C Hamano <gitster@pobox.com>,
Git Mailing List <git@vger.kernel.org>
Subject: Re: [PATCH 4/4] Add support for 'namespace' history simplification
Date: Mon, 3 Nov 2008 14:05:04 -0800 (PST) [thread overview]
Message-ID: <alpine.LFD.2.00.0811031358460.3419@nehalem.linux-foundation.org> (raw)
In-Reply-To: <adf1fd3d0811031345j4582e109jaf95aede0f33eff7@mail.gmail.com>
On Mon, 3 Nov 2008, Santi Béjar wrote:
>
> I tried it once, but I had problems simplifying the merges, and it is trivial...
It depends on the new --simplify-merges code which does that.
> Not that it matters a lot, but if you try it on master you get some
> extra merges without a ref like:
>
> 373a273 (Merge git-gui 0.11.0, 2008-08-17)
Umm? Your point is?
That merge itself doesn't have a ref, but it's required becase there are
refs along both legs of the merge - one side has the "gitgui-0.11.0" tag,
while the other has (for example) v16.0-rc3.
> f44bc33 (Sync with 1.5.6.5, 2008-08-06)
Again, the merge doesn't have a ref, but it's needed because there are
refs on both parents (v1.5.6.5 vs v1.6.0-rc[01]).
So no, --simplify-namespace in no way guarantees that all resulting
commits will have refs pointing to them, because it also needs to return
enough of the merges to make it a real and meaningful DAG.
The one thing I note is that when you have lots and lots of refs in the
gitk output, the gitk window itself becomes very ugly. I'd love to get rid
of the black line between the ref (tag or branch name) and the circle,
because with "gitk --simplify-namespace" it ends up looking like some kind
of insane "ladder" due to all those vertical lines.
And they really aren't necessary, and it would probably be better to just
make selecting a commit highlight the whole row (and thus avoid any
ambiguity between that highlighted commit message and the circle in the
graph it goes with if you have a very wide graph).
But I can't read tcl/tk enough to even figure out where it's being
painted.
Linus
next prev parent reply other threads:[~2008-11-03 22:07 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-11-03 13:39 [Q] Abbreviated history graph? Brian Foster
2008-11-03 14:20 ` Santi Béjar
2008-11-03 14:55 ` Santi Béjar
2008-11-03 15:46 ` Brian Foster
2008-11-03 16:08 ` Santi Béjar
2008-11-03 19:32 ` Linus Torvalds
2008-11-03 19:33 ` [PATCH 1/4] Add a 'source' decorator for commits Linus Torvalds
2008-11-03 19:35 ` [PATCH 2/4] revision: make tree comparison functions take commits rather than trees Linus Torvalds
2008-11-03 19:39 ` [PATCH 3/4] Make '--decorate' set an explicit 'show_decorations' flag Linus Torvalds
2008-11-03 19:43 ` [PATCH 4/4] Add support for 'namespace' history simplification Linus Torvalds
2008-11-03 21:45 ` Santi Béjar
2008-11-03 22:05 ` Linus Torvalds [this message]
2008-11-03 22:34 ` Santi Béjar
2008-11-03 22:28 ` Robin Rosenberg
2008-11-04 21:33 ` Clemens Buchacher
2008-11-03 20:15 ` [Q] Abbreviated history graph? Linus Torvalds
2008-11-03 20:34 ` Linus Torvalds
2008-11-04 8:45 ` Junio C Hamano
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=alpine.LFD.2.00.0811031358460.3419@nehalem.linux-foundation.org \
--to=torvalds@linux-foundation.org \
--cc=brian.foster@innova-card.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=santi@agolina.net \
/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