Re: [RFC/PATCH] gitweb: Create special from-file/to-file header for combined diff

On Mon, 28 May 2007, Petr Baudis wrote:

On Mon, May 28, 2007 at 08:50:51AM CEST, Jakub Narebski wrote:

quoted

Junio C Hamano wrote:

quoted

Jakub Narebski [off-list ref] writes:

quoted

Instead of using default, diff(1) like from-file/to-file header for
combined diff (for a merge commit), which looks like:

  --- a/git-gui/git-gui.sh
  +++ b/_git-gui/git-gui.sh_

(where _link_ denotes [hidden] hyperlink), create from-file(n)/to-file
header, using n/file for each or parents, e.g.:

  --- 1/_git-gui/git-gui.sh_
  --- 2/_git-gui.sh_
  +++ b/_git-gui/git-gui.sh_

Sounds quite straightforward to implement, and diff with 1/
would be useful to recreate what the person who did the merge
pulled in, for most of the time.  I suspect diff with 2/ is
almost always uninteresting, though.

I like it too.

quoted

Errr... it _is_ implemented in this patch, although code is not perfect
and has some unnecessary repetitions.

It just shows links to older versions of the blob, doesn't it? Links to
diffs themselves would be useful too.

For me most important were those from names (or /dev/null) in the case
of rename or copying (or file which was not present in some branches).

The link is to older version of blob; the links to appropriate blobdiffs
(and with earlier PATCH/RFC "gitweb: Provide links to individual
commitdiffs in difftree for merges" also to appropriate commitdiffs)
are in the difftree / whatchanged table.


Where to put link to diff? Should name be link to diff instead of
link to blob (to version at parent)?

-- 
Jakub Narebski
Poland