linux-kernel - Re: [PATCH] trace-cmd: Add BUILDING and INSTALL instructions to README

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-ID: <1269523868.2957.4060.camel@gandalf.stny.rr.com>
Date:	Thu, 25 Mar 2010 09:31:08 -0400
From:	Steven Rostedt <rostedt@...dmis.org>
To:	John Kacur <jkacur@...hat.com>
Cc:	lkml <linux-kernel@...r.kernel.org>,
	Clark Williams <williams@...hat.com>,
	Darren Hart <dvhltc@...ibm.com>
Subject: Re: [PATCH] trace-cmd: Add BUILDING and INSTALL instructions to
 README

On Thu, 2010-03-25 at 10:24 +0100, John Kacur wrote:
> - Add BUILDING and INSTALL instructions to the README
> - Change Makefile so that messages don't appear like errors.

Thanks,

Did you rebase your repo or do I need to pull in the other patches from
the mbox?

One comment below.

> 
> Signed-off-by: John Kacur <jkacur@...hat.com>
> ---
>  Makefile |    6 +++---
>  README   |   26 ++++++++++++++++++++++++++
>  2 files changed, 29 insertions(+), 3 deletions(-)
> 
> diff --git a/Makefile b/Makefile
> index 8a3f25c..04fe380 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -373,10 +373,10 @@ ifneq ($(dep_includes),)
>  endif
>  
>  show_gui_make:
> -	@echo "*** to build the gui, type \"make gui\" ***"
> +	@echo "Note: to build the gui, type \"make gui\""
>  
>  show_gui_done:
> -	@echo "*** gui build complete ***"
> +	@echo "gui build complete"
>  
>  PHONY += show_gui_make
>  
> @@ -397,7 +397,7 @@ install_cmd: all_cmd install_plugins
>  	$(INSTALL) trace-cmd '$(bindir_SQ)'
>  
>  install: install_cmd
> -	@echo "*** to install the gui, type \"make install_gui\" ***"
> +	@echo "Note: to install the gui, type \"make install_gui\""
>  
>  install_gui: install_cmd gui
>  	$(INSTALL) -d -m 755 '$(bindir_SQ)'
> diff --git a/README b/README
> index 7ac4390..d191197 100644
> --- a/README
> +++ b/README
> @@ -9,3 +9,29 @@ These files also make up the code to create the libraries
>  The applications are licensed under the GNU General Public License 2.0
>  (see COPYING) and the libraries are licensed under the GNU
>  Lesser General Public License 2.1 (See COPYING.LIB).
> +
> +BUILDING:
> +
> +To make trace-cmd
> +    make
> +
> +To make the gui
> +    make gui
> +
> +INSTALL:
> +
> +To install trace-cmd
> +    make install
> +
> +To install the gui
> +    make install_gui
> +
> +Note: The default install is relative to /usr/local
> +    The default install directory is /usr/local/bin
> +    The default plugin directory is /usr/local/share/trace-cmd/plugins
> +
> +To change the default, you can set 'prefix', eg
> +mkdir $HOME/test-trace
> +make prefix=$HOME/test-trace
> +make prefix=$HOME/test-trace install

Looks good, but I would use the example of this:

make prefix=~
make prefix=~ install

The above would install the executables into your home directory ~/bin
and the plugins into ~/.trace-cmd/plugins

-- Steve

> +


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/