search for: commit_subject_template

Hi, Sebastian Pop wrote: > Chris Lattner wrote: > > I'm in favor of it. Of course, the truly awesomest thing would be something like: > > > > [cfe-commits] r167788 - [lib/Analysis] Fix bad CFG construction bug when handling C++ 'try' statements. > > commit_subject_template = %(prefix)s r%(revision)s - %(log)s I just realized that this line does not match Chris' subject line as it doesn't print the directories touched by the patch. Please use the following instead: commit_subject_template = %(prefix)s r%(revision)s - [%(dirs)s] %(log)s Thanks, Sebastian --...

...wrote: >> Chris Lattner wrote: >> > I'm in favor of it. Of course, the truly awesomest thing would be something like: >> > >> > [cfe-commits] r167788 - [lib/Analysis] Fix bad CFG construction bug when handling C++ 'try' statements. >> >> commit_subject_template = %(prefix)s r%(revision)s - %(log)s > > I just realized that this line does not match Chris' subject line as it doesn't > print the directories touched by the patch. Please use the following instead: > > commit_subject_template = %(prefix)s r%(revision)s - [%(dirs)s] %(log)s...

...tner wrote: > >> > I'm in favor of it. Of course, the truly awesomest thing would be something like: > >> > > >> > [cfe-commits] r167788 - [lib/Analysis] Fix bad CFG construction bug when handling C++ 'try' statements. > >> > >> commit_subject_template = %(prefix)s r%(revision)s - %(log)s > > > > I just realized that this line does not match Chris' subject line as it doesn't > > print the directories touched by the patch. Please use the following instead: > > > > commit_subject_template = %(prefix)s r%(revisi...

...>> > I'm in favor of it. Of course, the truly awesomest thing would be something like: >> >> > >> >> > [cfe-commits] r167788 - [lib/Analysis] Fix bad CFG construction bug when handling C++ 'try' statements. >> >> >> >> commit_subject_template = %(prefix)s r%(revision)s - %(log)s Using r172358 as an example (a moderately large commit chosen otherwise at random), the current commit message subject is: [llvm-commits] [llvm] r172358 - in /llvm/trunk: include/llvm/ADT/StringRef.h include/llvm/Analysis/DOTGraphTraitsPass.h include/llvm/Obje...

...http://bugs.debian.org/cgi-bin/bugreport.cgi?msg=17;bug=379534 Note that you can directly patch the installed script: on my machine it is located in /usr/local/lib/python2.7/dist-packages/svnmailer/notifier/_mail.py and then replace or add the following line to the config file /etc/svn-mailer.conf commit_subject_template = %(prefix)s r%(revision)s - %(log)s Also 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...

..._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_header = SVN-Repository %(REPOS)s +commit_subject_template = %(prefix)s r%(revision)s - %(log)s [branch] for_repos = .*/llvm-project$ Note that I have not addressed the path prefix suggestion from David: >>> For what it's worth, (I think) Chris' suggestion of including the >>> directories was about including them "sma...

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

...>> > I'm in favor of it. Of course, the truly awesomest thing would be something like: >> >> > >> >> > [cfe-commits] r167788 - [lib/Analysis] Fix bad CFG construction bug when handling C++ 'try' statements. >> >> >> >> commit_subject_template = %(prefix)s r%(revision)s - %(log)s Using r172358 as an example (a moderately large commit chosen otherwise at random), the current commit message subject is: [llvm-commits] [llvm] r172358 - in /llvm/trunk: include/llvm/ADT/StringRef.h include/llvm/Analysis/DOTGraphTraitsPass.h include/llvm/Obje...

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

..._to)s > %(from)s %(to)s > generate_diffs = add copy modify propchange > I'll go ahead and make this change. - Daniel > > [authors] > @@ -360,6 +360,7 @@ > mail_type = single > show_applied_charset = nondefault > custom_header = SVN-Repository %(REPOS)s > +commit_subject_template = %(prefix)s r%(revision)s - %(log)s > > [branch] > for_repos = .*/llvm-project$ > > > > Note that I have not addressed the path prefix suggestion from David: > > >>> For what it's worth, (I think) Chris' suggestion of including the > >>> di...

...propchange > > > > I'll go ahead and make this change. > > - Daniel > > > > > > [authors] > > @@ -360,6 +360,7 @@ > > mail_type = single > > show_applied_charset = nondefault > > custom_header = SVN-Repository %(REPOS)s > > +commit_subject_template = %(prefix)s r%(revision)s - %(log)s > > > > [branch] > > for_repos = .*/llvm-project$ > > > > > > > > Note that I have not addressed the path prefix suggestion from David: > > > > >>> For what it's worth, (I think) Chris' sug...