Re: [RP] patch to README

[Jérémie Courrèges-Anglas]

Jeff Abrahamson <address@hidden> writes:

> I didn't find instructions on how you'd like patches.  Let me know if you'd
> rather something else.

Almost any submission method is fine, as long as it's not a github pull
request. :)

> Jeff Abrahamson
> +33 6 24 40 01 57
> +44 7920 594 255
>
> http://jeff.purple.com/
> http://blog.purple.com/jeff/
>
> From 6695e2d177f323a2df0866a926496388f488ad67 Mon Sep 17 00:00:00 2001
> From: Jeff Abrahamson <address@hidden>
> Date: Fri, 18 Apr 2014 18:32:33 +0200
> Subject: [PATCH] Fix build instructions.
>
> 1.  Propose use of sudo rather than old-fashioned su.  Lead by
>     example.
>
> 2.  Reminder to run autogen.sh when pulling source from savannah git.
> ---
>  README | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/README b/README
> index 5189352..507c6d9 100644
> --- a/README
> +++ b/README
> @@ -28,7 +28,10 @@ $ ./configure && make
>  
>  if you want to install it, simply type 'make install' as a privileged user:
>  
> -# make install
> +$ sudo make install
> +
> +If you have cloned the ratpoison git repository, you may need to run
> +autogen.sh before running configure.

Good idea.

commit 8a8e8019f4c5c060df8815a075ef0874831ea842
Author: Jérémie Courrèges-Anglas <address@hidden>
Date:   Mon Apr 21 22:49:31 2014 +0200

    README tweaks from Jeff Abrahamson.
    
    * mention the use of sudo instead of a privileged shell
    * advertize the use of ./autogen.sh for users that pull from git
    * bonus: mention automake and the fact that recent autotools releases
      are needed

diff --git a/README b/README
index 5189352..5ff5f6b 100644
--- a/README
+++ b/README
@@ -22,13 +22,20 @@ that cripples EMACS and other quality pieces of software.
 Building
 --------
 
-ratpoison uses autoconf. To build it:
+ratpoison uses autoconf and automake. To build it:
 
 $ ./configure && make
 
-if you want to install it, simply type 'make install' as a privileged user:
+if you want to install it system-wide, simply type 'make install' as
+a privileged user:
 
-# make install
+$ sudo make install
+
+If you retrieved ratpoison from the source repository, you will have to
+install somewhat recent versions of autoconf and automake before
+running:
+
+$ ./autogen.sh
 
 Customization
 -------------


-- 
jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF  DDCC 0DFA 74AE 1524 E7EE