Re: patch to README

[email protected] (Jérémie Courrèges-Anglas)
Newsgroups gmane.comp.window-managers.ratpoison.devel
Message-ID <[email protected]>
Jeff Abrahamson <[email protected]> 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 <[email protected]>
> 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 <[email protected]>
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

_______________________________________________
Ratpoison-devel mailing list
[email protected]
https://lists.nongnu.org/mailman/listinfo/ratpoison-devel
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.