[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

little [PATCH] for mailer.py

From: Mathias Weinert <mathias.weinert_at_gfa-net.de>
Date: 2006-03-29 10:53:59 CEST

Hi there,

I am using mailer.py quite a while now and I find it more usefull if
the log message comes at the top of the mail, just after author, date
and revision number but not after all the changed paths.

Especially when many paths are changed with one commit I first want
to read the log message to see what the commit is about and then scroll
through the list of changed paths. Otherwise I have to scroll through
the whole list without knowing at all why these paths are changed.

BTW, I still hope that someone will commit my other patch to mailer.py
(Original post: http://svn.haxx.se/dev/archive-2006-02/0293.shtml,
several follow-ups exist).

Mathias

[[[
* tools/hook-scripts/mailer/mailer.py
  Show log message before changed paths (instead of after)
]]]

--- tools/hook-scripts/mailer/mailer.py.orig 2005-12-15 01:57:49.000000000 +0100
+++ tools/hook-scripts/mailer/mailer.py 2006-03-29 10:05:51.073150700 +0200
@@ -882,8 +882,8 @@
 
     w = self.output.write
 
- w('Author: %s\nDate: %s\nNew Revision: %s\n\n'
- % (data.author, data.date, data.rev))
+ w('Author: %s\nDate: %s\nNew Revision: %s\n\nLog: %s\n\n'
+ % (data.author, data.date, data.rev, data.log))
 
     # print summary sections
     self._render_list('Added', data.added_data)
@@ -900,8 +900,6 @@
       else:
         w('and changes in other areas\n')
 
- w('\nLog:\n%s\n' % data.log)
-
     self._render_diffs(data.diffs)
     if data.other_diffs:
       w('\nDiffs of changes in other areas also in this revision:\n')

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Mar 29 10:54:35 2006

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.