Re: Patch to README-release

2012-02-09 Thread Jim Meyering
Reuben Thomas wrote:
> Ping? This is quite a simple patch, basically just a couple of
> punctuation fixes!

Hi Reuben,
The patch is ok (capitalize one letter and insert a newline), but is
perhaps too small, especially since I think you have a few more affecting
the same file.  I've gone ahead and adjusted the one-line summary
and pushed it anyway.

>From eab243b0c190e877bcdfe4242ccce63bf3677529 Mon Sep 17 00:00:00 2001
From: Reuben Thomas 
Date: Thu, 9 Feb 2012 11:06:39 +0100
Subject: [PATCH] README-release: capitalize a word and split a line

* top/README-release: Fix punctuation and spacing.
---
 ChangeLog  |5 +
 top/README-release |6 --
 2 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index f7f003f..f0e6b31 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2011-12-22  Reuben Thomas  
+
+   README-release: capitalize a word and split a line
+   * top/README-release: Fix punctuation and spacing.
+
 2012-02-08  Akim Demaille  

fatal-signal: use C prototypes (with explicit void).
diff --git a/top/README-release b/top/README-release
index d6e032a..25e1dbf 100644
--- a/top/README-release
+++ b/top/README-release
@@ -1,6 +1,6 @@
 Here are most of the steps we (maintainers) follow when making a release.

-* start from a clean, up-to-date git directory.
+* Start from a clean, up-to-date git directory:

 make -k maintainer-clean || { ./configure && make maintainer-clean; }
 git checkout master
@@ -11,7 +11,9 @@ Here are most of the steps we (maintainers) follow when 
making a release.
   the complete list of tools.

 * Ensure that you're on "master" with no uncommitted diffs.
-  This should produce no output: git checkout master; git diff
+  This should produce no output:
+
+git checkout master; git diff

 * Ensure that you've pushed all changes that belong in the release
   and that the NixOS/Hydra autobuilder is reporting all is well:
--
1.7.9.190.gf1d33



Re: Patch to README-release

2012-02-09 Thread Paul Eggert
On 02/08/2012 03:33 PM, Reuben Thomas wrote:
> Ping?

Are you talking about the patch in
?
That looks OK to me.  Jim?



Re: Patch to README-release

2012-02-08 Thread Reuben Thomas
Ping? This is quite a simple patch, basically just a couple of
punctuation fixes!

-- 
http://rrt.sc3d.org