linux-perf-users.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Re: [PATCH] perf config: Fix example command in manpage to conform to  syntax specified in the SYNOPSIS section.
       [not found] <4NRiGHoarpAH9Bm8v4IALfprSaF_-io8e27O4A486cEAHRfNccrU0jOWM4Bep37oMSUSNk565zwqiWac_l-nYQXfPBWgAtDjWBJVfO4SbrY=@protonmail.com>
@ 2020-12-17 14:26 ` Arnaldo Carvalho de Melo
  2020-12-17 14:38   ` Arnaldo Carvalho de Melo
  0 siblings, 1 reply; 2+ messages in thread
From: Arnaldo Carvalho de Melo @ 2020-12-17 14:26 UTC (permalink / raw)
  To: Nick Thompson; +Cc: trivial, linux-perf-users, acme

Em Wed, Dec 16, 2020 at 05:29:43PM +0000, Nick Thompson escreveu:
> Attached is a trivial fix to the syntax in the perf config man page.


Thanks, I've added this to complement the commit log message, and as
well your Signed-off-by, ok?

- Arnaldo


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] perf config: Fix example command in manpage to conform to syntax specified in the SYNOPSIS section.
  2020-12-17 14:26 ` [PATCH] perf config: Fix example command in manpage to conform to syntax specified in the SYNOPSIS section Arnaldo Carvalho de Melo
@ 2020-12-17 14:38   ` Arnaldo Carvalho de Melo
  0 siblings, 0 replies; 2+ messages in thread
From: Arnaldo Carvalho de Melo @ 2020-12-17 14:38 UTC (permalink / raw)
  To: Nick Thompson
  Cc: trivial, linux-perf-users, Adrian Hunter, Ian Rogers, Jiri Olsa,
	Namhyung Kim, Arnaldo Carvalho de Melo

Em Thu, Dec 17, 2020 at 11:26:19AM -0300, Arnaldo Carvalho de Melo escreveu:
> Em Wed, Dec 16, 2020 at 05:29:43PM +0000, Nick Thompson escreveu:
> > Attached is a trivial fix to the syntax in the perf config man page.
> 
> 
> Thanks, I've added this to complement the commit log message, and as
> well your Signed-off-by, ok?


Oops, I mean this:

perf config: Fix example command in manpage to conform to syntax specified in the SYNOPSIS section.

Committer testing:

With the previously documented example:

  $ perf config --user report sort-order=srcline
  The config variable does not contain a section name: report
  $

With the fixed example line:

  $ perf config --user report.sort-order=srcline
  $ perf config --user report.sort-order
  report.sort-order=srcline
  $

Signed-off-by: Nick Thompson <nathompson7@protonmail.com>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Ian Rogers <irogers@google.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Link: https://lore.kernel.org/linux-perf-users/20201217142619.GA14524@redhat.com/
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-12-17 14:39 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <4NRiGHoarpAH9Bm8v4IALfprSaF_-io8e27O4A486cEAHRfNccrU0jOWM4Bep37oMSUSNk565zwqiWac_l-nYQXfPBWgAtDjWBJVfO4SbrY=@protonmail.com>
2020-12-17 14:26 ` [PATCH] perf config: Fix example command in manpage to conform to syntax specified in the SYNOPSIS section Arnaldo Carvalho de Melo
2020-12-17 14:38   ` Arnaldo Carvalho de Melo

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).