Re: Git, ding-cvslog (gmane.emacs.gnus.cvs), ding-patches (gmane.emacs.gnus.commits)

[Ted Zlatanov <tzz@lifelogs.com>]

On Wed, 01 Sep 2010 22:22:05 +0200 asjo@koldfront.dk (Adam Sjøgren) wrote: 

AS> I am trying to sort of follow what happens by looking at the emails, and
AS> to me everything before the:

AS>   - Log -----------------------------------------------------------------

AS> -line is noise (or stuff I would be happy to scroll down and look at, in
AS> the rare cases I need that information). While log and diff are the
AS> parts I most often want to read.

AS> So I suggest that the part that looks like this at the top:

AS>          via  f95141317eb39d9290d2fb9e7794b42db17b8991 (commit)
AS>         from  d7c1cd6edd6b738730b0a646b7e8a257d30194df (commit)

AS>   Those revisions listed above that are new to this repository have
AS>   not appeared on any other notification email; so we list those
AS>   revisions in full, below.

AS> also is moved after the log and diff-parts.

I can do it, sure, but the revision list is a table of contents and
pretty nice in any non-trivial situation so I'd leave it on top.  Would
something like the following work for you?

Ted


diff --git a/etc/post-receive b/etc/post-receive
index 37ec5f9..9309ea3 100755
--- a/etc/post-receive
+++ b/etc/post-receive
@@ -405,10 +405,6 @@ generate_update_branch_email()
 
 	echo ""
 	if [ -z "$rewind_only" ]; then
-		echo "Those revisions listed above that are new to this repository have"
-		echo "not appeared on any other notification email; so we list those"
-		echo "revisions in full, below."
-
 		echo ""
 		echo $LOGBEGIN
 		show_new_revisions
@@ -418,6 +414,11 @@ generate_update_branch_email()
 		# revisions added by this update" message
 
 		echo $LOGEND
+
+		echo "Those revisions listed above that are new to this repository have"
+		echo "not appeared on any other notification email; so we listed those"
+		echo "revisions in full, above."
+
 	else
 		echo "No new revisions were added by this update."
 	fi