From ffbf37df326cbe1b2cb2ff27fae8fbff0c8f9bcc Mon Sep 17 00:00:00 2001 From: David Goulet Date: Thu, 2 Feb 2012 10:19:04 -0500 Subject: [PATCH] Revert "Simplify create_session of lttng create command" This reverts commit 318e4c832cc9a4209dcea885ef2a66cfd16f99d2. The create_session function was changed making the session name not having the date and time append to it anymore. This completely broke the default behavior of automatic session creation. Only one possible session name was 'auto'. Signed-off-by: David Goulet --- src/bin/lttng/commands/create.c | 21 +++++++++++++-------- 1 file changed, 13 insertions(+), 8 deletions(-) diff --git a/src/bin/lttng/commands/create.c b/src/bin/lttng/commands/create.c index f60fe619e..8052826a2 100644 --- a/src/bin/lttng/commands/create.c +++ b/src/bin/lttng/commands/create.c @@ -67,7 +67,7 @@ static void usage(FILE *ofp) */ static int create_session() { - int ret; + int ret, have_name = 0; char datetime[16]; char *session_name, *traces_path = NULL, *alloc_path = NULL; time_t rawtime; @@ -80,32 +80,37 @@ static int create_session() /* Auto session name creation */ if (opt_session_name == NULL) { - ret = asprintf(&session_name, "auto"); + ret = asprintf(&session_name, "auto-%s", datetime); if (ret < 0) { perror("asprintf session name"); - ret = CMD_ERROR; goto error; } DBG("Auto session name set to %s", session_name); } else { session_name = opt_session_name; + have_name = 1; } /* Auto output path */ if (opt_output_path == NULL) { alloc_path = strdup(config_get_default_path()); if (alloc_path == NULL) { - ERR("Home path not found.\n" - "Please specify an output path using -o, --output PATH\n"); + ERR("Home path not found.\n \ + Please specify an output path using -o, --output PATH"); ret = CMD_FATAL; goto error; } - ret = asprintf(&traces_path, "%s/" DEFAULT_TRACE_DIR_NAME "/%s-%s", - alloc_path, session_name, datetime); + if (have_name) { + ret = asprintf(&traces_path, "%s/" DEFAULT_TRACE_DIR_NAME + "/%s-%s", alloc_path, session_name, datetime); + } else { + ret = asprintf(&traces_path, "%s/" DEFAULT_TRACE_DIR_NAME + "/%s", alloc_path, session_name); + } + if (ret < 0) { perror("asprintf trace dir name"); - ret = CMD_ERROR; goto error; } } else { -- 2.34.1