diff options
| author | Steven Rostedt <srostedt@redhat.com> | 2009-12-09 22:12:07 -0500 |
|---|---|---|
| committer | Steven Rostedt <rostedt@goodmis.org> | 2009-12-09 22:12:07 -0500 |
| commit | 2d802a522d04d89fc9277cd5c79dafb7af62fae4 (patch) | |
| tree | c9498fbf39d1dba8b9f1e41d45ebd97f9e79edcd | |
| parent | eef499fb3baf1ae058520bf16ced41c1dd45f9c1 (diff) | |
Do not record if no plugin or event was specified
An new user may just try doing a record and see what happens, and
will be surprised when there is no output to show.
This patch adds an error message if no event or plugin is specified.
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
| -rw-r--r-- | trace-cmd.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/trace-cmd.c b/trace-cmd.c index 22441b8..52ae03c 100644 --- a/trace-cmd.c +++ b/trace-cmd.c | |||
| @@ -1191,7 +1191,7 @@ int main (int argc, char **argv) | |||
| 1191 | case 'o': | 1191 | case 'o': |
| 1192 | if (!record) | 1192 | if (!record) |
| 1193 | die("start does not take output\n" | 1193 | die("start does not take output\n" |
| 1194 | "Did you mean 'record'?\n"); | 1194 | "Did you mean 'record'?"); |
| 1195 | if (output) | 1195 | if (output) |
| 1196 | die("only one output file allowed"); | 1196 | die("only one output file allowed"); |
| 1197 | output = optarg; | 1197 | output = optarg; |
| @@ -1264,6 +1264,9 @@ int main (int argc, char **argv) | |||
| 1264 | die("Command start does not take any commands\n" | 1264 | die("Command start does not take any commands\n" |
| 1265 | "Did you mean 'record'?"); | 1265 | "Did you mean 'record'?"); |
| 1266 | 1266 | ||
| 1267 | if (!events && !plugin) | ||
| 1268 | die("no event or plugin was specified... aborting"); | ||
| 1269 | |||
| 1267 | if (record) { | 1270 | if (record) { |
| 1268 | if (output) | 1271 | if (output) |
| 1269 | output_file = output; | 1272 | output_file = output; |
