[Powertop] [PATCH UPDATED] powertop: update build instructions

[Powertop] [PATCH UPDATED] powertop: update build instructions

[Mohammad Merajul Islam Molla]

[-- Attachment #1: Type: text/plain, Size: 657 bytes --]

update powertop build instuctions in README file.

Signed-off-by: Mohammad Merajul Islam Molla <meraj.enigma(a)gmail.com>
---
 README | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/README b/README
index cd38cc5..bc48d3d 100644
--- a/README
+++ b/README
@@ -2,9 +2,10 @@
 ------------------------------
 
 To build and install PowerTOP type the following commands, 
+	./autogen.sh
 	./configure 
-	./make
-	./make install
+	make
+	sudo make install
 
 Note: For Android (running Intel Architecture ) there is a Android.mk 
 that was provided by community members, and at this time is supported
-- 
1.9.1

Re: [Powertop] [PATCH UPDATED] powertop: update build instructions

[Sergey Senozhatsky]

[-- Attachment #1: Type: text/plain, Size: 790 bytes --]

On (07/13/14 14:07), Mohammad Merajul Islam Molla wrote:
> update powertop build instuctions in README file.
> 

thanks.

	-ss

> Signed-off-by: Mohammad Merajul Islam Molla <meraj.enigma(a)gmail.com>
> ---
>  README | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/README b/README
> index cd38cc5..bc48d3d 100644
> --- a/README
> +++ b/README
> @@ -2,9 +2,10 @@
>  ------------------------------
>  
>  To build and install PowerTOP type the following commands, 
> +	./autogen.sh
>  	./configure 
> -	./make
> -	./make install
> +	make
> +	sudo make install
>  
>  Note: For Android (running Intel Architecture ) there is a Android.mk 
>  that was provided by community members, and at this time is supported
> -- 
> 1.9.1
> 

Re: [Powertop] [PATCH UPDATED] powertop: update build instructions

[Magnus Fromreide]

[-- Attachment #1: Type: text/plain, Size: 1271 bytes --]

On Sun, Jul 13, 2014 at 02:07:54PM +0600, Mohammad Merajul Islam Molla wrote:
> update powertop build instuctions in README file.

Should the README file reflect the repository code or the distributed
packages?

Is it not part of distribution package generation to run autogen so that
the old README file was fine, the only change one could imagine is adding
a README.hacking that states that you might need to run autogen.sh
before the commit.

/MF

> Signed-off-by: Mohammad Merajul Islam Molla <meraj.enigma(a)gmail.com>
> ---
>  README | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/README b/README
> index cd38cc5..bc48d3d 100644
> --- a/README
> +++ b/README
> @@ -2,9 +2,10 @@
>  ------------------------------
>  
>  To build and install PowerTOP type the following commands, 
> +	./autogen.sh
>  	./configure 
> -	./make
> -	./make install
> +	make
> +	sudo make install
>  
>  Note: For Android (running Intel Architecture ) there is a Android.mk 
>  that was provided by community members, and at this time is supported
> -- 
> 1.9.1
> 
> _______________________________________________
> PowerTop mailing list
> PowerTop(a)lists.01.org
> https://lists.01.org/mailman/listinfo/powertop

Re: [Powertop] [PATCH UPDATED] powertop: update build instructions

[Arjan van de Ven]

[-- Attachment #1: Type: text/plain, Size: 765 bytes --]

On 7/13/2014 10:41 AM, Magnus Fromreide wrote:
> On Sun, Jul 13, 2014 at 02:07:54PM +0600, Mohammad Merajul Islam Molla wrote:
>> update powertop build instuctions in README file.
>
> Should the README file reflect the repository code or the distributed
> packages?
>
> Is it not part of distribution package generation to run autogen so that
> the old README file was fine, the only change one could imagine is adding
> a README.hacking that states that you might need to run autogen.sh
> before the commit.

it is supposed to be part of distribution package generation, but we've had a few slipups
in that regard in recent releases unfortunately.

I more imagine a section in the readme than a second file, I mean, it's like one extra paragraph