* SOFTWARE.
*/
-#define BT_LOG_TAG "CLI-CFG-CLI-ARGS"
+#define BT_LOG_TAG "CLI/CFG-CLI-ARGS"
#include "logging.h"
#include <errno.h>
#include <inttypes.h>
#include <babeltrace2/babeltrace.h>
#include "common/common.h"
-#include <popt.h>
#include <glib.h>
#include <sys/types.h>
+#include "argpar/argpar.h"
#include "babeltrace2-cfg.h"
#include "babeltrace2-cfg-cli-args.h"
#include "babeltrace2-cfg-cli-args-connect.h"
+#include "babeltrace2-cfg-cli-params-arg.h"
+#include "babeltrace2-plugins.h"
+#include "babeltrace2-cfg-src-auto-disc.h"
#include "common/version.h"
-/*
- * Error printf() macro which prepends "Error: " the first time it's
- * called. This gives a nicer feel than having a bunch of error prefixes
- * (since the following lines usually describe the error and possible
- * solutions), or the error prefix just at the end.
- */
-#define printf_err(fmt, args...) \
- do { \
- if (is_first_error) { \
- fprintf(stderr, "Command line error: "); \
- is_first_error = false; \
- } \
- fprintf(stderr, fmt, ##args); \
- } while (0)
-
-static bool is_first_error = true;
-static const int cli_default_log_level = BT_LOG_WARN;
+static const int cli_default_log_level = BT_LOG_WARNING;
/* INI-style parsing FSM states */
enum ini_parsing_fsm_state {
LEGACY_OUTPUT_FORMAT_DUMMY,
};
-/*
- * Prints the "out of memory" error.
- */
-static
-void print_err_oom(void)
-{
- printf_err("Out of memory\n");
-}
-
-/*
- * Appends an "expecting token" error to the INI-style parsing state's
- * error buffer.
- */
-static
-void ini_append_error_expecting(struct ini_parsing_state *state,
- GScanner *scanner, const char *expecting)
-{
- size_t i;
- size_t pos;
-
- g_string_append_printf(state->ini_error, "Expecting %s:\n", expecting);
-
- /* Only print error if there's one line */
- if (strchr(state->arg, '\n') != NULL || strlen(state->arg) == 0) {
- return;
- }
-
- g_string_append_printf(state->ini_error, "\n %s\n", state->arg);
- pos = g_scanner_cur_position(scanner) + 4;
-
- if (!g_scanner_eof(scanner)) {
- pos--;
- }
-
- for (i = 0; i < pos; ++i) {
- g_string_append_printf(state->ini_error, " ");
- }
-
- g_string_append_printf(state->ini_error, "^\n\n");
-}
-
-/* Parse the next token as an unsigned integer. */
-static
-bt_value *ini_parse_uint(struct ini_parsing_state *state)
-{
- bt_value *value = NULL;
- GTokenType token_type = g_scanner_get_next_token(state->scanner);
-
- if (token_type != G_TOKEN_INT) {
- ini_append_error_expecting(state, state->scanner,
- "integer value");
- goto end;
- }
-
- value = bt_value_unsigned_integer_create_init(
- state->scanner->value.v_int64);
-
-end:
- return value;
-}
-
-/* Parse the next token as a number and return its negation. */
-static
-bt_value *ini_parse_neg_number(struct ini_parsing_state *state)
-{
- bt_value *value = NULL;
- GTokenType token_type = g_scanner_get_next_token(state->scanner);
-
- switch (token_type) {
- case G_TOKEN_INT:
- {
- /* Negative integer */
- uint64_t int_val = state->scanner->value.v_int64;
-
- if (int_val > (((uint64_t) INT64_MAX) + 1)) {
- g_string_append_printf(state->ini_error,
- "Integer value -%" PRIu64 " is outside the range of a 64-bit signed integer\n",
- int_val);
- } else {
- value = bt_value_signed_integer_create_init(
- -((int64_t) int_val));
- }
-
- break;
- }
- case G_TOKEN_FLOAT:
- /* Negative floating point number */
- value = bt_value_real_create_init(-state->scanner->value.v_float);
- break;
- default:
- ini_append_error_expecting(state, state->scanner, "value");
- break;
- }
-
- return value;
-}
-
-static bt_value *ini_parse_value(struct ini_parsing_state *state);
-
-/*
- * Parse the current and following tokens as an array. Arrays are formatted as
- * an opening `[`, a list of comma-separated values and a closing `]`. For
- * convenience we support an optional trailing comma, after the last value.
- *
- * The current token of the parser must be the opening square bracket of the
- * array.
- */
-static
-bt_value *ini_parse_array(struct ini_parsing_state *state)
-{
- /* The [ character must have already been ingested. */
- BT_ASSERT(g_scanner_cur_token(state->scanner) == G_TOKEN_CHAR);
- BT_ASSERT(g_scanner_cur_value(state->scanner).v_char == '[');
-
- bt_value *array_value;
- GTokenType token_type;
-
- array_value = bt_value_array_create ();
- token_type = g_scanner_get_next_token(state->scanner);
-
- /* While the current token is not a ]... */
- while (!(token_type == G_TOKEN_CHAR && g_scanner_cur_value(state->scanner).v_char == ']')) {
- /* Parse the item... */
- bt_value *item_value;
- bt_value_status status;
-
- item_value = ini_parse_value(state);
- if (!item_value) {
- goto error;
- }
-
- /* ... and add it to the result array. */
- status = bt_value_array_append_element(array_value, item_value);
- BT_VALUE_PUT_REF_AND_RESET(item_value);
-
- if (status != BT_VALUE_STATUS_OK) {
- goto error;
- }
-
- /*
- * Ingest the token following the value, it should be either a
- * comma or closing square brace.
- */
- token_type = g_scanner_get_next_token(state->scanner);
-
- if (token_type == G_TOKEN_CHAR && g_scanner_cur_value(state->scanner).v_char == ',') {
- /*
- * Ingest the token following the comma. If it happens
- * to be a closing square bracket, we'll exit the loop
- * and we are done (we allow trailing commas).
- * Otherwise, we are ready for the next ini_parse_value call.
- */
- token_type = g_scanner_get_next_token(state->scanner);
- } else if (token_type != G_TOKEN_CHAR || g_scanner_cur_value(state->scanner).v_char != ']') {
- ini_append_error_expecting(state, state->scanner, ", or ]");
- goto error;
- }
- }
-
- goto end;
-
-error:
- BT_VALUE_PUT_REF_AND_RESET(array_value);
-
-end:
- return array_value;
-}
-
-/*
- * Parse the current token (and the following ones if needed) as a value, return
- * it as a bt_value.
- */
-static
-bt_value *ini_parse_value(struct ini_parsing_state *state)
-{
- bt_value *value = NULL;
- GTokenType token_type = state->scanner->token;
-
- switch (token_type) {
- case G_TOKEN_CHAR:
- if (state->scanner->value.v_char == '-') {
- /* Negative number */
- value = ini_parse_neg_number(state);
- } else if (state->scanner->value.v_char == '+') {
- /* Unsigned integer */
- value = ini_parse_uint(state);
- } else if (state->scanner->value.v_char == '[') {
- /* Array */
- value = ini_parse_array(state);
- } else {
- ini_append_error_expecting(state, state->scanner, "value");
- }
- break;
- case G_TOKEN_INT:
- {
- /* Positive, signed integer */
- uint64_t int_val = state->scanner->value.v_int64;
-
- if (int_val > INT64_MAX) {
- g_string_append_printf(state->ini_error,
- "Integer value %" PRIu64 " is outside the range of a 64-bit signed integer\n",
- int_val);
- } else {
- value = bt_value_signed_integer_create_init(
- (int64_t) int_val);
- }
- break;
- }
- case G_TOKEN_FLOAT:
- /* Positive floating point number */
- value = bt_value_real_create_init(state->scanner->value.v_float);
- break;
- case G_TOKEN_STRING:
- /* Quoted string */
- value = bt_value_string_create_init(state->scanner->value.v_string);
- break;
- case G_TOKEN_IDENTIFIER:
- {
- /*
- * Using symbols would be appropriate here,
- * but said symbols are allowed as map key,
- * so it's easier to consider everything an
- * identifier.
- *
- * If one of the known symbols is not
- * recognized here, then fall back to creating
- * a string value.
- */
- const char *id = state->scanner->value.v_identifier;
-
- if (!strcmp(id, "null") || !strcmp(id, "NULL") ||
- !strcmp(id, "nul")) {
- value = bt_value_null;
- } else if (!strcmp(id, "true") || !strcmp(id, "TRUE") ||
- !strcmp(id, "yes") ||
- !strcmp(id, "YES")) {
- value = bt_value_bool_create_init(true);
- } else if (!strcmp(id, "false") ||
- !strcmp(id, "FALSE") ||
- !strcmp(id, "no") ||
- !strcmp(id, "NO")) {
- value = bt_value_bool_create_init(false);
- } else {
- value = bt_value_string_create_init(id);
- }
- break;
- }
- default:
- /* Unset value variable will trigger the error */
- ini_append_error_expecting(state, state->scanner, "value");
- break;
- }
-
- return value;
-}
-
-static
-int ini_handle_state(struct ini_parsing_state *state)
-{
- int ret = 0;
- GTokenType token_type;
- bt_value *value = NULL;
-
- token_type = g_scanner_get_next_token(state->scanner);
- if (token_type == G_TOKEN_EOF) {
- if (state->expecting != INI_EXPECT_COMMA) {
- switch (state->expecting) {
- case INI_EXPECT_EQUAL:
- ini_append_error_expecting(state,
- state->scanner, "'='");
- break;
- case INI_EXPECT_VALUE:
- ini_append_error_expecting(state,
- state->scanner, "value");
- break;
- case INI_EXPECT_MAP_KEY:
- ini_append_error_expecting(state,
- state->scanner, "unquoted map key");
- break;
- default:
- break;
- }
- goto error;
- }
-
- /* We're done! */
- ret = 1;
- goto success;
- }
-
- switch (state->expecting) {
- case INI_EXPECT_MAP_KEY:
- if (token_type != G_TOKEN_IDENTIFIER) {
- ini_append_error_expecting(state, state->scanner,
- "unquoted map key");
- goto error;
- }
-
- free(state->last_map_key);
- state->last_map_key =
- strdup(state->scanner->value.v_identifier);
- if (!state->last_map_key) {
- g_string_append(state->ini_error,
- "Out of memory\n");
- goto error;
- }
-
- if (bt_value_map_has_entry(state->params,
- state->last_map_key)) {
- g_string_append_printf(state->ini_error,
- "Duplicate parameter key: `%s`\n",
- state->last_map_key);
- goto error;
- }
-
- state->expecting = INI_EXPECT_EQUAL;
- goto success;
- case INI_EXPECT_EQUAL:
- if (token_type != G_TOKEN_CHAR) {
- ini_append_error_expecting(state,
- state->scanner, "'='");
- goto error;
- }
-
- if (state->scanner->value.v_char != '=') {
- ini_append_error_expecting(state,
- state->scanner, "'='");
- goto error;
- }
-
- state->expecting = INI_EXPECT_VALUE;
- goto success;
- case INI_EXPECT_VALUE:
- {
- value = ini_parse_value(state);
- if (!value) {
- goto error;
- }
-
- state->expecting = INI_EXPECT_COMMA;
- goto success;
- }
- case INI_EXPECT_COMMA:
- if (token_type != G_TOKEN_CHAR) {
- ini_append_error_expecting(state,
- state->scanner, "','");
- goto error;
- }
-
- if (state->scanner->value.v_char != ',') {
- ini_append_error_expecting(state,
- state->scanner, "','");
- goto error;
- }
-
- state->expecting = INI_EXPECT_MAP_KEY;
- goto success;
- default:
- abort();
- }
-
-error:
- ret = -1;
- goto end;
-
-success:
- if (value) {
- if (bt_value_map_insert_entry(state->params,
- state->last_map_key, value)) {
- /* Only override return value on error */
- ret = -1;
- }
- }
-
-end:
- BT_VALUE_PUT_REF_AND_RESET(value);
- return ret;
-}
-
-/*
- * Converts an INI-style argument to an equivalent map value object.
- *
- * Return value is owned by the caller.
- */
-static
-bt_value *bt_value_from_ini(const char *arg,
- GString *ini_error)
-{
- /* Lexical scanner configuration */
- GScannerConfig scanner_config = {
- /* Skip whitespaces */
- .cset_skip_characters = " \t\n",
-
- /* Identifier syntax is: [a-zA-Z_][a-zA-Z0-9_.:-]* */
- .cset_identifier_first =
- G_CSET_a_2_z
- "_"
- G_CSET_A_2_Z,
- .cset_identifier_nth =
- G_CSET_a_2_z
- "_0123456789-.:"
- G_CSET_A_2_Z,
-
- /* "hello" and "Hello" two different keys */
- .case_sensitive = TRUE,
-
- /* No comments */
- .cpair_comment_single = NULL,
- .skip_comment_multi = TRUE,
- .skip_comment_single = TRUE,
- .scan_comment_multi = FALSE,
-
- /*
- * Do scan identifiers, including 1-char identifiers,
- * but NULL is a normal identifier.
- */
- .scan_identifier = TRUE,
- .scan_identifier_1char = TRUE,
- .scan_identifier_NULL = FALSE,
-
- /*
- * No specific symbols: null and boolean "symbols" are
- * scanned as plain identifiers.
- */
- .scan_symbols = FALSE,
- .symbol_2_token = FALSE,
- .scope_0_fallback = FALSE,
-
- /*
- * Scan "0b"-, "0"-, and "0x"-prefixed integers, but not
- * integers prefixed with "$".
- */
- .scan_binary = TRUE,
- .scan_octal = TRUE,
- .scan_float = TRUE,
- .scan_hex = TRUE,
- .scan_hex_dollar = FALSE,
-
- /* Convert scanned numbers to integer tokens */
- .numbers_2_int = TRUE,
-
- /* Support both integers and floating-point numbers */
- .int_2_float = FALSE,
-
- /* Scan integers as 64-bit signed integers */
- .store_int64 = TRUE,
-
- /* Only scan double-quoted strings */
- .scan_string_sq = FALSE,
- .scan_string_dq = TRUE,
-
- /* Do not converter identifiers to string tokens */
- .identifier_2_string = FALSE,
-
- /* Scan characters as G_TOKEN_CHAR token */
- .char_2_token = FALSE,
- };
- struct ini_parsing_state state = {
- .scanner = NULL,
- .params = NULL,
- .expecting = INI_EXPECT_MAP_KEY,
- .arg = arg,
- .ini_error = ini_error,
- };
-
- state.params = bt_value_map_create();
- if (!state.params) {
- goto error;
- }
-
- state.scanner = g_scanner_new(&scanner_config);
- if (!state.scanner) {
- goto error;
- }
-
- /* Let the scan begin */
- g_scanner_input_text(state.scanner, arg, strlen(arg));
-
- while (true) {
- int ret = ini_handle_state(&state);
-
- if (ret < 0) {
- /* Error */
- goto error;
- } else if (ret > 0) {
- /* Done */
- break;
- }
- }
-
- goto end;
-
-error:
- BT_VALUE_PUT_REF_AND_RESET(state.params);
-
-end:
- if (state.scanner) {
- g_scanner_destroy(state.scanner);
- }
-
- free(state.last_map_key);
- return state.params;
-}
-
-/*
- * Returns the parameters map value object from a command-line
- * parameter option's argument.
- *
- * Return value is owned by the caller.
- */
-static
-bt_value *bt_value_from_arg(const char *arg)
-{
- bt_value *params = NULL;
- GString *ini_error = NULL;
-
- ini_error = g_string_new(NULL);
- if (!ini_error) {
- print_err_oom();
- goto end;
- }
-
- /* Try INI-style parsing */
- params = bt_value_from_ini(arg, ini_error);
- if (!params) {
- printf_err("%s", ini_error->str);
- goto end;
- }
-
-end:
- if (ini_error) {
- g_string_free(ini_error, TRUE);
- }
-
- return params;
-}
+#define BT_CLI_LOGE_APPEND_CAUSE_OOM() BT_CLI_LOGE_APPEND_CAUSE("Out of memory.")
/*
* Returns the plugin name, component class name, component class type,
BT_ASSERT(comp_cls_type);
if (!bt_common_string_is_printable(arg)) {
- printf_err("Argument contains a non-printable character\n");
+ BT_CLI_LOGE_APPEND_CAUSE("Argument contains a non-printable character.");
goto error;
}
/* Parse the component class type */
gs_comp_cls_type = bt_common_string_until(at, ".:\\", ".", &end_pos);
if (!gs_comp_cls_type || at[end_pos] == '\0') {
- printf_err("Missing component class type (`source`, `filter`, or `sink`)\n");
+ BT_CLI_LOGE_APPEND_CAUSE("Missing component class type (`source`, `filter`, or `sink`).");
goto error;
}
} else if (strcmp(gs_comp_cls_type->str, "sink") == 0) {
*comp_cls_type = BT_COMPONENT_CLASS_TYPE_SINK;
} else {
- printf_err("Unknown component class type: `%s`\n",
+ BT_CLI_LOGE_APPEND_CAUSE("Unknown component class type: `%s`.",
gs_comp_cls_type->str);
goto error;
}
/* Parse the plugin name */
gs_plugin = bt_common_string_until(at, ".:\\", ".", &end_pos);
if (!gs_plugin || gs_plugin->len == 0 || at[end_pos] == '\0') {
- printf_err("Missing plugin or component class name\n");
+ BT_CLI_LOGE_APPEND_CAUSE("Missing plugin or component class name.");
goto error;
}
/* Parse the component class name */
gs_comp_cls = bt_common_string_until(at, ".:\\", ".", &end_pos);
if (!gs_comp_cls || gs_comp_cls->len == 0) {
- printf_err("Missing component class name\n");
+ BT_CLI_LOGE_APPEND_CAUSE("Missing component class name.");
goto error;
}
cfg_component = g_new0(struct bt_config_component, 1);
if (!cfg_component) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
cfg_component->type = type;
cfg_component->plugin_name = g_string_new(plugin_name);
if (!cfg_component->plugin_name) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
cfg_component->comp_cls_name = g_string_new(comp_cls_name);
if (!cfg_component->comp_cls_name) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
cfg_component->instance_name = g_string_new(NULL);
if (!cfg_component->instance_name) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
/* Start with empty parameters */
cfg_component->params = bt_value_map_create();
if (!cfg_component->params) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
return;
}
- for (at = list; at != NULL; at = g_list_next(at)) {
+ for (at = list; at; at = g_list_next(at)) {
g_string_free(at->data, TRUE);
}
scanner = g_scanner_new(&scanner_config);
if (!scanner) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
}
return scanner;
names = bt_value_array_create();
if (!names) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
{
const char *identifier = scanner->value.v_identifier;
- if (!strcmp(identifier, "payload") ||
- !strcmp(identifier, "args") ||
- !strcmp(identifier, "arg")) {
+ if (strcmp(identifier, "payload") == 0 ||
+ strcmp(identifier, "args") == 0 ||
+ strcmp(identifier, "arg") == 0) {
found_item = true;
if (bt_value_array_append_string_element(names,
"payload")) {
goto error;
}
- } else if (!strcmp(identifier, "context") ||
- !strcmp(identifier, "ctx")) {
+ } else if (strcmp(identifier, "context") == 0 ||
+ strcmp(identifier, "ctx") == 0) {
found_item = true;
if (bt_value_array_append_string_element(names,
"context")) {
goto error;
}
- } else if (!strcmp(identifier, "scope") ||
- !strcmp(identifier, "header")) {
+ } else if (strcmp(identifier, "scope") == 0 ||
+ strcmp(identifier, "header") == 0) {
found_item = true;
if (bt_value_array_append_string_element(names,
identifier)) {
goto error;
}
- } else if (!strcmp(identifier, "all")) {
+ } else if (strcmp(identifier, "all") == 0) {
found_all = true;
if (bt_value_array_append_string_element(names,
identifier)) {
goto error;
}
- } else if (!strcmp(identifier, "none")) {
+ } else if (strcmp(identifier, "none") == 0) {
found_none = true;
if (bt_value_array_append_string_element(names,
identifier)) {
goto error;
}
} else {
- printf_err("Unknown name: `%s`\n",
+ BT_CLI_LOGE_APPEND_CAUSE("Unknown name: `%s`.",
identifier);
goto error;
}
end:
if (found_none && found_all) {
- printf_err("Only either `all` or `none` can be specified in the list given to the --names option, but not both.\n");
+ BT_CLI_LOGE_APPEND_CAUSE("Only either `all` or `none` can be specified in the list given to the --names option, but not both.");
goto error;
}
/*
fields = bt_value_array_create();
if (!fields) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
{
const char *identifier = scanner->value.v_identifier;
- if (!strcmp(identifier, "trace") ||
- !strcmp(identifier, "trace:hostname") ||
- !strcmp(identifier, "trace:domain") ||
- !strcmp(identifier, "trace:procname") ||
- !strcmp(identifier, "trace:vpid") ||
- !strcmp(identifier, "loglevel") ||
- !strcmp(identifier, "emf") ||
- !strcmp(identifier, "callsite") ||
- !strcmp(identifier, "all")) {
+ if (strcmp(identifier, "trace") == 0 ||
+ strcmp(identifier, "trace:hostname") == 0 ||
+ strcmp(identifier, "trace:domain") == 0 ||
+ strcmp(identifier, "trace:procname") == 0 ||
+ strcmp(identifier, "trace:vpid") == 0 ||
+ strcmp(identifier, "loglevel") == 0 ||
+ strcmp(identifier, "emf") == 0 ||
+ strcmp(identifier, "callsite") == 0 ||
+ strcmp(identifier, "all") == 0) {
if (bt_value_array_append_string_element(fields,
identifier)) {
goto error;
}
} else {
- printf_err("Unknown field: `%s`\n",
+ BT_CLI_LOGE_APPEND_CAUSE("Unknown field: `%s`.",
identifier);
goto error;
}
tmpstr = g_string_new(NULL);
if (!tmpstr) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
ret = -1;
goto end;
}
default_value = g_string_new(NULL);
if (!default_value) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
ret = -1;
goto end;
}
bool is_default = false;
if (!str_obj) {
- printf_err("Unexpected error\n");
+ BT_CLI_LOGE_APPEND_CAUSE("Unexpected error.");
ret = -1;
goto end;
}
g_string_append(tmpstr, "-");
/* Special-case for "all" and "none". */
- if (!strcmp(suffix, "all")) {
+ if (strcmp(suffix, "all") == 0) {
is_default = true;
g_string_assign(default_value, "show");
- } else if (!strcmp(suffix, "none")) {
+ } else if (strcmp(suffix, "none") == 0) {
is_default = true;
g_string_assign(default_value, "hide");
}
return ret;
}
-/* popt options */
+/* argpar options */
enum {
OPT_NONE = 0,
OPT_BASE_PARAMS,
OPT_INPUT_FORMAT,
OPT_LIST,
OPT_LOG_LEVEL,
- OPT_NAME,
OPT_NAMES,
OPT_NO_DELTA,
OPT_OMIT_HOME_PLUGIN_PATH,
OPT_OUTPUT,
OPT_OUTPUT_FORMAT,
OPT_PARAMS,
- OPT_PATH,
OPT_PLUGIN_PATH,
OPT_RESET_BASE_PARAMS,
OPT_RETRY_DURATION,
OPT_RUN_ARGS_0,
OPT_STREAM_INTERSECTION,
OPT_TIMERANGE,
- OPT_URL,
OPT_VERBOSE,
+ OPT_VERSION,
};
enum bt_config_component_dest {
int ret = 0;
if (cfg_comp->instance_name->len == 0) {
- printf_err("Found an unnamed component\n");
+ BT_CLI_LOGE_APPEND_CAUSE("Found an unnamed component.");
ret = -1;
goto end;
}
if (bt_value_map_has_entry(instance_names,
cfg_comp->instance_name->str)) {
- printf_err("Duplicate component instance name:\n %s\n",
+ BT_CLI_LOGE_APPEND_CAUSE("Duplicate component instance name:\n %s",
cfg_comp->instance_name->str);
ret = -1;
goto end;
if (bt_value_map_insert_entry(instance_names,
cfg_comp->instance_name->str, bt_value_null)) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
ret = -1;
goto end;
}
end:
if (ret) {
- printf_err("Cannot append plugin paths from BABELTRACE_PLUGIN_PATH\n");
+ BT_CLI_LOGE_APPEND_CAUSE("Cannot append plugin paths from BABELTRACE_PLUGIN_PATH.");
}
return ret;
if (bt_common_is_setuid_setgid()) {
BT_LOGI_STR("Skipping non-system plugin paths for setuid/setgid binary.");
} else {
- char *home_plugin_dir =
- bt_common_get_home_plugin_path();
+ char *home_plugin_dir = bt_common_get_home_plugin_path(
+ BT_LOG_OUTPUT_LEVEL);
if (home_plugin_dir) {
ret = bt_config_append_plugin_paths(
free(home_plugin_dir);
if (ret) {
- printf_err("Invalid home plugin path\n");
+ BT_CLI_LOGE_APPEND_CAUSE("Invalid home plugin path.");
goto error;
}
}
if (!omit_system_plugin_path) {
if (bt_config_append_plugin_paths(plugin_paths,
bt_common_get_system_plugin_path())) {
- printf_err("Invalid system plugin path\n");
+ BT_CLI_LOGE_APPEND_CAUSE("Invalid system plugin path.");
goto error;
}
}
return 0;
error:
- printf_err("Cannot append home and system plugin paths\n");
+ BT_CLI_LOGE_APPEND_CAUSE("Cannot append home and system plugin paths.");
return -1;
}
/* Create config */
cfg = g_new0(struct bt_config, 1);
if (!cfg) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
} else {
cfg->plugin_paths = bt_value_array_create();
if (!cfg->plugin_paths) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
}
cfg->cmd_data.run.sources = g_ptr_array_new_with_free_func(
(GDestroyNotify) bt_object_put_ref);
if (!cfg->cmd_data.run.sources) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
cfg->cmd_data.run.filters = g_ptr_array_new_with_free_func(
(GDestroyNotify) bt_object_put_ref);
if (!cfg->cmd_data.run.filters) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
cfg->cmd_data.run.sinks = g_ptr_array_new_with_free_func(
(GDestroyNotify) bt_object_put_ref);
if (!cfg->cmd_data.run.sinks) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
cfg->cmd_data.run.connections = g_ptr_array_new_with_free_func(
(GDestroyNotify) bt_config_connection_destroy);
if (!cfg->cmd_data.run.connections) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
cfg->cmd_data.query.object = g_string_new(NULL);
if (!cfg->cmd_data.query.object) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
cfg->cmd_data.print_ctf_metadata.path = g_string_new(NULL);
if (!cfg->cmd_data.print_ctf_metadata.path) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
cfg->cmd_data.print_ctf_metadata.output_path = g_string_new(NULL);
if (!cfg->cmd_data.print_ctf_metadata.output_path) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
cfg->cmd_data.print_lttng_live_sessions.url = g_string_new(NULL);
if (!cfg->cmd_data.print_lttng_live_sessions.url) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
cfg->cmd_data.print_lttng_live_sessions.output_path =
g_string_new(NULL);
if (!cfg->cmd_data.print_lttng_live_sessions.output_path) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
}
if (bt_config_append_plugin_paths(plugin_paths, arg)) {
- printf_err("Invalid --plugin-path option's argument:\n %s\n",
+ BT_CLI_LOGE_APPEND_CAUSE("Invalid --plugin-path option's argument:\n %s",
arg);
ret = -1;
goto end;
fprintf(fp, "babeltrace2 from a shell.\n");
}
+static
+bool help_option_is_specified(
+ const struct bt_argpar_parse_ret *argpar_parse_ret)
+{
+ int i;
+ bool specified = false;
+
+ for (i = 0; i < argpar_parse_ret->items->len; i++) {
+ struct bt_argpar_item *argpar_item =
+ g_ptr_array_index(argpar_parse_ret->items, i);
+ struct bt_argpar_item_opt *argpar_item_opt;
+
+ if (argpar_item->type != BT_ARGPAR_ITEM_TYPE_OPT) {
+ continue;
+ }
+
+ argpar_item_opt = (struct bt_argpar_item_opt *) argpar_item;
+ if (argpar_item_opt->descr->id == OPT_HELP) {
+ specified = true;
+ break;
+ }
+ }
+
+ return specified;
+}
/*
* Prints the help command usage.
}
static
-struct poptOption help_long_options[] = {
- /* longName, shortName, argInfo, argPtr, value, descrip, argDesc */
- { "help", 'h', POPT_ARG_NONE, NULL, OPT_HELP, NULL, NULL },
- { "omit-home-plugin-path", '\0', POPT_ARG_NONE, NULL, OPT_OMIT_HOME_PLUGIN_PATH, NULL, NULL },
- { "omit-system-plugin-path", '\0', POPT_ARG_NONE, NULL, OPT_OMIT_SYSTEM_PLUGIN_PATH, NULL, NULL },
- { "plugin-path", '\0', POPT_ARG_STRING, NULL, OPT_PLUGIN_PATH, NULL, NULL },
- { NULL, 0, '\0', NULL, 0, NULL, NULL },
+const struct bt_argpar_opt_descr help_options[] = {
+ /* id, short_name, long_name, with_arg */
+ { OPT_HELP, 'h', "help", false },
+ { OPT_OMIT_HOME_PLUGIN_PATH, '\0', "omit-home-plugin-path", false },
+ { OPT_OMIT_SYSTEM_PLUGIN_PATH, '\0', "omit-system-plugin-path", false },
+ { OPT_PLUGIN_PATH, '\0', "plugin-path", true },
+ BT_ARGPAR_OPT_DESCR_SENTINEL
};
/*
bool force_omit_home_plugin_path,
const bt_value *initial_plugin_paths, int default_log_level)
{
- poptContext pc = NULL;
- char *arg = NULL;
- int opt;
- int ret;
+ int ret, i;
struct bt_config *cfg = NULL;
- const char *leftover;
+ const char *leftover = NULL;
char *plugin_name = NULL, *comp_cls_name = NULL;
+ struct bt_argpar_parse_ret argpar_parse_ret;
*retcode = 0;
cfg = bt_config_help_create(initial_plugin_paths, default_log_level);
}
/* Parse options */
- pc = poptGetContext(NULL, argc, (const char **) argv,
- help_long_options, 0);
- if (!pc) {
- printf_err("Cannot get popt context\n");
+ argpar_parse_ret = bt_argpar_parse(argc, argv, help_options, true);
+ if (argpar_parse_ret.error) {
+ BT_CLI_LOGE_APPEND_CAUSE(
+ "While parsing `help` command's command-line arguments: %s",
+ argpar_parse_ret.error->str);
goto error;
}
- poptReadDefaultConfig(pc, 0);
+ if (help_option_is_specified(&argpar_parse_ret)) {
+ print_help_usage(stdout);
+ *retcode = -1;
+ BT_OBJECT_PUT_REF_AND_RESET(cfg);
+ goto end;
+ }
+
+ for (i = 0; i < argpar_parse_ret.items->len; i++) {
+ struct bt_argpar_item *argpar_item =
+ g_ptr_array_index(argpar_parse_ret.items, i);
- while ((opt = poptGetNextOpt(pc)) > 0) {
- arg = poptGetOptArg(pc);
+ if (argpar_item->type == BT_ARGPAR_ITEM_TYPE_OPT) {
+ struct bt_argpar_item_opt *argpar_item_opt;
+ const char *arg;
+ argpar_item_opt = (struct bt_argpar_item_opt *) argpar_item;
+ arg = argpar_item_opt->arg;
- switch (opt) {
- case OPT_PLUGIN_PATH:
- if (bt_config_append_plugin_paths_check_setuid_setgid(
- cfg->plugin_paths, arg)) {
+ switch (argpar_item_opt->descr->id) {
+ case OPT_PLUGIN_PATH:
+ if (bt_config_append_plugin_paths_check_setuid_setgid(
+ cfg->plugin_paths, arg)) {
+ goto error;
+ }
+ break;
+ case OPT_OMIT_SYSTEM_PLUGIN_PATH:
+ cfg->omit_system_plugin_path = true;
+ break;
+ case OPT_OMIT_HOME_PLUGIN_PATH:
+ cfg->omit_home_plugin_path = true;
+ break;
+ default:
+ BT_CLI_LOGE_APPEND_CAUSE("Unknown command-line option specified (option code %d).",
+ argpar_item_opt->descr->id);
goto error;
}
- break;
- case OPT_OMIT_SYSTEM_PLUGIN_PATH:
- cfg->omit_system_plugin_path = true;
- break;
- case OPT_OMIT_HOME_PLUGIN_PATH:
- cfg->omit_home_plugin_path = true;
- break;
- case OPT_HELP:
- print_help_usage(stdout);
- *retcode = -1;
- BT_OBJECT_PUT_REF_AND_RESET(cfg);
- goto end;
- default:
- printf_err("Unknown command-line option specified (option code %d)\n",
- opt);
- goto error;
- }
+ } else {
+ struct bt_argpar_item_non_opt *argpar_item_non_opt
+ = (struct bt_argpar_item_non_opt *) argpar_item;
- free(arg);
- arg = NULL;
- }
+ if (leftover) {
+ BT_CLI_LOGE_APPEND_CAUSE("Extraneous command-line argument specified to `help` command: `%s`.",
+ argpar_item_non_opt->arg);
+ goto error;
+ }
- /* Check for option parsing error */
- if (opt < -1) {
- printf_err("While parsing command-line options, at option %s: %s\n",
- poptBadOption(pc, 0), poptStrerror(opt));
- goto error;
+ leftover = argpar_item_non_opt->arg;
+ }
}
- leftover = poptGetArg(pc);
if (leftover) {
plugin_comp_cls_names(leftover, NULL,
&plugin_name, &comp_cls_name,
g_free(plugin_name);
g_free(comp_cls_name);
- if (pc) {
- poptFreeContext(pc);
- }
+ bt_argpar_parse_ret_fini(&argpar_parse_ret);
- free(arg);
return cfg;
}
}
static
-struct poptOption query_long_options[] = {
- /* longName, shortName, argInfo, argPtr, value, descrip, argDesc */
- { "help", 'h', POPT_ARG_NONE, NULL, OPT_HELP, NULL, NULL },
- { "omit-home-plugin-path", '\0', POPT_ARG_NONE, NULL, OPT_OMIT_HOME_PLUGIN_PATH, NULL, NULL },
- { "omit-system-plugin-path", '\0', POPT_ARG_NONE, NULL, OPT_OMIT_SYSTEM_PLUGIN_PATH, NULL, NULL },
- { "params", 'p', POPT_ARG_STRING, NULL, OPT_PARAMS, NULL, NULL },
- { "plugin-path", '\0', POPT_ARG_STRING, NULL, OPT_PLUGIN_PATH, NULL, NULL },
- { NULL, 0, '\0', NULL, 0, NULL, NULL },
+const struct bt_argpar_opt_descr query_options[] = {
+ /* id, short_name, long_name, with_arg */
+ { OPT_HELP, 'h', "help", false },
+ { OPT_OMIT_HOME_PLUGIN_PATH, '\0', "omit-home-plugin-path", false },
+ { OPT_OMIT_SYSTEM_PLUGIN_PATH, '\0', "omit-system-plugin-path", false },
+ { OPT_PARAMS, 'p', "params", true },
+ { OPT_PLUGIN_PATH, '\0', "plugin-path", true },
+ BT_ARGPAR_OPT_DESCR_SENTINEL
};
/*
const bt_value *initial_plugin_paths,
int default_log_level)
{
- poptContext pc = NULL;
- char *arg = NULL;
- int opt;
- int ret;
+ int ret, i;
struct bt_config *cfg = NULL;
- const char *leftover;
+ const char *component_class_spec = NULL;
+ const char *query_object = NULL;
bt_value *params;
+ GString *error_str = NULL;
+ struct bt_argpar_parse_ret argpar_parse_ret;
params = bt_value_null;
bt_value_get_ref(bt_value_null);
goto error;
}
+ error_str = g_string_new(NULL);
+ if (!error_str) {
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
+ goto error;
+ }
+
cfg->omit_system_plugin_path = force_omit_system_plugin_path;
cfg->omit_home_plugin_path = force_omit_home_plugin_path;
ret = append_env_var_plugin_paths(cfg->plugin_paths);
}
/* Parse options */
- pc = poptGetContext(NULL, argc, (const char **) argv,
- query_long_options, 0);
- if (!pc) {
- printf_err("Cannot get popt context\n");
+ argpar_parse_ret = bt_argpar_parse(argc, argv, query_options, true);
+ if (argpar_parse_ret.error) {
+ BT_CLI_LOGE_APPEND_CAUSE(
+ "While parsing `query` command's command-line arguments: %s",
+ argpar_parse_ret.error->str);
goto error;
}
- poptReadDefaultConfig(pc, 0);
+ if (help_option_is_specified(&argpar_parse_ret)) {
+ print_query_usage(stdout);
+ *retcode = -1;
+ BT_OBJECT_PUT_REF_AND_RESET(cfg);
+ goto end;
+ }
- while ((opt = poptGetNextOpt(pc)) > 0) {
- arg = poptGetOptArg(pc);
+ for (i = 0; i < argpar_parse_ret.items->len; i++) {
+ struct bt_argpar_item *argpar_item =
+ g_ptr_array_index(argpar_parse_ret.items, i);
- switch (opt) {
- case OPT_PLUGIN_PATH:
- if (bt_config_append_plugin_paths_check_setuid_setgid(
- cfg->plugin_paths, arg)) {
+ if (argpar_item->type == BT_ARGPAR_ITEM_TYPE_OPT) {
+ struct bt_argpar_item_opt *argpar_item_opt =
+ (struct bt_argpar_item_opt *) argpar_item;
+ const char *arg = argpar_item_opt->arg;
+
+ switch (argpar_item_opt->descr->id) {
+ case OPT_PLUGIN_PATH:
+ if (bt_config_append_plugin_paths_check_setuid_setgid(
+ cfg->plugin_paths, arg)) {
+ goto error;
+ }
+ break;
+ case OPT_OMIT_SYSTEM_PLUGIN_PATH:
+ cfg->omit_system_plugin_path = true;
+ break;
+ case OPT_OMIT_HOME_PLUGIN_PATH:
+ cfg->omit_home_plugin_path = true;
+ break;
+ case OPT_PARAMS:
+ {
+ bt_value_put_ref(params);
+ params = cli_value_from_arg(arg, error_str);
+ if (!params) {
+ BT_CLI_LOGE_APPEND_CAUSE("Invalid format for --params option's argument:\n %s",
+ error_str->str);
+ goto error;
+ }
+ break;
+ }
+ default:
+ BT_CLI_LOGE_APPEND_CAUSE("Unknown command-line option specified (option code %d).",
+ argpar_item_opt->descr->id);
goto error;
}
- break;
- case OPT_OMIT_SYSTEM_PLUGIN_PATH:
- cfg->omit_system_plugin_path = true;
- break;
- case OPT_OMIT_HOME_PLUGIN_PATH:
- cfg->omit_home_plugin_path = true;
- break;
- case OPT_PARAMS:
- {
- bt_value_put_ref(params);
- params = bt_value_from_arg(arg);
- if (!params) {
- printf_err("Invalid format for --params option's argument:\n %s\n",
- arg);
+ } else {
+ struct bt_argpar_item_non_opt *argpar_item_non_opt
+ = (struct bt_argpar_item_non_opt *) argpar_item;
+
+ /*
+ * We need exactly two leftover arguments which are the
+ * mandatory component class specification and query object.
+ */
+ if (!component_class_spec) {
+ component_class_spec = argpar_item_non_opt->arg;
+ } else if (!query_object) {
+ query_object = argpar_item_non_opt->arg;
+ } else {
+ BT_CLI_LOGE_APPEND_CAUSE("Extraneous command-line argument specified to `query` command: `%s`.",
+ argpar_item_non_opt->arg);
goto error;
}
- break;
- }
- case OPT_HELP:
- print_query_usage(stdout);
- *retcode = -1;
- BT_OBJECT_PUT_REF_AND_RESET(cfg);
- goto end;
- default:
- printf_err("Unknown command-line option specified (option code %d)\n",
- opt);
- goto error;
}
-
- free(arg);
- arg = NULL;
- }
-
- /* Check for option parsing error */
- if (opt < -1) {
- printf_err("While parsing command-line options, at option %s: %s\n",
- poptBadOption(pc, 0), poptStrerror(opt));
- goto error;
}
- /*
- * We need exactly two leftover arguments which are the
- * mandatory component class specification and query object.
- */
- leftover = poptGetArg(pc);
- if (leftover) {
- cfg->cmd_data.query.cfg_component =
- bt_config_component_from_arg(leftover,
- default_log_level);
- if (!cfg->cmd_data.query.cfg_component) {
- printf_err("Invalid format for component class specification:\n %s\n",
- leftover);
- goto error;
- }
-
- BT_ASSERT(params);
- BT_OBJECT_MOVE_REF(cfg->cmd_data.query.cfg_component->params,
- params);
- } else {
+ if (!component_class_spec || !query_object) {
print_query_usage(stdout);
*retcode = -1;
BT_OBJECT_PUT_REF_AND_RESET(cfg);
goto end;
}
- leftover = poptGetArg(pc);
- if (leftover) {
- if (strlen(leftover) == 0) {
- printf_err("Invalid empty object\n");
- goto error;
- }
-
- g_string_assign(cfg->cmd_data.query.object, leftover);
- } else {
- print_query_usage(stdout);
- *retcode = -1;
- BT_OBJECT_PUT_REF_AND_RESET(cfg);
- goto end;
+ cfg->cmd_data.query.cfg_component =
+ bt_config_component_from_arg(component_class_spec,
+ default_log_level);
+ if (!cfg->cmd_data.query.cfg_component) {
+ BT_CLI_LOGE_APPEND_CAUSE("Invalid format for component class specification:\n %s",
+ component_class_spec);
+ goto error;
}
- leftover = poptGetArg(pc);
- if (leftover) {
- printf_err("Unexpected argument: %s\n", leftover);
+ BT_ASSERT(params);
+ BT_OBJECT_MOVE_REF(cfg->cmd_data.query.cfg_component->params, params);
+
+ if (strlen(query_object) == 0) {
+ BT_CLI_LOGE_APPEND_CAUSE("Invalid empty object.");
goto error;
}
+ g_string_assign(cfg->cmd_data.query.object, query_object);
+
if (append_home_and_system_plugin_paths_cfg(cfg)) {
goto error;
}
BT_OBJECT_PUT_REF_AND_RESET(cfg);
end:
- if (pc) {
- poptFreeContext(pc);
+ bt_argpar_parse_ret_fini(&argpar_parse_ret);
+
+ if (error_str) {
+ g_string_free(error_str, TRUE);
}
bt_value_put_ref(params);
- free(arg);
return cfg;
}
}
static
-struct poptOption list_plugins_long_options[] = {
- /* longName, shortName, argInfo, argPtr, value, descrip, argDesc */
- { "help", 'h', POPT_ARG_NONE, NULL, OPT_HELP, NULL, NULL },
- { "omit-home-plugin-path", '\0', POPT_ARG_NONE, NULL, OPT_OMIT_HOME_PLUGIN_PATH, NULL, NULL },
- { "omit-system-plugin-path", '\0', POPT_ARG_NONE, NULL, OPT_OMIT_SYSTEM_PLUGIN_PATH, NULL, NULL },
- { "plugin-path", '\0', POPT_ARG_STRING, NULL, OPT_PLUGIN_PATH, NULL, NULL },
- { NULL, 0, '\0', NULL, 0, NULL, NULL },
+const struct bt_argpar_opt_descr list_plugins_options[] = {
+ /* id, short_name, long_name, with_arg */
+ { OPT_HELP, 'h', "help", false },
+ { OPT_OMIT_HOME_PLUGIN_PATH, '\0', "omit-home-plugin-path", false },
+ { OPT_OMIT_SYSTEM_PLUGIN_PATH, '\0', "omit-system-plugin-path", false },
+ { OPT_PLUGIN_PATH, '\0', "plugin-path", true },
+ BT_ARGPAR_OPT_DESCR_SENTINEL
};
/*
bool force_omit_home_plugin_path,
const bt_value *initial_plugin_paths)
{
- poptContext pc = NULL;
- char *arg = NULL;
- int opt;
- int ret;
+ int ret, i;
struct bt_config *cfg = NULL;
- const char *leftover;
+ struct bt_argpar_parse_ret argpar_parse_ret = { 0 };
*retcode = 0;
cfg = bt_config_list_plugins_create(initial_plugin_paths);
}
/* Parse options */
- pc = poptGetContext(NULL, argc, (const char **) argv,
- list_plugins_long_options, 0);
- if (!pc) {
- printf_err("Cannot get popt context\n");
+ argpar_parse_ret = bt_argpar_parse(argc, argv, list_plugins_options, true);
+ if (argpar_parse_ret.error) {
+ BT_CLI_LOGE_APPEND_CAUSE(
+ "While parsing `list-plugins` command's command-line arguments: %s",
+ argpar_parse_ret.error->str);
goto error;
}
- poptReadDefaultConfig(pc, 0);
+ if (help_option_is_specified(&argpar_parse_ret)) {
+ print_list_plugins_usage(stdout);
+ *retcode = -1;
+ BT_OBJECT_PUT_REF_AND_RESET(cfg);
+ goto end;
+ }
+
+ for (i = 0; i < argpar_parse_ret.items->len; i++) {
+ struct bt_argpar_item *argpar_item =
+ g_ptr_array_index(argpar_parse_ret.items, i);
+ struct bt_argpar_item_opt *argpar_item_opt;
+ const char *arg;
+
+ if (argpar_item->type == BT_ARGPAR_ITEM_TYPE_NON_OPT) {
+ struct bt_argpar_item_non_opt *argpar_item_non_opt
+ = (struct bt_argpar_item_non_opt *) argpar_item;
+
+ BT_CLI_LOGE_APPEND_CAUSE("Unexpected argument: `%s`.",
+ argpar_item_non_opt->arg);
+ goto error;
+ }
- while ((opt = poptGetNextOpt(pc)) > 0) {
- arg = poptGetOptArg(pc);
+ argpar_item_opt = (struct bt_argpar_item_opt *) argpar_item;
+ arg = argpar_item_opt->arg;
- switch (opt) {
+ switch (argpar_item_opt->descr->id) {
case OPT_PLUGIN_PATH:
if (bt_config_append_plugin_paths_check_setuid_setgid(
cfg->plugin_paths, arg)) {
case OPT_OMIT_HOME_PLUGIN_PATH:
cfg->omit_home_plugin_path = true;
break;
- case OPT_HELP:
- print_list_plugins_usage(stdout);
- *retcode = -1;
- BT_OBJECT_PUT_REF_AND_RESET(cfg);
- goto end;
default:
- printf_err("Unknown command-line option specified (option code %d)\n",
- opt);
+ BT_CLI_LOGE_APPEND_CAUSE("Unknown command-line option specified (option code %d).",
+ argpar_item_opt->descr->id);
goto error;
}
-
- free(arg);
- arg = NULL;
- }
-
- /* Check for option parsing error */
- if (opt < -1) {
- printf_err("While parsing command-line options, at option %s: %s\n",
- poptBadOption(pc, 0), poptStrerror(opt));
- goto error;
- }
-
- leftover = poptGetArg(pc);
- if (leftover) {
- printf_err("Unexpected argument: %s\n", leftover);
- goto error;
}
if (append_home_and_system_plugin_paths_cfg(cfg)) {
BT_OBJECT_PUT_REF_AND_RESET(cfg);
end:
- if (pc) {
- poptFreeContext(pc);
- }
+ bt_argpar_parse_ret_fini(&argpar_parse_ret);
- free(arg);
return cfg;
}
fprintf(fp, " for all the following components until\n");
fprintf(fp, " --reset-base-params is encountered\n");
fprintf(fp, " (see the expected format of PARAMS below)\n");
- fprintf(fp, " -c, --component=[NAME:]TYPE.PLUGIN.CLS\n");
+ fprintf(fp, " -c, --component=NAME:TYPE.PLUGIN.CLS\n");
fprintf(fp, " Instantiate the component class CLS of type\n");
fprintf(fp, " TYPE (`source`, `filter`, or `sink`) found\n");
fprintf(fp, " in the plugin PLUGIN, add it to the graph,\n");
- fprintf(fp, " and optionally name it NAME (you can also\n");
- fprintf(fp, " specify the name with --name)\n");
+ fprintf(fp, " and name it NAME");
fprintf(fp, " -x, --connect=CONNECTION Connect two created components (see the\n");
fprintf(fp, " expected format of CONNECTION below)\n");
fprintf(fp, " -l, --log-level=LVL Set the log level of the current component to LVL\n");
fprintf(fp, " (`N`, `V`, `D`, `I`, `W`, `E`, or `F`)\n");
- fprintf(fp, " -n, --name=NAME Set the name of the current component\n");
- fprintf(fp, " to NAME (must be unique amongst all the\n");
- fprintf(fp, " names of the created components)\n");
fprintf(fp, " --omit-home-plugin-path Omit home plugins from plugin search path\n");
fprintf(fp, " (~/.local/lib/babeltrace2/plugins)\n");
fprintf(fp, " --omit-system-plugin-path Omit system plugins from plugin search path\n");
fprintf(fp, "\n");
fprintf(fp, "UPSTREAM and DOWNSTREAM are names of the upstream and downstream\n");
fprintf(fp, "components to connect together. You must escape the following characters\n\n");
- fprintf(fp, "with `\\`: `\\`, `.`, and `:`. You can set the name of the current\n");
- fprintf(fp, "component with the --name option.\n");
+ fprintf(fp, "with `\\`: `\\`, `.`, and `:`. You must set the name of the current\n");
+ fprintf(fp, "component using the NAME prefix of the --component option.\n");
fprintf(fp, "\n");
fprintf(fp, "UPSTREAM-PORT and DOWNSTREAM-PORT are optional globbing patterns to\n");
fprintf(fp, "identify the upstream and downstream ports to use for the connection.\n");
bool force_omit_home_plugin_path,
const bt_value *initial_plugin_paths, int default_log_level)
{
- poptContext pc = NULL;
- char *arg = NULL;
struct bt_config_component *cur_cfg_comp = NULL;
- enum bt_config_component_dest cur_cfg_comp_dest =
- BT_CONFIG_COMPONENT_DEST_UNKNOWN;
bt_value *cur_base_params = NULL;
- int opt, ret = 0;
+ int ret = 0;
struct bt_config *cfg = NULL;
bt_value *instance_names = NULL;
bt_value *connection_args = NULL;
char error_buf[256] = { 0 };
long retry_duration = -1;
- bt_value_status status;
- struct poptOption run_long_options[] = {
- { "base-params", 'b', POPT_ARG_STRING, NULL, OPT_BASE_PARAMS, NULL, NULL },
- { "component", 'c', POPT_ARG_STRING, NULL, OPT_COMPONENT, NULL, NULL },
- { "connect", 'x', POPT_ARG_STRING, NULL, OPT_CONNECT, NULL, NULL },
- { "help", 'h', POPT_ARG_NONE, NULL, OPT_HELP, NULL, NULL },
- { "log-level", 'l', POPT_ARG_STRING, NULL, OPT_LOG_LEVEL, NULL, NULL },
- { "name", 'n', POPT_ARG_STRING, NULL, OPT_NAME, NULL, NULL },
- { "omit-home-plugin-path", '\0', POPT_ARG_NONE, NULL, OPT_OMIT_HOME_PLUGIN_PATH, NULL, NULL },
- { "omit-system-plugin-path", '\0', POPT_ARG_NONE, NULL, OPT_OMIT_SYSTEM_PLUGIN_PATH, NULL, NULL },
- { "params", 'p', POPT_ARG_STRING, NULL, OPT_PARAMS, NULL, NULL },
- { "plugin-path", '\0', POPT_ARG_STRING, NULL, OPT_PLUGIN_PATH, NULL, NULL },
- { "reset-base-params", 'r', POPT_ARG_NONE, NULL, OPT_RESET_BASE_PARAMS, NULL, NULL },
- { "retry-duration", '\0', POPT_ARG_LONG, &retry_duration, OPT_RETRY_DURATION, NULL, NULL },
- { NULL, 0, '\0', NULL, 0, NULL, NULL },
+ bt_value_map_extend_status extend_status;
+ GString *error_str = NULL;
+ struct bt_argpar_parse_ret argpar_parse_ret = { 0 };
+ int i;
+
+ static const struct bt_argpar_opt_descr run_options[] = {
+ { OPT_BASE_PARAMS, 'b', "base-params", true },
+ { OPT_COMPONENT, 'c', "component", true },
+ { OPT_CONNECT, 'x', "connect", true },
+ { OPT_HELP, 'h', "help", false },
+ { OPT_LOG_LEVEL, 'l', "log-level", true },
+ { OPT_OMIT_HOME_PLUGIN_PATH, '\0', "omit-home-plugin-path", false },
+ { OPT_OMIT_SYSTEM_PLUGIN_PATH, '\0', "omit-system-plugin-path", false },
+ { OPT_PARAMS, 'p', "params", true },
+ { OPT_PLUGIN_PATH, '\0', "plugin-path", true },
+ { OPT_RESET_BASE_PARAMS, 'r', "reset-base-params", false },
+ { OPT_RETRY_DURATION, '\0', "retry-duration", true },
+ BT_ARGPAR_OPT_DESCR_SENTINEL
};
*retcode = 0;
- if (argc <= 1) {
+ error_str = g_string_new(NULL);
+ if (!error_str) {
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
+ goto error;
+ }
+
+ if (argc < 1) {
print_run_usage(stdout);
*retcode = -1;
goto end;
cfg->omit_home_plugin_path = force_omit_home_plugin_path;
cur_base_params = bt_value_map_create();
if (!cur_base_params) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
instance_names = bt_value_map_create();
if (!instance_names) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
connection_args = bt_value_array_create();
if (!connection_args) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
}
/* Parse options */
- pc = poptGetContext(NULL, argc, (const char **) argv,
- run_long_options, 0);
- if (!pc) {
- printf_err("Cannot get popt context\n");
+ argpar_parse_ret = bt_argpar_parse(argc, argv, run_options, true);
+ if (argpar_parse_ret.error) {
+ BT_CLI_LOGE_APPEND_CAUSE(
+ "While parsing `run` command's command-line arguments: %s",
+ argpar_parse_ret.error->str);
goto error;
}
- poptReadDefaultConfig(pc, 0);
+ if (help_option_is_specified(&argpar_parse_ret)) {
+ print_run_usage(stdout);
+ *retcode = -1;
+ BT_OBJECT_PUT_REF_AND_RESET(cfg);
+ goto end;
+ }
+
+ for (i = 0; i < argpar_parse_ret.items->len; i++) {
+ struct bt_argpar_item *argpar_item =
+ g_ptr_array_index(argpar_parse_ret.items, i);
+ struct bt_argpar_item_opt *argpar_item_opt;
+ const char *arg;
+
+ /* This command does not accept leftover arguments. */
+ if (argpar_item->type == BT_ARGPAR_ITEM_TYPE_NON_OPT) {
+ struct bt_argpar_item_non_opt *argpar_nonopt_item =
+ (struct bt_argpar_item_non_opt *) argpar_item;
+
+ BT_CLI_LOGE_APPEND_CAUSE("Unexpected argument: `%s`",
+ argpar_nonopt_item->arg);
+ goto error;
+ }
- while ((opt = poptGetNextOpt(pc)) > 0) {
- arg = poptGetOptArg(pc);
+ argpar_item_opt = (struct bt_argpar_item_opt *) argpar_item;
+ arg = argpar_item_opt->arg;
- switch (opt) {
+ switch (argpar_item_opt->descr->id) {
case OPT_PLUGIN_PATH:
if (bt_config_append_plugin_paths_check_setuid_setgid(
cfg->plugin_paths, arg)) {
break;
case OPT_COMPONENT:
{
- enum bt_config_component_dest new_dest;
-
- if (cur_cfg_comp) {
- ret = add_run_cfg_comp_check_name(cfg,
- cur_cfg_comp, cur_cfg_comp_dest,
- instance_names);
- BT_OBJECT_PUT_REF_AND_RESET(cur_cfg_comp);
- if (ret) {
- goto error;
- }
- }
+ enum bt_config_component_dest dest;
+ BT_OBJECT_PUT_REF_AND_RESET(cur_cfg_comp);
cur_cfg_comp = bt_config_component_from_arg(arg,
default_log_level);
if (!cur_cfg_comp) {
- printf_err("Invalid format for --component option's argument:\n %s\n",
+ BT_CLI_LOGE_APPEND_CAUSE("Invalid format for --component option's argument:\n %s",
arg);
goto error;
}
switch (cur_cfg_comp->type) {
case BT_COMPONENT_CLASS_TYPE_SOURCE:
- new_dest = BT_CONFIG_COMPONENT_DEST_SOURCE;
+ dest = BT_CONFIG_COMPONENT_DEST_SOURCE;
break;
case BT_COMPONENT_CLASS_TYPE_FILTER:
- new_dest = BT_CONFIG_COMPONENT_DEST_FILTER;
+ dest = BT_CONFIG_COMPONENT_DEST_FILTER;
break;
case BT_COMPONENT_CLASS_TYPE_SINK:
- new_dest = BT_CONFIG_COMPONENT_DEST_SINK;
+ dest = BT_CONFIG_COMPONENT_DEST_SINK;
break;
default:
abort();
BT_ASSERT(cur_base_params);
bt_value_put_ref(cur_cfg_comp->params);
- status = bt_value_copy(cur_base_params,
- &cur_cfg_comp->params);
- if (status != BT_VALUE_STATUS_OK) {
- print_err_oom();
+ if (bt_value_copy(cur_base_params,
+ &cur_cfg_comp->params) < 0) {
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
+ goto error;
+ }
+
+ ret = add_run_cfg_comp_check_name(cfg,
+ cur_cfg_comp, dest,
+ instance_names);
+ if (ret) {
goto error;
}
- cur_cfg_comp_dest = new_dest;
break;
}
case OPT_PARAMS:
bt_value *params_to_set;
if (!cur_cfg_comp) {
- printf_err("Cannot add parameters to unavailable component:\n %s\n",
+ BT_CLI_LOGE_APPEND_CAUSE("Cannot add parameters to unavailable component:\n %s",
arg);
goto error;
}
- params = bt_value_from_arg(arg);
+ params = cli_value_from_arg(arg, error_str);
if (!params) {
- printf_err("Invalid format for --params option's argument:\n %s\n",
- arg);
+ BT_CLI_LOGE_APPEND_CAUSE("Invalid format for --params option's argument:\n %s",
+ error_str->str);
goto error;
}
- status = bt_value_map_extend(cur_cfg_comp->params,
- params, ¶ms_to_set);
+ extend_status = bt_value_map_extend(
+ cur_cfg_comp->params, params, ¶ms_to_set);
BT_VALUE_PUT_REF_AND_RESET(params);
- if (status != BT_VALUE_STATUS_OK) {
- printf_err("Cannot extend current component parameters with --params option's argument:\n %s\n",
+ if (extend_status != BT_VALUE_MAP_EXTEND_STATUS_OK) {
+ BT_CLI_LOGE_APPEND_CAUSE("Cannot extend current component parameters with --params option's argument:\n %s",
arg);
goto error;
}
BT_OBJECT_MOVE_REF(cur_cfg_comp->params, params_to_set);
break;
}
- case OPT_NAME:
- if (!cur_cfg_comp) {
- printf_err("Cannot set the name of unavailable component:\n %s\n",
- arg);
- goto error;
- }
-
- g_string_assign(cur_cfg_comp->instance_name, arg);
- break;
case OPT_LOG_LEVEL:
if (!cur_cfg_comp) {
- printf_err("Cannot set the log level of unavailable component:\n %s\n",
+ BT_CLI_LOGE_APPEND_CAUSE("Cannot set the log level of unavailable component:\n %s",
arg);
goto error;
}
cur_cfg_comp->log_level =
bt_log_get_level_from_string(arg);
if (cur_cfg_comp->log_level < 0) {
- printf_err("Invalid argument for --log-level option:\n %s\n",
+ BT_CLI_LOGE_APPEND_CAUSE("Invalid argument for --log-level option:\n %s",
arg);
goto error;
}
break;
case OPT_BASE_PARAMS:
{
- bt_value *params =
- bt_value_from_arg(arg);
+ bt_value *params = cli_value_from_arg(arg, error_str);
if (!params) {
- printf_err("Invalid format for --base-params option's argument:\n %s\n",
- arg);
+ BT_CLI_LOGE_APPEND_CAUSE("Invalid format for --base-params option's argument:\n %s",
+ error_str->str);
goto error;
}
BT_VALUE_PUT_REF_AND_RESET(cur_base_params);
cur_base_params = bt_value_map_create();
if (!cur_base_params) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
break;
case OPT_CONNECT:
if (bt_value_array_append_string_element(
connection_args, arg)) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
break;
- case OPT_RETRY_DURATION:
+ case OPT_RETRY_DURATION: {
+ gchar *end;
+ size_t arg_len = strlen(argpar_item_opt->arg);
+
+ retry_duration = g_ascii_strtoll(argpar_item_opt->arg, &end, 10);
+
+ if (arg_len == 0 || end != (argpar_item_opt->arg + arg_len)) {
+ BT_CLI_LOGE_APPEND_CAUSE(
+ "Could not parse --retry-duration option's argument as an unsigned integer: `%s`",
+ argpar_item_opt->arg);
+ goto error;
+ }
+
if (retry_duration < 0) {
- printf_err("--retry-duration option's argument must be positive or 0: %ld\n",
+ BT_CLI_LOGE_APPEND_CAUSE("--retry-duration option's argument must be positive or 0: %ld",
retry_duration);
goto error;
}
cfg->cmd_data.run.retry_duration_us =
(uint64_t) retry_duration;
break;
- case OPT_HELP:
- print_run_usage(stdout);
- *retcode = -1;
- BT_OBJECT_PUT_REF_AND_RESET(cfg);
- goto end;
+ }
default:
- printf_err("Unknown command-line option specified (option code %d)\n",
- opt);
+ BT_CLI_LOGE_APPEND_CAUSE("Unknown command-line option specified (option code %d).",
+ argpar_item_opt->descr->id);
goto error;
}
-
- free(arg);
- arg = NULL;
}
- /* Check for option parsing error */
- if (opt < -1) {
- printf_err("While parsing command-line options, at option %s: %s\n",
- poptBadOption(pc, 0), poptStrerror(opt));
- goto error;
- }
-
- /* This command does not accept leftover arguments */
- if (poptPeekArg(pc)) {
- printf_err("Unexpected argument: %s\n", poptPeekArg(pc));
- goto error;
- }
-
- /* Add current component */
- if (cur_cfg_comp) {
- ret = add_run_cfg_comp_check_name(cfg, cur_cfg_comp,
- cur_cfg_comp_dest, instance_names);
- BT_OBJECT_PUT_REF_AND_RESET(cur_cfg_comp);
- if (ret) {
- goto error;
- }
- }
+ BT_OBJECT_PUT_REF_AND_RESET(cur_cfg_comp);
if (cfg->cmd_data.run.sources->len == 0) {
- printf_err("Incomplete graph: no source component\n");
+ BT_CLI_LOGE_APPEND_CAUSE("Incomplete graph: no source component.");
goto error;
}
if (cfg->cmd_data.run.sinks->len == 0) {
- printf_err("Incomplete graph: no sink component\n");
+ BT_CLI_LOGE_APPEND_CAUSE("Incomplete graph: no sink component.");
goto error;
}
connection_args,
error_buf, 256);
if (ret) {
- printf_err("Cannot creation connections:\n%s", error_buf);
+ BT_CLI_LOGE_APPEND_CAUSE("Cannot creation connections:\n%s", error_buf);
goto error;
}
BT_OBJECT_PUT_REF_AND_RESET(cfg);
end:
- if (pc) {
- poptFreeContext(pc);
+ if (error_str) {
+ g_string_free(error_str, TRUE);
}
- free(arg);
+ bt_argpar_parse_ret_fini(&argpar_parse_ret);
BT_OBJECT_PUT_REF_AND_RESET(cur_cfg_comp);
BT_VALUE_PUT_REF_AND_RESET(cur_base_params);
BT_VALUE_PUT_REF_AND_RESET(instance_names);
struct bt_config *cfg = NULL;
const char **argv;
int64_t i, len;
- const size_t argc = bt_value_array_get_size(run_args) + 1;
+ const size_t argc = bt_value_array_get_size(run_args);
argv = calloc(argc, sizeof(*argv));
if (!argv) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto end;
}
- argv[0] = "run";
-
len = bt_value_array_get_size(run_args);
if (len < 0) {
- printf_err("Invalid executable arguments\n");
+ BT_CLI_LOGE_APPEND_CAUSE("Invalid executable arguments.");
goto end;
}
for (i = 0; i < len; i++) {
BT_ASSERT(arg_value);
arg = bt_value_string_get(arg_value);
BT_ASSERT(arg);
- argv[i + 1] = arg;
+ argv[i] = arg;
}
cfg = bt_config_run_from_args(argc, argv, retcode,
fprintf(fp, " TYPE (`source`, `filter`, or `sink`) found\n");
fprintf(fp, " in the plugin PLUGIN, add it to the\n");
fprintf(fp, " conversion graph, and optionally name it\n");
- fprintf(fp, " NAME (you can also specify the name with\n");
- fprintf(fp, " --name)\n");
+ fprintf(fp, " NAME\n");
fprintf(fp, " -l, --log-level=LVL Set the log level of the current component to LVL\n");
fprintf(fp, " (`N`, `V`, `D`, `I`, `W`, `E`, or `F`)\n");
- fprintf(fp, " --name=NAME Set the name of the current component\n");
- fprintf(fp, " to NAME (must be unique amongst all the\n");
- fprintf(fp, " names of the created components)\n");
fprintf(fp, " --omit-home-plugin-path Omit home plugins from plugin search path\n");
fprintf(fp, " (~/.local/lib/babeltrace2/plugins)\n");
fprintf(fp, " --omit-system-plugin-path Omit system plugins from plugin search path\n");
fprintf(fp, " -p, --params=PARAMS Add initialization parameters PARAMS to the\n");
fprintf(fp, " current component (see the expected format\n");
fprintf(fp, " of PARAMS below)\n");
- fprintf(fp, " -P, --path=PATH Set the `path` string parameter of the\n");
- fprintf(fp, " current component to PATH\n");
fprintf(fp, " --plugin-path=PATH[:PATH]... Add PATH to the list of paths from which\n");
+ fprintf(fp, " dynamic plugins can be loaded\n");
fprintf(fp, " --retry-duration=DUR When babeltrace2(1) needs to retry to run\n");
fprintf(fp, " the graph later, retry in DUR µs\n");
fprintf(fp, " (default: 100000)\n");
fprintf(fp, " formatted for `xargs -0`, and quit\n");
fprintf(fp, " --stream-intersection Only process events when all streams\n");
fprintf(fp, " are active\n");
- fprintf(fp, " -u, --url=URL Set the `url` string parameter of the\n");
- fprintf(fp, " current component to URL\n");
fprintf(fp, " -h, --help Show this help and quit\n");
fprintf(fp, "\n");
fprintf(fp, "Implicit `source.ctf.fs` component options:\n");
}
static
-struct poptOption convert_long_options[] = {
- /* longName, shortName, argInfo, argPtr, value, descrip, argDesc */
- { "begin", 'b', POPT_ARG_STRING, NULL, OPT_BEGIN, NULL, NULL },
- { "clock-cycles", '\0', POPT_ARG_NONE, NULL, OPT_CLOCK_CYCLES, NULL, NULL },
- { "clock-date", '\0', POPT_ARG_NONE, NULL, OPT_CLOCK_DATE, NULL, NULL },
- { "clock-force-correlate", '\0', POPT_ARG_NONE, NULL, OPT_CLOCK_FORCE_CORRELATE, NULL, NULL },
- { "clock-gmt", '\0', POPT_ARG_NONE, NULL, OPT_CLOCK_GMT, NULL, NULL },
- { "clock-offset", '\0', POPT_ARG_STRING, NULL, OPT_CLOCK_OFFSET, NULL, NULL },
- { "clock-offset-ns", '\0', POPT_ARG_STRING, NULL, OPT_CLOCK_OFFSET_NS, NULL, NULL },
- { "clock-seconds", '\0', POPT_ARG_NONE, NULL, OPT_CLOCK_SECONDS, NULL, NULL },
- { "color", '\0', POPT_ARG_STRING, NULL, OPT_COLOR, NULL, NULL },
- { "component", 'c', POPT_ARG_STRING, NULL, OPT_COMPONENT, NULL, NULL },
- { "debug", 'd', POPT_ARG_NONE, NULL, OPT_DEBUG, NULL, NULL },
- { "debug-info-dir", 0, POPT_ARG_STRING, NULL, OPT_DEBUG_INFO_DIR, NULL, NULL },
- { "debug-info-full-path", 0, POPT_ARG_NONE, NULL, OPT_DEBUG_INFO_FULL_PATH, NULL, NULL },
- { "debug-info-target-prefix", 0, POPT_ARG_STRING, NULL, OPT_DEBUG_INFO_TARGET_PREFIX, NULL, NULL },
- { "end", 'e', POPT_ARG_STRING, NULL, OPT_END, NULL, NULL },
- { "fields", 'f', POPT_ARG_STRING, NULL, OPT_FIELDS, NULL, NULL },
- { "help", 'h', POPT_ARG_NONE, NULL, OPT_HELP, NULL, NULL },
- { "input-format", 'i', POPT_ARG_STRING, NULL, OPT_INPUT_FORMAT, NULL, NULL },
- { "log-level", 'l', POPT_ARG_STRING, NULL, OPT_LOG_LEVEL, NULL, NULL },
- { "name", '\0', POPT_ARG_STRING, NULL, OPT_NAME, NULL, NULL },
- { "names", 'n', POPT_ARG_STRING, NULL, OPT_NAMES, NULL, NULL },
- { "debug-info", '\0', POPT_ARG_NONE, NULL, OPT_DEBUG_INFO, NULL, NULL },
- { "no-delta", '\0', POPT_ARG_NONE, NULL, OPT_NO_DELTA, NULL, NULL },
- { "omit-home-plugin-path", '\0', POPT_ARG_NONE, NULL, OPT_OMIT_HOME_PLUGIN_PATH, NULL, NULL },
- { "omit-system-plugin-path", '\0', POPT_ARG_NONE, NULL, OPT_OMIT_SYSTEM_PLUGIN_PATH, NULL, NULL },
- { "output", 'w', POPT_ARG_STRING, NULL, OPT_OUTPUT, NULL, NULL },
- { "output-format", 'o', POPT_ARG_STRING, NULL, OPT_OUTPUT_FORMAT, NULL, NULL },
- { "params", 'p', POPT_ARG_STRING, NULL, OPT_PARAMS, NULL, NULL },
- { "path", 'P', POPT_ARG_STRING, NULL, OPT_PATH, NULL, NULL },
- { "plugin-path", '\0', POPT_ARG_STRING, NULL, OPT_PLUGIN_PATH, NULL, NULL },
- { "retry-duration", '\0', POPT_ARG_STRING, NULL, OPT_RETRY_DURATION, NULL, NULL },
- { "run-args", '\0', POPT_ARG_NONE, NULL, OPT_RUN_ARGS, NULL, NULL },
- { "run-args-0", '\0', POPT_ARG_NONE, NULL, OPT_RUN_ARGS_0, NULL, NULL },
- { "stream-intersection", '\0', POPT_ARG_NONE, NULL, OPT_STREAM_INTERSECTION, NULL, NULL },
- { "timerange", '\0', POPT_ARG_STRING, NULL, OPT_TIMERANGE, NULL, NULL },
- { "url", 'u', POPT_ARG_STRING, NULL, OPT_URL, NULL, NULL },
- { "verbose", 'v', POPT_ARG_NONE, NULL, OPT_VERBOSE, NULL, NULL },
- { NULL, 0, '\0', NULL, 0, NULL, NULL },
+const struct bt_argpar_opt_descr convert_options[] = {
+ /* id, short_name, long_name, with_arg */
+ { OPT_BEGIN, 'b', "begin", true },
+ { OPT_CLOCK_CYCLES, '\0', "clock-cycles", false },
+ { OPT_CLOCK_DATE, '\0', "clock-date", false },
+ { OPT_CLOCK_FORCE_CORRELATE, '\0', "clock-force-correlate", false },
+ { OPT_CLOCK_GMT, '\0', "clock-gmt", false },
+ { OPT_CLOCK_OFFSET, '\0', "clock-offset", true },
+ { OPT_CLOCK_OFFSET_NS, '\0', "clock-offset-ns", true },
+ { OPT_CLOCK_SECONDS, '\0', "clock-seconds", false },
+ { OPT_COLOR, '\0', "color", true },
+ { OPT_COMPONENT, 'c', "component", true },
+ { OPT_DEBUG, 'd', "debug", false },
+ { OPT_DEBUG_INFO_DIR, '\0', "debug-info-dir", true },
+ { OPT_DEBUG_INFO_FULL_PATH, '\0', "debug-info-full-path", false },
+ { OPT_DEBUG_INFO_TARGET_PREFIX, '\0', "debug-info-target-prefix", true },
+ { OPT_END, 'e', "end", true },
+ { OPT_FIELDS, 'f', "fields", true },
+ { OPT_HELP, 'h', "help", false },
+ { OPT_INPUT_FORMAT, 'i', "input-format", true },
+ { OPT_LOG_LEVEL, 'l', "log-level", true },
+ { OPT_NAMES, 'n', "names", true },
+ { OPT_DEBUG_INFO, '\0', "debug-info", false },
+ { OPT_NO_DELTA, '\0', "no-delta", false },
+ { OPT_OMIT_HOME_PLUGIN_PATH, '\0', "omit-home-plugin-path", false },
+ { OPT_OMIT_SYSTEM_PLUGIN_PATH, '\0', "omit-system-plugin-path", false },
+ { OPT_OUTPUT, 'w', "output", true },
+ { OPT_OUTPUT_FORMAT, 'o', "output-format", true },
+ { OPT_PARAMS, 'p', "params", true },
+ { OPT_PLUGIN_PATH, '\0', "plugin-path", true },
+ { OPT_RETRY_DURATION, '\0', "retry-duration", true },
+ { OPT_RUN_ARGS, '\0', "run-args", false },
+ { OPT_RUN_ARGS_0, '\0', "run-args-0", false },
+ { OPT_STREAM_INTERSECTION, '\0', "stream-intersection", false },
+ { OPT_TIMERANGE, '\0', "timerange", true },
+ { OPT_VERBOSE, 'v', "verbose", false },
+ BT_ARGPAR_OPT_DESCR_SENTINEL
};
static
GString *auto_name = g_string_new(NULL);
if (!auto_name) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto end;
}
struct implicit_component_args {
bool exists;
+
+ /* The component class name (e.g. src.ctf.fs). */
GString *comp_arg;
+
+ /* The component instance name. */
GString *name_arg;
+
GString *params_arg;
bt_value *extra_params;
};
if (bt_value_map_insert_entry(existing_names, name->str,
bt_value_null)) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
ret = -1;
goto end;
}
{
int ret = 0;
size_t i;
+ GString *component_arg_for_run = NULL;
if (!impl_args->exists) {
goto end;
}
- if (bt_value_array_append_string_element(run_args, "--component")) {
- print_err_oom();
+ component_arg_for_run = g_string_new(NULL);
+ if (!component_arg_for_run) {
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
- if (bt_value_array_append_string_element(run_args, impl_args->comp_arg->str)) {
- print_err_oom();
- goto error;
- }
+ /* Build the full `name:type.plugin.cls`. */
+ BT_ASSERT(!strchr(impl_args->name_arg->str, '\\'));
+ BT_ASSERT(!strchr(impl_args->name_arg->str, ':'));
+ g_string_printf(component_arg_for_run, "%s:%s",
+ impl_args->name_arg->str, impl_args->comp_arg->str);
- if (bt_value_array_append_string_element(run_args, "--name")) {
- print_err_oom();
+ if (bt_value_array_append_string_element(run_args, "--component")) {
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
- if (bt_value_array_append_string_element(run_args, impl_args->name_arg->str)) {
- print_err_oom();
+ if (bt_value_array_append_string_element(run_args,
+ component_arg_for_run->str)) {
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
if (impl_args->params_arg->len > 0) {
if (bt_value_array_append_string_element(run_args, "--params")) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
if (bt_value_array_append_string_element(run_args,
impl_args->params_arg->str)) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
}
arg = bt_value_string_get(elem);
ret = bt_value_array_append_string_element(run_args, arg);
if (ret) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
}
ret = -1;
end:
+ if (component_arg_for_run) {
+ g_string_free(component_arg_for_run, TRUE);
+ }
+
return ret;
}
+/* Free the fields of a `struct implicit_component_args`. */
+
static
void finalize_implicit_component_args(struct implicit_component_args *args)
{
bt_value_put_ref(args->extra_params);
}
+/* Destroy a dynamically-allocated `struct implicit_component_args`. */
+
+static
+void destroy_implicit_component_args(struct implicit_component_args *args)
+{
+ finalize_implicit_component_args(args);
+ g_free(args);
+}
+
+/* Initialize the fields of an already allocated `struct implicit_component_args`. */
+
static
int init_implicit_component_args(struct implicit_component_args *args,
const char *comp_arg, bool exists)
!args->params_arg || !args->extra_params) {
ret = -1;
finalize_implicit_component_args(args);
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto end;
}
return ret;
}
+/* Dynamically allocate and initialize a `struct implicit_component_args`. */
+
+static
+struct implicit_component_args *create_implicit_component_args(
+ const char *comp_arg)
+{
+ struct implicit_component_args *args;
+ int status;
+
+ args = g_new(struct implicit_component_args, 1);
+ if (!args) {
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
+ goto end;
+ }
+
+ status = init_implicit_component_args(args, comp_arg, true);
+ if (status != 0) {
+ g_free(args);
+ args = NULL;
+ }
+
+end:
+ return args;
+}
+
static
void append_implicit_component_param(struct implicit_component_args *args,
const char *key, const char *value)
append_param_arg(args->params_arg, key, value);
}
+/*
+ * Append the given parameter (`key=value`) to all component specifications
+ * in `implicit_comp_args` (an array of `struct implicit_component_args *`)
+ * which match `comp_arg`.
+ *
+ * Return the number of matching components.
+ */
+
+static
+int append_multiple_implicit_components_param(GPtrArray *implicit_comp_args,
+ const char *comp_arg, const char *key, const char *value)
+{
+ int i;
+ int n = 0;
+
+ for (i = 0; i < implicit_comp_args->len; i++) {
+ struct implicit_component_args *args = implicit_comp_args->pdata[i];
+
+ if (strcmp(args->comp_arg->str, comp_arg) == 0) {
+ append_implicit_component_param(args, key, value);
+ n++;
+ }
+ }
+
+ return n;
+}
+
/* Escape value to make it suitable to use as a string parameter value. */
static
gchar *escape_string_value(const char *value)
ret = g_string_new(NULL);
if (!ret) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto end;
}
buf = g_string_new(NULL);
if (!buf) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
GString *parameter = NULL;
if (bt_value_array_append_string_element(args, "--params")) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
ret = -1;
goto end;
}
parameter = g_string_new(NULL);
if (!parameter) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
ret = -1;
goto end;
}
g_string_printf(parameter, "%s=%s", key, str_value);
if (bt_value_array_append_string_element(args, parameter->str)) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
ret = -1;
goto end;
}
str_value = bt_value_string_create_init(value);
if (!str_value) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
ret = -1;
goto end;
}
return append_string_parameter_to_args(args->extra_params, key, value);
}
+/*
+ * Escapes `.`, `:`, and `\` of `input` with `\`.
+ */
static
-int convert_append_name_param(enum bt_config_component_dest dest,
- GString *cur_name, GString *cur_name_prefix,
- bt_value *run_args,
- bt_value *all_names,
- GList **source_names, GList **filter_names,
- GList **sink_names)
+GString *escape_dot_colon(const char *input)
{
- int ret = 0;
+ GString *output = g_string_new(NULL);
+ const char *ch;
- if (cur_name_prefix->len > 0) {
- /* We're after a --component option */
- GString *name = NULL;
- bool append_name_opt = false;
-
- if (cur_name->len == 0) {
- /*
- * No explicit name was provided for the user
- * component.
- */
- name = get_component_auto_name(cur_name_prefix->str,
- all_names);
- append_name_opt = true;
- } else {
- /*
- * An explicit name was provided for the user
- * component.
- */
- if (bt_value_map_has_entry(all_names,
- cur_name->str)) {
- printf_err("Duplicate component instance name:\n %s\n",
- cur_name->str);
- goto error;
- }
-
- name = g_string_new(cur_name->str);
- }
-
- if (!name) {
- print_err_oom();
- goto error;
- }
-
- /*
- * Remember this name globally, for the uniqueness of
- * all component names.
- */
- if (bt_value_map_insert_entry(all_names, name->str, bt_value_null)) {
- print_err_oom();
- goto error;
- }
-
- /*
- * Append the --name option if necessary.
- */
- if (append_name_opt) {
- if (bt_value_array_append_string_element(run_args, "--name")) {
- print_err_oom();
- goto error;
- }
-
- if (bt_value_array_append_string_element(run_args, name->str)) {
- print_err_oom();
- goto error;
- }
- }
-
- /*
- * Remember this name specifically for the type of the
- * component. This is to create connection arguments.
- */
- switch (dest) {
- case BT_CONFIG_COMPONENT_DEST_SOURCE:
- *source_names = g_list_append(*source_names, name);
- break;
- case BT_CONFIG_COMPONENT_DEST_FILTER:
- *filter_names = g_list_append(*filter_names, name);
- break;
- case BT_CONFIG_COMPONENT_DEST_SINK:
- *sink_names = g_list_append(*sink_names, name);
- break;
- default:
- abort();
- }
-
- g_string_assign(cur_name_prefix, "");
- }
-
- goto end;
-
-error:
- ret = -1;
-
-end:
- return ret;
-}
-
-/*
- * Escapes `.`, `:`, and `\` of `input` with `\`.
- */
-static
-GString *escape_dot_colon(const char *input)
-{
- GString *output = g_string_new(NULL);
- const char *ch;
-
- if (!output) {
- print_err_oom();
- goto end;
- }
+ if (!output) {
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
+ goto end;
+ }
for (ch = input; *ch != '\0'; ch++) {
if (*ch == '\\' || *ch == '.' || *ch == ':') {
GString *arg = g_string_new(NULL);
if (!e_upstream_name || !e_downstream_name || !arg) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
ret = -1;
goto end;
}
ret = bt_value_array_append_string_element(run_args, "--connect");
if (ret) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
ret = -1;
goto end;
}
g_string_append(arg, e_downstream_name->str);
ret = bt_value_array_append_string_element(run_args, arg->str);
if (ret) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
ret = -1;
goto end;
}
BT_ASSERT(sink_names);
/* Connect all sources to the first filter */
- for (source_at = source_names; source_at != NULL; source_at = g_list_next(source_at)) {
+ for (source_at = source_names; source_at; source_at = g_list_next(source_at)) {
GString *source_name = source_at->data;
GString *filter_name = filter_at->data;
filter_at = g_list_next(filter_at);
/* Connect remaining filters */
- for (; filter_at != NULL; filter_prev = filter_at, filter_at = g_list_next(filter_at)) {
+ for (; filter_at; filter_prev = filter_at, filter_at = g_list_next(filter_at)) {
GString *filter_name = filter_at->data;
GString *filter_prev_name = filter_prev->data;
}
/* Connect last filter to all sinks */
- for (sink_at = sink_names; sink_at != NULL; sink_at = g_list_next(sink_at)) {
+ for (sink_at = sink_names; sink_at; sink_at = g_list_next(sink_at)) {
GString *filter_name = filter_prev->data;
GString *sink_name = sink_at->data;
BT_ASSERT(list);
if (!gs) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto end;
}
return ret;
}
+/*
+ * Create `struct implicit_component_args` structures for each of the source
+ * components we identified. Add them to `component_args`.
+ */
+
+static
+void create_implicit_component_args_from_auto_discovered_sources(
+ const struct auto_source_discovery *auto_disc, GPtrArray *component_args)
+{
+ gchar *cc_name = NULL;
+ struct implicit_component_args *comp = NULL;
+ int status;
+ guint i, len;
+
+ len = auto_disc->results->len;
+
+ for (i = 0; i < len; i++) {
+ struct auto_source_discovery_result *res =
+ g_ptr_array_index(auto_disc->results, i);
+
+ g_free(cc_name);
+ cc_name = g_strdup_printf("source.%s.%s", res->plugin_name, res->source_cc_name);
+ if (!cc_name) {
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
+ goto end;
+ }
+
+ comp = create_implicit_component_args(cc_name);
+ if (!comp) {
+ goto end;
+ }
+
+ status = append_parameter_to_args(comp->extra_params, "inputs", res->inputs);
+ if (status != 0) {
+ goto end;
+ }
+
+ g_ptr_array_add(component_args, comp);
+ comp = NULL;
+ }
+
+end:
+ g_free(cc_name);
+
+ if (comp) {
+ destroy_implicit_component_args(comp);
+ }
+}
+
+/*
+ * As we iterate the arguments to the convert command, this tracks what is the
+ * type of the current item, to which some contextual options (e.g. --params)
+ * apply to.
+ */
+enum convert_current_item_type {
+ /* There is no current item. */
+ CONVERT_CURRENT_ITEM_TYPE_NONE,
+
+ /* Current item is a component. */
+ CONVERT_CURRENT_ITEM_TYPE_COMPONENT,
+};
+
/*
* Creates a Babeltrace config object from the arguments of a convert
* command.
bool force_omit_home_plugin_path,
const bt_value *initial_plugin_paths, int *default_log_level)
{
- poptContext pc = NULL;
- char *arg = NULL;
- enum bt_config_component_dest cur_comp_dest =
- BT_CONFIG_COMPONENT_DEST_UNKNOWN;
- int opt, ret = 0;
+ enum convert_current_item_type current_item_type =
+ CONVERT_CURRENT_ITEM_TYPE_NONE;
+ int ret = 0;
struct bt_config *cfg = NULL;
bool got_input_format_opt = false;
bool got_output_format_opt = false;
bool trimmer_has_begin = false;
bool trimmer_has_end = false;
bool stream_intersection_mode = false;
- GString *cur_name = NULL;
- GString *cur_name_prefix = NULL;
- const char *leftover = NULL;
bool print_run_args = false;
bool print_run_args_0 = false;
bool print_ctf_metadata = false;
GList *filter_names = NULL;
GList *sink_names = NULL;
bt_value *leftovers = NULL;
- struct implicit_component_args implicit_ctf_input_args = { 0 };
struct implicit_component_args implicit_ctf_output_args = { 0 };
struct implicit_component_args implicit_lttng_live_args = { 0 };
struct implicit_component_args implicit_dummy_args = { 0 };
size_t i;
struct bt_common_lttng_live_url_parts lttng_live_url_parts = { 0 };
char *output = NULL;
+ struct auto_source_discovery auto_disc = { NULL };
+ GString *auto_disc_comp_name = NULL;
+ struct bt_argpar_parse_ret argpar_parse_ret = { 0 };
+ GString *name_gstr = NULL;
+ GString *component_arg_for_run = NULL;
+
+ /*
+ * Array of `struct implicit_component_args *` created for the sources
+ * we have auto-discovered.
+ */
+ GPtrArray *discovered_source_args = NULL;
+
+ /*
+ * If set, restrict automatic source discovery to this component class
+ * of this plugin.
+ */
+ const char *auto_source_discovery_restrict_plugin_name = NULL;
+ const char *auto_source_discovery_restrict_component_class_name = NULL;
+
+ gchar *ctf_fs_source_clock_class_offset_arg = NULL;
+ gchar *ctf_fs_source_clock_class_offset_ns_arg = NULL;
(void) bt_value_copy(initial_plugin_paths, &plugin_paths);
*retcode = 0;
- if (argc <= 1) {
+ if (argc < 1) {
print_convert_usage(stdout);
*retcode = -1;
goto end;
}
- if (init_implicit_component_args(&implicit_ctf_input_args,
- "source.ctf.fs", false)) {
- goto error;
- }
-
if (init_implicit_component_args(&implicit_ctf_output_args,
"sink.ctf.fs", false)) {
goto error;
all_names = bt_value_map_create();
if (!all_names) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
run_args = bt_value_array_create();
if (!run_args) {
- print_err_oom();
- goto error;
- }
-
- cur_name = g_string_new(NULL);
- if (!cur_name) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
- cur_name_prefix = g_string_new(NULL);
- if (!cur_name_prefix) {
- print_err_oom();
+ component_arg_for_run = g_string_new(NULL);
+ if (!component_arg_for_run) {
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
leftovers = bt_value_array_create();
if (!leftovers) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
+ goto error;
+ }
+
+ if (auto_source_discovery_init(&auto_disc) != 0) {
+ goto error;
+ }
+
+ discovered_source_args =
+ g_ptr_array_new_with_free_func((GDestroyNotify) destroy_implicit_component_args);
+ if (!discovered_source_args) {
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
+ goto error;
+ }
+
+ auto_disc_comp_name = g_string_new(NULL);
+ if (!auto_disc_comp_name) {
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
* First pass: collect all arguments which need to be passed
* as is to the run command. This pass can also add --name
* arguments if needed to automatically name unnamed component
- * instances. Also it does the following transformations:
- *
- * --path=PATH -> --params=path="PATH"
- * --url=URL -> --params=url="URL"
+ * instances.
*
* Also it appends the plugin paths of --plugin-path to
* `plugin_paths`.
*/
- pc = poptGetContext(NULL, argc, (const char **) argv,
- convert_long_options, 0);
- if (!pc) {
- printf_err("Cannot get popt context\n");
+ argpar_parse_ret = bt_argpar_parse(argc, argv, convert_options, true);
+ if (argpar_parse_ret.error) {
+ BT_CLI_LOGE_APPEND_CAUSE(
+ "While parsing `convert` command's command-line arguments: %s",
+ argpar_parse_ret.error->str);
goto error;
}
- poptReadDefaultConfig(pc, 0);
+ if (help_option_is_specified(&argpar_parse_ret)) {
+ print_convert_usage(stdout);
+ *retcode = -1;
+ BT_OBJECT_PUT_REF_AND_RESET(cfg);
+ goto end;
+ }
- while ((opt = poptGetNextOpt(pc)) > 0) {
+ for (i = 0; i < argpar_parse_ret.items->len; i++) {
+ struct bt_argpar_item *argpar_item =
+ g_ptr_array_index(argpar_parse_ret.items, i);
+ struct bt_argpar_item_opt *argpar_item_opt;
char *name = NULL;
char *plugin_name = NULL;
char *comp_cls_name = NULL;
+ const char *arg;
+
+ if (argpar_item->type != BT_ARGPAR_ITEM_TYPE_OPT) {
+ continue;
+ }
- arg = poptGetOptArg(pc);
+ argpar_item_opt = (struct bt_argpar_item_opt *) argpar_item;
+ arg = argpar_item_opt->arg;
- switch (opt) {
+ switch (argpar_item_opt->descr->id) {
case OPT_COMPONENT:
{
bt_component_class_type type;
- const char *type_prefix;
- /* Append current component's name if needed */
- ret = convert_append_name_param(cur_comp_dest, cur_name,
- cur_name_prefix, run_args, all_names,
- &source_names, &filter_names, &sink_names);
- if (ret) {
- goto error;
- }
+ current_item_type = CONVERT_CURRENT_ITEM_TYPE_COMPONENT;
/* Parse the argument */
plugin_comp_cls_names(arg, &name, &plugin_name,
&comp_cls_name, &type);
if (!plugin_name || !comp_cls_name) {
- printf_err("Invalid format for --component option's argument:\n %s\n",
+ BT_CLI_LOGE_APPEND_CAUSE(
+ "Invalid format for --component option's argument:\n %s",
arg);
goto error;
}
if (name) {
- g_string_assign(cur_name, name);
+ /*
+ * Name was given by the user, verify it isn't
+ * taken.
+ */
+ if (bt_value_map_has_entry(all_names, name)) {
+ BT_CLI_LOGE_APPEND_CAUSE(
+ "Duplicate component instance name:\n %s",
+ name);
+ goto error;
+ }
+
+ name_gstr = g_string_new(name);
+ if (!name_gstr) {
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
+ goto error;
+ }
+
+ g_string_assign(component_arg_for_run, arg);
} else {
- g_string_assign(cur_name, "");
+ /* Name not given by user, generate one. */
+ name_gstr = get_component_auto_name(arg, all_names);
+ if (!name_gstr) {
+ goto error;
+ }
+
+ g_string_printf(component_arg_for_run, "%s:%s",
+ name_gstr->str, arg);
+ }
+
+ if (bt_value_array_append_string_element(run_args,
+ "--component")) {
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
+ goto error;
}
+ if (bt_value_array_append_string_element(run_args,
+ component_arg_for_run->str)) {
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
+ goto error;
+ }
+
+ /*
+ * Remember this name globally, for the uniqueness of
+ * all component names.
+ */
+ if (bt_value_map_insert_entry(all_names,
+ name_gstr->str, bt_value_null)) {
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
+ goto error;
+ }
+
+ /*
+ * Remember this name specifically for the type of the
+ * component. This is to create connection arguments.
+ *
+ * The list takes ownership of `name_gstr`.
+ */
switch (type) {
case BT_COMPONENT_CLASS_TYPE_SOURCE:
- cur_comp_dest = BT_CONFIG_COMPONENT_DEST_SOURCE;
- type_prefix = "source";
+ source_names = g_list_append(source_names, name_gstr);
break;
case BT_COMPONENT_CLASS_TYPE_FILTER:
- cur_comp_dest = BT_CONFIG_COMPONENT_DEST_FILTER;
- type_prefix = "filter";
+ filter_names = g_list_append(filter_names, name_gstr);
break;
case BT_COMPONENT_CLASS_TYPE_SINK:
- cur_comp_dest = BT_CONFIG_COMPONENT_DEST_SINK;
- type_prefix = "sink";
+ sink_names = g_list_append(sink_names, name_gstr);
break;
default:
abort();
}
+ name_gstr = NULL;
- if (bt_value_array_append_string_element(run_args,
- "--component")) {
- print_err_oom();
- goto error;
- }
-
- if (bt_value_array_append_string_element(run_args, arg)) {
- print_err_oom();
- goto error;
- }
-
- g_string_assign(cur_name_prefix, "");
- g_string_append_printf(cur_name_prefix, "%s.%s.%s",
- type_prefix, plugin_name, comp_cls_name);
free(name);
free(plugin_name);
free(comp_cls_name);
break;
}
case OPT_PARAMS:
- if (cur_name_prefix->len == 0) {
- printf_err("No current component of which to set parameters:\n %s\n",
+ if (current_item_type != CONVERT_CURRENT_ITEM_TYPE_COMPONENT) {
+ BT_CLI_LOGE_APPEND_CAUSE("No current component of which to set parameters:\n %s",
arg);
goto error;
}
if (bt_value_array_append_string_element(run_args,
"--params")) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
if (bt_value_array_append_string_element(run_args, arg)) {
- print_err_oom();
- goto error;
- }
- break;
- case OPT_PATH:
- if (cur_name_prefix->len == 0) {
- printf_err("No current component of which to set `path` parameter:\n %s\n",
- arg);
- goto error;
- }
-
- if (append_string_parameter_to_args(run_args, "path", arg)) {
- goto error;
- }
- break;
- case OPT_URL:
- if (cur_name_prefix->len == 0) {
- printf_err("No current component of which to set `url` parameter:\n %s\n",
- arg);
- goto error;
- }
-
-
- if (append_string_parameter_to_args(run_args, "url", arg)) {
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
break;
- case OPT_NAME:
- if (cur_name_prefix->len == 0) {
- printf_err("No current component to name:\n %s\n",
- arg);
- goto error;
- }
-
- if (bt_value_array_append_string_element(run_args, "--name")) {
- print_err_oom();
- goto error;
- }
-
- if (bt_value_array_append_string_element(run_args, arg)) {
- print_err_oom();
- goto error;
- }
-
- g_string_assign(cur_name, arg);
- break;
case OPT_LOG_LEVEL:
- if (cur_name_prefix->len == 0) {
- printf_err("No current component to assign a log level to:\n %s\n",
+ if (current_item_type != CONVERT_CURRENT_ITEM_TYPE_COMPONENT) {
+ BT_CLI_LOGE_APPEND_CAUSE("No current component to assign a log level to:\n %s",
arg);
goto error;
}
if (bt_value_array_append_string_element(run_args, "--log-level")) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
if (bt_value_array_append_string_element(run_args, arg)) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
if (bt_value_array_append_string_element(run_args,
"--omit-home-plugin-path")) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
break;
case OPT_RETRY_DURATION:
if (bt_value_array_append_string_element(run_args,
"--retry-duration")) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
if (bt_value_array_append_string_element(run_args, arg)) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
break;
if (bt_value_array_append_string_element(run_args,
"--omit-system-plugin-path")) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
break;
if (bt_value_array_append_string_element(run_args,
"--plugin-path")) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
if (bt_value_array_append_string_element(run_args, arg)) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
break;
- case OPT_HELP:
- print_convert_usage(stdout);
- *retcode = -1;
- BT_OBJECT_PUT_REF_AND_RESET(cfg);
- goto end;
case OPT_BEGIN:
case OPT_CLOCK_CYCLES:
case OPT_CLOCK_DATE:
/* Ignore in this pass */
break;
default:
- printf_err("Unknown command-line option specified (option code %d)\n",
- opt);
+ BT_CLI_LOGE_APPEND_CAUSE("Unknown command-line option specified (option code %d).",
+ argpar_item_opt->descr->id);
goto error;
}
-
- free(arg);
- arg = NULL;
}
- /* Append current component's name if needed */
- ret = convert_append_name_param(cur_comp_dest, cur_name,
- cur_name_prefix, run_args, all_names, &source_names,
- &filter_names, &sink_names);
- if (ret) {
- goto error;
- }
-
- /* Check for option parsing error */
- if (opt < -1) {
- printf_err("While parsing command-line options, at option %s: %s\n",
- poptBadOption(pc, 0), poptStrerror(opt));
- goto error;
- }
-
- poptFreeContext(pc);
- free(arg);
- arg = NULL;
-
/*
* Second pass: transform the convert-specific options and
* arguments into implicit component instances for the run
* command.
*/
- pc = poptGetContext(NULL, argc, (const char **) argv,
- convert_long_options, 0);
- if (!pc) {
- printf_err("Cannot get popt context\n");
- goto error;
- }
+ for (i = 0; i < argpar_parse_ret.items->len; i++) {
+ struct bt_argpar_item *argpar_item =
+ g_ptr_array_index(argpar_parse_ret.items, i);
+ struct bt_argpar_item_opt *argpar_item_opt;
+ const char *arg;
- poptReadDefaultConfig(pc, 0);
+ if (argpar_item->type != BT_ARGPAR_ITEM_TYPE_OPT) {
+ continue;
+ }
- while ((opt = poptGetNextOpt(pc)) > 0) {
- arg = poptGetOptArg(pc);
+ argpar_item_opt = (struct bt_argpar_item_opt *) argpar_item;
+ arg = argpar_item_opt->arg;
- switch (opt) {
+ switch (argpar_item_opt->descr->id) {
case OPT_BEGIN:
if (trimmer_has_begin) {
printf("At --begin option: --begin or --timerange option already specified\n %s\n",
ret = split_timerange(arg, &begin, &end);
if (ret) {
- printf_err("Invalid --timerange option's argument: expecting BEGIN,END or [BEGIN,END]:\n %s\n",
+ BT_CLI_LOGE_APPEND_CAUSE("Invalid --timerange option's argument: expecting BEGIN,END or [BEGIN,END]:\n %s",
arg);
goto error;
}
implicit_text_args.exists = true;
break;
case OPT_CLOCK_OFFSET:
- implicit_ctf_input_args.exists = true;
- append_implicit_component_param(
- &implicit_ctf_input_args,
- "clock-class-offset-s", arg);
+ if (ctf_fs_source_clock_class_offset_arg) {
+ BT_CLI_LOGE_APPEND_CAUSE("Duplicate --clock-offset option\n");
+ goto error;
+ }
+
+ ctf_fs_source_clock_class_offset_arg = g_strdup(arg);
+ if (!ctf_fs_source_clock_class_offset_arg) {
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
+ goto error;
+ }
break;
case OPT_CLOCK_OFFSET_NS:
- implicit_ctf_input_args.exists = true;
- append_implicit_component_param(
- &implicit_ctf_input_args,
- "clock-class-offset-ns", arg);
+ if (ctf_fs_source_clock_class_offset_ns_arg) {
+ BT_CLI_LOGE_APPEND_CAUSE("Duplicate --clock-offset-ns option\n");
+ goto error;
+ }
+
+ ctf_fs_source_clock_class_offset_ns_arg = g_strdup(arg);
+ if (!ctf_fs_source_clock_class_offset_ns_arg) {
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
+ goto error;
+ }
break;
case OPT_CLOCK_SECONDS:
append_implicit_component_param(
break;
case OPT_INPUT_FORMAT:
if (got_input_format_opt) {
- printf_err("Duplicate --input-format option\n");
+ BT_CLI_LOGE_APPEND_CAUSE("Duplicate --input-format option.");
goto error;
}
got_input_format_opt = true;
if (strcmp(arg, "ctf") == 0) {
- implicit_ctf_input_args.exists = true;
+ auto_source_discovery_restrict_plugin_name = "ctf";
+ auto_source_discovery_restrict_component_class_name = "fs";
} else if (strcmp(arg, "lttng-live") == 0) {
+ auto_source_discovery_restrict_plugin_name = "ctf";
+ auto_source_discovery_restrict_component_class_name = "lttng-live";
implicit_lttng_live_args.exists = true;
} else {
- printf_err("Unknown legacy input format:\n %s\n",
+ BT_CLI_LOGE_APPEND_CAUSE("Unknown legacy input format:\n %s",
arg);
goto error;
}
break;
case OPT_OUTPUT_FORMAT:
if (got_output_format_opt) {
- printf_err("Duplicate --output-format option\n");
+ BT_CLI_LOGE_APPEND_CAUSE("Duplicate --output-format option.");
goto error;
}
} else if (strcmp(arg, "ctf-metadata") == 0) {
print_ctf_metadata = true;
} else {
- printf_err("Unknown legacy output format:\n %s\n",
+ BT_CLI_LOGE_APPEND_CAUSE("Unknown legacy output format:\n %s",
arg);
goto error;
}
break;
case OPT_OUTPUT:
if (output) {
- printf_err("Duplicate --output option\n");
+ BT_CLI_LOGE_APPEND_CAUSE("Duplicate --output option");
goto error;
}
output = strdup(arg);
if (!output) {
- print_err_oom();
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
break;
case OPT_RUN_ARGS:
if (print_run_args_0) {
- printf_err("Cannot specify --run-args and --run-args-0\n");
+ BT_CLI_LOGE_APPEND_CAUSE("Cannot specify --run-args and --run-args-0.");
goto error;
}
break;
case OPT_RUN_ARGS_0:
if (print_run_args) {
- printf_err("Cannot specify --run-args and --run-args-0\n");
+ BT_CLI_LOGE_APPEND_CAUSE("Cannot specify --run-args and --run-args-0.");
goto error;
}
break;
case OPT_STREAM_INTERSECTION:
/*
- * Applies to all traces implementing the trace-info
- * query.
+ * Applies to all traces implementing the
+ * babeltrace.trace-info query.
*/
stream_intersection_mode = true;
break;
case OPT_VERBOSE:
- if (*default_log_level != BT_LOG_VERBOSE &&
+ if (*default_log_level != BT_LOG_TRACE &&
*default_log_level != BT_LOG_DEBUG) {
*default_log_level = BT_LOG_INFO;
}
break;
case OPT_DEBUG:
- *default_log_level = BT_LOG_VERBOSE;
+ *default_log_level = BT_LOG_TRACE;
break;
default:
break;
}
-
- free(arg);
- arg = NULL;
- }
-
- /* Check for option parsing error */
- if (opt < -1) {
- printf_err("While parsing command-line options, at option %s: %s\n",
- poptBadOption(pc, 0), poptStrerror(opt));
- goto error;
}
/*
}
/* Consume and keep leftover arguments */
- while ((leftover = poptGetArg(pc))) {
- bt_value_status status = bt_value_array_append_string_element(leftovers, leftover);
- if (status != BT_VALUE_STATUS_OK) {
- print_err_oom();
+ for (i = 0; i < argpar_parse_ret.items->len; i++) {
+ struct bt_argpar_item *argpar_item =
+ g_ptr_array_index(argpar_parse_ret.items, i);
+ struct bt_argpar_item_non_opt *argpar_item_non_opt;
+
+ if (argpar_item->type != BT_ARGPAR_ITEM_TYPE_NON_OPT) {
+ continue;
+ }
+
+ argpar_item_non_opt = (struct bt_argpar_item_non_opt *) argpar_item;
+
+ if (bt_value_array_append_string_element(leftovers, argpar_item_non_opt->arg) !=
+ BT_VALUE_ARRAY_APPEND_ELEMENT_STATUS_OK) {
+ BT_CLI_LOGE_APPEND_CAUSE_OOM();
goto error;
}
}
const bt_value *bt_val_leftover;
if (bt_value_array_is_empty(leftovers)) {
- printf_err("--output-format=ctf-metadata specified without a path\n");
+ BT_CLI_LOGE_APPEND_CAUSE("--output-format=ctf-metadata specified without a path.");
goto error;
}
if (bt_value_array_get_size(leftovers) > 1) {
- printf_err("Too many paths specified for --output-format=ctf-metadata\n");
+ BT_CLI_LOGE_APPEND_CAUSE("Too many paths specified for --output-format=ctf-metadata.");
goto error;
}
*/
if (implicit_ctf_output_args.exists) {
if (!output) {
- printf_err("--output-format=ctf specified without --output (trace output path)\n");
+ BT_CLI_LOGE_APPEND_CAUSE("--output-format=ctf specified without --output (trace output path).");
goto error;
}
* sink.ctf.fs implicit components.
*/
if (implicit_text_args.exists) {
- printf_err("Ambiguous --output option: --output-format=ctf specified but another option implies --output-format=text\n");
+ BT_CLI_LOGE_APPEND_CAUSE("Ambiguous --output option: --output-format=ctf specified but another option implies --output-format=text.");
goto error;
}
}
const bt_value *bt_val_leftover;
if (bt_value_array_get_size(leftovers) > 1) {
- printf_err("Too many URLs specified for --output-format=lttng-live\n");
+ BT_CLI_LOGE_APPEND_CAUSE("Too many URLs specified for --input-format=lttng-live.");
goto error;
}
bt_common_parse_lttng_live_url(bt_value_string_get(bt_val_leftover),
error_buf, sizeof(error_buf));
if (!lttng_live_url_parts.proto) {
- printf_err("Invalid LTTng live URL format: %s\n",
+ BT_CLI_LOGE_APPEND_CAUSE("Invalid LTTng live URL format: %s.",
error_buf);
goto error;
}
goto error;
}
} else {
- /*
- * Create one source.ctf.fs component, pass it an array
- * with the leftovers.
- * Note that it still has to be named later.
- */
- implicit_ctf_input_args.exists = true;
- ret = append_parameter_to_args(implicit_ctf_input_args.extra_params,
- "paths", leftovers);
- if (ret) {
+ int status;
+
+ status = auto_discover_source_components(plugin_paths, leftovers,
+ auto_source_discovery_restrict_plugin_name,
+ auto_source_discovery_restrict_component_class_name,
+ *default_log_level >= 0 ? *default_log_level : cli_default_log_level,
+ &auto_disc);
+
+ if (status != 0) {
goto error;
}
+
+ create_implicit_component_args_from_auto_discovered_sources(
+ &auto_disc, discovered_source_args);
}
}
- /*
- * Ensure mutual exclusion between implicit `source.ctf.fs` and
- * `source.ctf.lttng-live` components.
- */
- if (implicit_ctf_input_args.exists && implicit_lttng_live_args.exists) {
- printf_err("Cannot create both implicit `%s` and `%s` components\n",
- implicit_ctf_input_args.comp_arg->str,
- implicit_lttng_live_args.comp_arg->str);
- goto error;
+ /* If --clock-offset was given, apply it to any src.ctf.fs component. */
+ if (ctf_fs_source_clock_class_offset_arg) {
+ int n;
+
+ n = append_multiple_implicit_components_param(
+ discovered_source_args, "source.ctf.fs", "clock-class-offset-s",
+ ctf_fs_source_clock_class_offset_arg);
+
+ if (n == 0) {
+ BT_CLI_LOGE_APPEND_CAUSE("--clock-offset specified, but no source.ctf.fs component instantiated.");
+ goto error;
+ }
}
- /*
- * If the implicit `source.ctf.fs` or `source.ctf.lttng-live`
- * components exists, make sure there's at least one leftover
- * (which is the path or URL).
- */
- if (implicit_ctf_input_args.exists && bt_value_array_is_empty(leftovers)) {
- printf_err("Missing path for implicit `%s` component\n",
- implicit_ctf_input_args.comp_arg->str);
- goto error;
+ /* If --clock-offset-ns was given, apply it to any src.ctf.fs component. */
+ if (ctf_fs_source_clock_class_offset_ns_arg) {
+ int n;
+
+ n = append_multiple_implicit_components_param(
+ discovered_source_args, "source.ctf.fs", "clock-class-offset-ns",
+ ctf_fs_source_clock_class_offset_ns_arg);
+
+ if (n == 0) {
+ BT_CLI_LOGE_APPEND_CAUSE("--clock-offset-ns specified, but no source.ctf.fs component instantiated.");
+ goto error;
+ }
}
+ /*
+ * If the implicit `source.ctf.lttng-live` component exists, make sure
+ * there's at least one leftover (which is the URL).
+ */
if (implicit_lttng_live_args.exists && bt_value_array_is_empty(leftovers)) {
- printf_err("Missing URL for implicit `%s` component\n",
+ BT_CLI_LOGE_APPEND_CAUSE("Missing URL for implicit `%s` component.",
implicit_lttng_live_args.comp_arg->str);
goto error;
}
/* Assign names to implicit components */
- ret = assign_name_to_implicit_component(&implicit_ctf_input_args,
- "source-ctf-fs", all_names, &source_names, true);
- if (ret) {
- goto error;
+ for (i = 0; i < discovered_source_args->len; i++) {
+ struct implicit_component_args *args;
+ int j;
+
+ args = discovered_source_args->pdata[i];
+
+ g_string_printf(auto_disc_comp_name, "auto-disc-%s", args->comp_arg->str);
+
+ /* Give it a name like `auto-disc-src-ctf-fs`. */
+ for (j = 0; j < auto_disc_comp_name->len; j++) {
+ if (auto_disc_comp_name->str[j] == '.') {
+ auto_disc_comp_name->str[j] = '-';
+ }
+ }
+
+ ret = assign_name_to_implicit_component(args,
+ auto_disc_comp_name->str, all_names, &source_names, true);
+ if (ret) {
+ goto error;
+ }
}
ret = assign_name_to_implicit_component(&implicit_lttng_live_args,
/* Make sure there's at least one source and one sink */
if (!source_names) {
- printf_err("No source component\n");
+ BT_CLI_LOGE_APPEND_CAUSE("No source component.");
goto error;
}
if (!sink_names) {
- printf_err("No sink component\n");
+ BT_CLI_LOGE_APPEND_CAUSE("No sink component.");
goto error;
}
* Append the equivalent run arguments for the implicit
* components.
*/
- ret = append_run_args_for_implicit_component(&implicit_ctf_input_args, run_args);
- if (ret) {
- goto error;
+ for (i = 0; i < discovered_source_args->len; i++) {
+ struct implicit_component_args *args =
+ discovered_source_args->pdata[i];
+
+ ret = append_run_args_for_implicit_component(args, run_args);
+ if (ret) {
+ goto error;
+ }
}
ret = append_run_args_for_implicit_component(&implicit_lttng_live_args,
ret = convert_auto_connect(run_args, source_names, filter_names,
sink_names);
if (ret) {
- printf_err("Cannot auto-connect components\n");
+ BT_CLI_LOGE_APPEND_CAUSE("Cannot auto-connect components.");
goto error;
}
*/
if (print_run_args || print_run_args_0) {
if (stream_intersection_mode) {
- printf_err("Cannot specify --stream-intersection with --run-args or --run-args-0\n");
+ BT_CLI_LOGE_APPEND_CAUSE("Cannot specify --stream-intersection with --run-args or --run-args-0.");
goto error;
}
BT_OBJECT_PUT_REF_AND_RESET(cfg);
end:
- if (pc) {
- poptFreeContext(pc);
+ /*
+ * If the log level is still unset at this point, set it to
+ * the program's default.
+ */
+ if (*default_log_level < 0) {
+ *default_log_level = cli_default_log_level;
}
- free(arg);
+ bt_argpar_parse_ret_fini(&argpar_parse_ret);
+
free(output);
- if (cur_name) {
- g_string_free(cur_name, TRUE);
+ if (component_arg_for_run) {
+ g_string_free(component_arg_for_run, TRUE);
}
- if (cur_name_prefix) {
- g_string_free(cur_name_prefix, TRUE);
+ if (name_gstr) {
+ g_string_free(name_gstr, TRUE);
}
bt_value_put_ref(run_args);
destroy_glist_of_gstring(filter_names);
destroy_glist_of_gstring(sink_names);
bt_value_put_ref(leftovers);
- finalize_implicit_component_args(&implicit_ctf_input_args);
finalize_implicit_component_args(&implicit_ctf_output_args);
finalize_implicit_component_args(&implicit_lttng_live_args);
finalize_implicit_component_args(&implicit_dummy_args);
finalize_implicit_component_args(&implicit_trimmer_args);
bt_value_put_ref(plugin_paths);
bt_common_destroy_lttng_live_url_parts(<tng_live_url_parts);
+ auto_source_discovery_fini(&auto_disc);
+
+ if (discovered_source_args) {
+ g_ptr_array_free(discovered_source_args, TRUE);
+ }
+
+ g_free(ctf_fs_source_clock_class_offset_arg);
+ g_free(ctf_fs_source_clock_class_offset_ns_arg);
+
+ if (auto_disc_comp_name) {
+ g_string_free(auto_disc_comp_name, TRUE);
+ }
+
return cfg;
}
{
struct bt_config *config = NULL;
int i;
- const char **command_argv = NULL;
+ int top_level_argc;
+ const char **top_level_argv;
int command_argc = -1;
+ const char **command_argv = NULL;
const char *command_name = NULL;
int default_log_level = -1;
+ struct bt_argpar_parse_ret argpar_parse_ret = { 0 };
+
+ /* Top-level option descriptions. */
+ static const struct bt_argpar_opt_descr descrs[] = {
+ { OPT_DEBUG, 'd', "debug", false },
+ { OPT_HELP, 'h', "help", false },
+ { OPT_LOG_LEVEL, 'l', "log-level", true },
+ { OPT_VERBOSE, 'v', "verbose", false },
+ { OPT_VERSION, 'V', "version", false},
+ BT_ARGPAR_OPT_DESCR_SENTINEL
+ };
enum command_type {
COMMAND_TYPE_NONE = -1,
goto end;
}
- for (i = 1; i < argc; i++) {
- const char *cur_arg = argv[i];
- const char *next_arg = i == (argc - 1) ? NULL : argv[i + 1];
-
- if (strcmp(cur_arg, "-d") == 0 ||
- strcmp(cur_arg, "--debug") == 0) {
- default_log_level = BT_LOG_VERBOSE;
- } else if (strcmp(cur_arg, "-v") == 0 ||
- strcmp(cur_arg, "--verbose") == 0) {
- if (default_log_level != BT_LOG_VERBOSE &&
- default_log_level != BT_LOG_DEBUG) {
- /*
- * Legacy: do not override a previous
- * --debug because --verbose and --debug
- * can be specified together (in this
- * case we want the lowest log level to
- * apply, VERBOSE).
- */
- default_log_level = BT_LOG_INFO;
- }
- } else if (strcmp(cur_arg, "--log-level") == 0 ||
- strcmp(cur_arg, "-l") == 0) {
- if (!next_arg) {
- printf_err("Missing log level value for --log-level option\n");
- *retcode = 1;
- goto end;
- }
+ /* Skip first argument, the name of the program. */
+ top_level_argc = argc - 1;
+ top_level_argv = argv + 1;
+ argpar_parse_ret = bt_argpar_parse(top_level_argc, top_level_argv,
+ descrs, false);
- default_log_level =
- bt_log_get_level_from_string(next_arg);
- if (default_log_level < 0) {
- printf_err("Invalid argument for --log-level option:\n %s\n",
- next_arg);
- *retcode = 1;
- goto end;
- }
-
- i++;
- } else if (strncmp(cur_arg, "--log-level=", 12) == 0) {
- const char *arg = &cur_arg[12];
-
- default_log_level = bt_log_get_level_from_string(arg);
- if (default_log_level < 0) {
- printf_err("Invalid argument for --log-level option:\n %s\n",
- arg);
- *retcode = 1;
- goto end;
- }
- } else if (strncmp(cur_arg, "-l", 2) == 0) {
- const char *arg = &cur_arg[2];
+ if (argpar_parse_ret.error) {
+ BT_CLI_LOGE_APPEND_CAUSE(
+ "While parsing command-line arguments: %s",
+ argpar_parse_ret.error->str);
+ goto error;
+ }
- default_log_level = bt_log_get_level_from_string(arg);
- if (default_log_level < 0) {
- printf_err("Invalid argument for --log-level option:\n %s\n",
- arg);
- *retcode = 1;
- goto end;
+ for (i = 0; i < argpar_parse_ret.items->len; i++) {
+ struct bt_argpar_item *item;
+
+ item = g_ptr_array_index(argpar_parse_ret.items, i);
+
+ if (item->type == BT_ARGPAR_ITEM_TYPE_OPT) {
+ struct bt_argpar_item_opt *item_opt =
+ (struct bt_argpar_item_opt *) item;
+
+ switch (item_opt->descr->id) {
+ case OPT_DEBUG:
+ default_log_level = BT_LOG_TRACE;
+ break;
+ case OPT_VERBOSE:
+ /*
+ * Legacy: do not override a previous
+ * --debug because --verbose and --debug
+ * can be specified together (in this
+ * case we want the lowest log level to
+ * apply, TRACE).
+ */
+ default_log_level = BT_LOG_INFO;
+ break;
+ case OPT_LOG_LEVEL:
+ default_log_level =
+ bt_log_get_level_from_string(item_opt->arg);
+ if (default_log_level < 0) {
+ BT_CLI_LOGE_APPEND_CAUSE(
+ "Invalid argument for --log-level option:\n %s",
+ item_opt->arg);
+ goto error;
+ }
+ break;
+ case OPT_VERSION:
+ print_version();
+ goto end;
+ case OPT_HELP:
+ print_gen_usage(stdout);
+ goto end;
}
- } else if (strcmp(cur_arg, "-V") == 0 ||
- strcmp(cur_arg, "--version") == 0) {
- print_version();
- goto end;
- } else if (strcmp(cur_arg, "-h") == 0 ||
- strcmp(cur_arg, "--help") == 0) {
- print_gen_usage(stdout);
- goto end;
- } else {
+ } else if (item->type == BT_ARGPAR_ITEM_TYPE_NON_OPT) {
+ struct bt_argpar_item_non_opt *item_non_opt =
+ (struct bt_argpar_item_non_opt *) item;
/*
* First unknown argument: is it a known command
* name?
*/
- command_argv = &argv[i];
- command_argc = argc - i;
+ command_argc =
+ top_level_argc - item_non_opt->orig_index - 1;
+ command_argv =
+ &top_level_argv[item_non_opt->orig_index + 1];
- if (strcmp(cur_arg, "convert") == 0) {
+ if (strcmp(item_non_opt->arg, "convert") == 0) {
command_type = COMMAND_TYPE_CONVERT;
- } else if (strcmp(cur_arg, "list-plugins") == 0) {
+ } else if (strcmp(item_non_opt->arg, "list-plugins") == 0) {
command_type = COMMAND_TYPE_LIST_PLUGINS;
- } else if (strcmp(cur_arg, "help") == 0) {
+ } else if (strcmp(item_non_opt->arg, "help") == 0) {
command_type = COMMAND_TYPE_HELP;
- } else if (strcmp(cur_arg, "query") == 0) {
+ } else if (strcmp(item_non_opt->arg, "query") == 0) {
command_type = COMMAND_TYPE_QUERY;
- } else if (strcmp(cur_arg, "run") == 0) {
+ } else if (strcmp(item_non_opt->arg, "run") == 0) {
command_type = COMMAND_TYPE_RUN;
} else {
/*
- * Unknown argument, but not a known
+ * Non-option argument, but not a known
* command name: assume the default
* `convert` command.
*/
command_type = COMMAND_TYPE_CONVERT;
command_name = "convert";
- command_argv = &argv[i - 1];
- command_argc = argc - i + 1;
+ command_argc++;
+ command_argv--;
}
break;
}
}
if (command_type == COMMAND_TYPE_NONE) {
+ if (argpar_parse_ret.ingested_orig_args == top_level_argc) {
+ /*
+ * We only got non-help, non-version general options
+ * like --verbose and --debug, without any other
+ * arguments, so we can't do anything useful: print the
+ * usage and quit.
+ */
+ print_gen_usage(stdout);
+ goto end;
+ }
+
/*
- * We only got non-help, non-version general options
- * like --verbose and --debug, without any other
- * arguments, so we can't do anything useful: print the
- * usage and quit.
+ * We stopped on an unknown option argument (and therefore
+ * didn't see a command name). Assume `convert` command.
*/
- print_gen_usage(stdout);
- goto end;
+ command_type = COMMAND_TYPE_CONVERT;
+ command_name = "convert";
+ command_argc =
+ top_level_argc - argpar_parse_ret.ingested_orig_args;
+ command_argv =
+ &top_level_argv[argpar_parse_ret.ingested_orig_args];
}
BT_ASSERT(command_argv);
}
if (config) {
- BT_ASSERT(default_log_level >= BT_LOG_VERBOSE);
+ BT_ASSERT(default_log_level >= BT_LOG_TRACE);
config->log_level = default_log_level;
config->command_name = command_name;
}
+ goto end;
+
+error:
+ *retcode = 1;
+
end:
+ bt_argpar_parse_ret_fini(&argpar_parse_ret);
bt_value_put_ref(initial_plugin_paths);
return config;
}