[LTP] [PATCH v2 1/6] doc: INSTALL: dma_thread_diotest.c: Remove runltp from doc

[Cyril Hrubis]

Hi!
> > > > > @@ -105,9 +105,9 @@ Quick Start
> > > > >  	$ ./configure
> > > > >  	$ make all
> > > > >  	# make install
> > > >         ^
> > > > 	This should be $ (not caused by this patch but worth fixing)
> 
> > > I can change it, but IMHO for creating /opt/ltp is needed to run it as root
> > > for a first time. Or one must chmod/chown /opt/ltp under root.
> 
> > Huh? The make install should just work fine. I'm just pointing out the
> > typo # vs $. Or did I miss something?
> 
> Yeah, I understand it and plan to change # => $. I talked about /opt (our
> default prefix) usually not being writable by regular users (installing any
> software using standard system prefix will have this problem). Root is needed
> only for creating /opt/ltp. That was the reason for the original user to use '#'.
> But as I wrote, I'm ok to change the prompt.

Ah, that's a bit confusing then because in shell script # is a comment.
I usually prefix commands that needs root with sudo instead. So what
about "$ sudo make install" ?

-- 
Cyril Hrubis
chrubis@suse.cz