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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Junio C Hamano wrote:

> I've minimally tried this on my private machine.  Looks pretty
> nice for simple merges, but I think we would want --cc not -c
> most of the time.

Please note that --cc would need some changes to both git_difftree_body 
and git_patchset_body; it is not just matter of replacing 
'-c' by '--cc'. Hunk simplification might mean that whole patch 
vanishes. So sometimes we have difftree (raw diff, whatchanged) line 
which does not have corresponding patch, and there should be no "patch" 
link (this is harder part). This mean also that in git_patchset_body we 
need sometimes to skip some difftree line / difftree info line (this is 
easier part).

-- 
Jakub Narebski
Poland
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]