Project

General

Profile

Actions
Copy link

Bug #835

closed
DT

lttng load --list-options and lttng save --list-options incorrectly report an error

Bug #835: lttng load --list-options and lttng save --list-options incorrectly report an error

Added by Daniel U. Thibault almost 12 years ago. Updated over 11 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Target version:
LTTng - 2.5
Start date:
09/11/2014
Due date:
% Done:

100%

Estimated time:

Description

Unlike all other 'lttng <command> --list-options' commands, 'lttng load --list-options' and 'lttng save --list-options' return 'Error: Undefined command' (error code 2) after listing the options. I suspect this will prevent bash commnand completion from operating correctly for those two.

On a related note, 'lttng --list-commands' still lists 'enable-consumer' and 'disable-consumer' as commands. Asking for their list-options returns a 'Warning: Some command(s) went wrong' (error 4). This maybe by design (for retro-compatibility) since the 'create' command still lists the obsolete options 'disable-consumer' and 'no-consumer'.

DG Updated by David Goulet almost 12 years ago Actions
Copy link
 #1

  • Status changed from New to Confirmed

Yeah they are missing indeed.

Also, I just pushed a patch that removes any thing related to enable/disable consumer command that are obsolete for quite some time.
See commit 62e55253348f999a97d323a3b1503478c4f3ff51

DT Updated by Daniel U. Thibault almost 12 years ago Actions
Copy link
 #2

Update: After adding manually the load and save commands to /etc/bash_completion.d/lttng, I get the following behaviour:

$ lttng load --h
   (I now hit TAB)
$ lttng load --husage: lttng load [OPTIONS] [SESSION]

Options:
  -h, --help               Show this help
[...]
                           before creating new one(s).
Error: Undefined command

(And the command line is till "live", awaiting an ENTER...)

This is because the load and save commands don't yet implement the --list-options query.

DG Updated by David Goulet over 11 years ago Actions
Copy link
 #3

  • Status changed from Confirmed to Resolved
  • % Done changed from 0 to 100

Applied in changeset tools|commit:13a810d5e44e4e8e5ce4c1a36456cd0e394d36c2.

Actions
Copy link

Also available in: PDF Atom