Messages in this thread | | | Date | Mon, 22 Oct 2012 12:07:19 +0200 | From | Borislav Petkov <> | Subject | Re: [PATCH] perf: Build install-man target when installing |
| |
On Mon, Oct 22, 2012 at 04:25:24PM +0900, Namhyung Kim wrote: > It will add additional dependencies of asciidoc, docbook-xsl and/or > something to default perf install command. I don't know it matters > much, but just wanted to say. What if a user want to install the perf > but the required packages for documents are not installed?
Ok, this is a valid point, more or less.
What we could do is leave the reference to "man perf-list" in perf list output and *not* build install-man unconditionally. This way, people who want to see the manpage, will need to install required packages. On distros the manpage is already present so all is taken care of.
This way, we avoid the text duplication of adding this to perf --help.
-- Regards/Gruss, Boris.
| |