Re: RFE: Add -p to the diff options to provide more context in commit mails

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

 



On Thu, 23 Apr 2009 18:35:41 -0400
Chuck Ebbert <cebbert@xxxxxxxxxx> wrote:

> This trivial patch adds more context to the diffs in commit mails:
> 
> --- CVSROOT.orig/syncmail
> +++ CVSROOT/syncmail
> @@ -147,7 +147,7 @@ def calculate_diff(filespec):
>          brief = ""
>          if optNoDiff:
>              brief = "--brief"
> -        diffcmd = '/usr/bin/cvs -f diff %s -kk -u -N -r %s -r %s %s'
> % (
> +        diffcmd = '/usr/bin/cvs -f diff %s -kk -u -p -N -r %s -r %s
> %s' % ( brief,
>              oldrev, newrev, file)
>          fp = os.popen(diffcmd)
> 

This looks nice to me... Since we are in a change freeze tho, we will
need to get it approved on the infrastructure list. 

Shall we just wait until after release, or would you like me to request
the change now?

kevin

Attachment: signature.asc
Description: PGP signature

-- 
fedora-devel-list mailing list
fedora-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/fedora-devel-list

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]
  Powered by Linux