From: Jakub Narebski <jnareb@gmail.com>
To: Shawn Pearce <spearce@spearce.org>
Cc: Carl Worth <cworth@cworth.org>, git@vger.kernel.org
Subject: Re: Avoiding uninteresting merges in Cairo
Date: Fri, 15 Dec 2006 16:57:05 +0100 [thread overview]
Message-ID: <200612151657.06909.jnareb@gmail.com> (raw)
In-Reply-To: <20061215150130.GB17860@spearce.org>
Shawn Pearce wrote:
> Jakub Narebski <jnareb@gmail.com> wrote:
>> Carl Worth wrote:
>>
>>> On Thu, 14 Dec 2006 22:25:30 -0500, Shawn Pearce wrote:
>>
>>>> * gitweb made it appear as though a whole lot of Carl's recent
>>>> work was somehow undone in the merge.
>>>
>>> That looks like a simple gitweb bug. None of the other tools, (gitk,
>>> git log -p), consider a trivial merge commit like this as having
>>> anything interesting in it worth displaying.
>>
>> It's not a bug, it is rather lack of feature (or misfeature).
>
> Its a bug.
>
> I'm not a gitweb user (meaning I almost never look at something
> in gitweb). But I'm clearly also not a Git newbie. ;-)
>
> I could not fathom why that merge commit was being displayed that
> way in gitweb. I had to clone the cairo project just so I could
> actually look at the commit with log/show/whatchanged/diff-tree,
> because I couldn't believe what I was seeing from gitweb.
Do you mean "commit" view or "commitdiff" view in gitweb for merges
is confusing?
If it is "commit" view, it is fairly easy to remove difftree/whatchanged
output below commit message for merges. However while git-show nor
git-diff-tree doesn't show diff for merge messages, the diftree output
in "commit" view might be taken as 'damages'; git diff --summary always
takes summary of diff against first parent.
If it is "commitdiff" view... well, I plan on adding combined diff
output to commitdiff, but I need raw (whatchanged) output with the
same files which would be shown in git-diff --cc for merges (compact
combined diff output). Otherwise I'd have to use combined (-c) output
in gitweb, rather than more terse --cc output.
--
Jakub Narebski
next prev parent reply other threads:[~2006-12-15 15:54 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-12-15 2:06 Avoiding uninteresting merges in Cairo Shawn Pearce
2006-12-15 3:17 ` Carl Worth
2006-12-15 3:25 ` Shawn Pearce
2006-12-15 4:01 ` Carl Worth
2006-12-15 7:53 ` Shawn Pearce
2006-12-15 14:21 ` Jakub Narebski
2006-12-15 15:01 ` Shawn Pearce
2006-12-15 15:57 ` Jakub Narebski [this message]
2006-12-15 16:03 ` Shawn Pearce
2006-12-15 16:53 ` [PATCH] gitweb: Do not show difftree for merges in "commit" view Jakub Narebski
2006-12-15 21:03 ` Avoiding uninteresting merges in Cairo 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=200612151657.06909.jnareb@gmail.com \
--to=jnareb@gmail.com \
--cc=cworth@cworth.org \
--cc=git@vger.kernel.org \
--cc=spearce@spearce.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).