git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH 0/6] gitweb: Add combined diff support

From
Jakub Narebski <jnareb@gmail.com>
Date
May 6, 2007, 23:10 UTC
Message-ID
<11784930091585-git-send-email-jnareb@gmail.com>

This series of patches adds support for showing combined diff for merge commits in gitweb, instead of using diff with the first parent (for 'commitdiff' view), or not showing difftree (for 'commit' view).

We use diff -c (diff --combined) and not diff --cc; the latter would require modifying git_difftree_body and git_patchset_body for the possibility that there is line in raw diff output format (in difftree) that doesn't correspond to any patch: the patch part was removed due to hunks simplification.

See also notes in individual patches.

Diffstat: =========

 gitweb/gitweb.css  |   17 ++
 gitweb/gitweb.perl |  480 +++++++++++++++++++++++++++++++++++++++++-----------
 2 files changed, 394 insertions(+), 103 deletions(-)

Table of contents: ==================

 [PATCH 1/6] gitweb: Add parsing of raw combined diff format to parse_difftree_raw_line
 [PATCH 2/6] gitweb: Add combined diff support to git_difftree_body
 [PATCH 3/6] gitweb: Add combined diff support to git_patchset_body
 [PATCH 4/6] gitweb: Make it possible to use pre-parsed info in git_difftree_body

Those patches prepare for adding support for combined diff for 'commitdiff' and 'commit' views.

 [PATCH 5/6] gitweb: Show combined diff for merge commits in 'commitdiff' view
 [PATCH 6/6] gitweb: Show combined diff for merge commits in 'commit' view

Those patches add (preliminary) generating combined diff for merge commits for 'commitdiff' view (if there was not provided commit to compare againts), and for 'commit' view.

 [PATCH 7/6] todo: todo: Remove "Gitweb diff on merge commits" entry
This patch updates TODO list in 'todo' branch.
-- 
Jakub Narebski
Next: Jakub Narebski
Message 1 of 12 in “gitweb: Add combined diff support”
  1. 0/6 gitweb: Add combined diff supportJakub Narebski, May 6, 2007
  2. 1/6 gitweb: Add parsing of raw combined diff format to parse_difftree_raw_lineJakub Narebski, May 6, 2007
  3. 2/6 gitweb: Add combined diff support to git_difftree_bodyJakub Narebski, May 6, 2007
  4. 3/6 gitweb: Add combined diff support to git_patchset_bodyJakub Narebski, May 6, 2007
  5. 4/6 gitweb: Make it possible to use pre-parsed info in git_difftree_bodyJakub Narebski, May 6, 2007
  6. 5/6 gitweb: Show combined diff for merge commits in 'commitdiff' viewJakub Narebski, May 6, 2007
  7. 6/6 gitweb: Show combined diff for merge commits in 'commit' viewJakub Narebski, May 6, 2007
  8. 7/6 todo: Remove "Gitweb diff on merge commits" entryJakub Narebski, May 6, 2007
  9. Junio C HamanoMay 8, 2007
  10. Jakub NarebskiMay 8, 2007
  11. 7/6 gitweb: Empty patch for merge means trivial merge, not no differencesJakub Narebski, May 16, 2007
  12. Junio C HamanoMay 16, 2007

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.