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

[Steven Rostedt <rostedt@goodmis.org>]

On Thu, 2010-03-25 at 10:23 -0400, John Kacur wrote:
> ----- "Steven Rostedt" <rostedt@goodmis.org> wrote:
> 
> > 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?
> 
> I rebased the repo and did a push -f
> 

Thanks I'll pull it.


> > > +
> > > +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
> > 
> 
> 
> That is an oddity, I tried to show a general case.
> We could append more info about that special case if you wish.

Yeah, it was the default case ;-)

The general case would probably be better to:

make prefix=/usr

-- Steve