search for: diff_command

...r wrote: > I'm a bit hesitant to just hack up the installed copy of svnmailer, are we > sure this patch hasn't been dealt with in an upstream version? I don't see that change in the last dev version: http://storage.perlig.de/svnmailer/svnmailer-1.1.0-dev-r1373.tar.bz2 > > -diff_command = /usr/bin/diff -u -L %(label_from)s -L %(label_to)s > > %(from)s %(to)s > > +diff_command = /usr/bin/diff -up -L %(label_from)s -L %(label_to)s > > %(from)s %(to)s > > generate_diffs = add copy modify propchange > > > > I'll go ahead and make this change....

...sure this patch hasn't been dealt with in an upstream version? > > --- svn-mailer.conf.orig 2013-01-25 15:39:23.000000000 -0600 > +++ svn-mailer.conf 2013-01-25 15:42:02.000000000 -0600 > @@ -1,6 +1,6 @@ > [general] > sendmail_command = /usr/sbin/sendmail > -diff_command = /usr/bin/diff -u -L %(label_from)s -L %(label_to)s > %(from)s %(to)s > +diff_command = /usr/bin/diff -up -L %(label_from)s -L %(label_to)s > %(from)s %(to)s > generate_diffs = add copy modify propchange > I'll go ahead and make this change. - Daniel > > [authors] &...

...ant to just hack up the installed copy of svnmailer, are > we > > sure this patch hasn't been dealt with in an upstream version? > > I don't see that change in the last dev version: > http://storage.perlig.de/svnmailer/svnmailer-1.1.0-dev-r1373.tar.bz2 > > > > -diff_command = /usr/bin/diff -u -L %(label_from)s -L %(label_to)s > > > %(from)s %(to)s > > > +diff_command = /usr/bin/diff -up -L %(label_from)s -L %(label_to)s > > > %(from)s %(to)s > > > generate_diffs = add copy modify propchange > > > > > > > I'...

...: (the first change to print the context of diff is something that I was missing more than the subject) --- svn-mailer.conf.orig 2013-01-25 15:39:23.000000000 -0600 +++ svn-mailer.conf 2013-01-25 15:42:02.000000000 -0600 @@ -1,6 +1,6 @@ [general] sendmail_command = /usr/sbin/sendmail -diff_command = /usr/bin/diff -u -L %(label_from)s -L %(label_to)s %(from)s %(to)s +diff_command = /usr/bin/diff -up -L %(label_from)s -L %(label_to)s %(from)s %(to)s generate_diffs = add copy modify propchange [authors] @@ -360,6 +360,7 @@ mail_type = single show_applied_charset = nondefault custom_heade...

Hi Daniel, Daniel Dunbar wrote: > Hi Sebastion, > > I've attached the current configuration file from the server. > > I'm not sure how far you want to go down the "trying to get realize nice > summary path" lines, but if svn_mailer somehow supported running an > external script to process the commit and come up with the path that would > be ideal for

...alled copy of svnmailer, are >> we >> > sure this patch hasn't been dealt with in an upstream version? >> >> I don't see that change in the last dev version: >> http://storage.perlig.de/svnmailer/svnmailer-1.1.0-dev-r1373.tar.bz2 >> >> > > -diff_command = /usr/bin/diff -u -L %(label_from)s -L %(label_to)s >> > > %(from)s %(to)s >> > > +diff_command = /usr/bin/diff -up -L %(label_from)s -L %(label_to)s >> > > %(from)s %(to)s >> > > generate_diffs = add copy modify propchange >> > > >&g...

...in an upstream version? > > > > > > --- svn-mailer.conf.orig 2013-01-25 15:39:23.000000000 -0600 > > +++ svn-mailer.conf 2013-01-25 15:42:02.000000000 -0600 > > @@ -1,6 +1,6 @@ > > [general] > > sendmail_command = /usr/sbin/sendmail > > -diff_command = /usr/bin/diff -u -L %(label_from)s -L %(label_to)s > > %(from)s %(to)s > > +diff_command = /usr/bin/diff -up -L %(label_from)s -L %(label_to)s > > %(from)s %(to)s > > generate_diffs = add copy modify propchange > > > > I'll go ahead and make this change. &...

...lso as David Blaikie has pointed out, we are missing an upper bound on the subject line length, so you may want to consider adding this: max_subject_length = 120 While we are at it, let's also fix the missing context, by using a custom diff command with the -p flag. Here is the config line: diff_command = /usr/bin/diff -up -L %(label_from)s -L %(label_to)s %(from)s %(to)s Can somebody do these modifications to llvm.org's /etc/svn-mailer.conf? Thanks, Sebastian -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation

On Nov 15, 2012, at 2:28 PM, Chandler Carruth <chandlerc at google.com> wrote: >> [cfe-commits] r167788 - in /cfe/trunk: lib/Analysis/CFG.cpp test/Analysis/dead-stores.cpp >> >> with >> >> [cfe-commits] r167788 - Fix bad CFG construction bug when handling C++ 'try' statements. >> >> The first gives me just enough information (in my