From: Simon Marchi <[email protected]>

Improve a little bit the dummy implementation of
_lttng_complete_sessions and add two calls where it can be useful.

Reported-by: Daniel U. Thibault <[email protected]>
Signed-off-by: Simon Marchi <[email protected]>
---
 extras/lttng-bash_completion |   14 +++++++++++++-
 1 files changed, 13 insertions(+), 1 deletions(-)

diff --git a/extras/lttng-bash_completion b/extras/lttng-bash_completion
index cc81557..bff8df7 100644
--- a/extras/lttng-bash_completion
+++ b/extras/lttng-bash_completion
@@ -16,7 +16,12 @@
 #
 
 _lttng_complete_sessions() {
-       # TODO, maybe have a lttng list --simple or something like that
+       # TODO
+       # This code does nothing for now. When there is a mecanism to get the
+       # existing sessions, use it to fill the sessions variable.
+       local sessions
+       sessions=""
+       COMPREPLY=( $(compgen -W "${sessions}" -- $cur) )
        return
 }
 
@@ -181,6 +186,9 @@ _lttng_cmd_list() {
                COMPREPLY=( $(compgen -W "${list_opts}" -- $cur) )
                return
                ;;
+       *)
+               _lttng_complete_sessions
+               return
        esac
 }
 
@@ -193,6 +201,10 @@ _lttng_cmd_setsession() {
                COMPREPLY=( $(compgen -W "${set_session_opts}" -- $cur) )
                return
                ;;
+       *)
+               _lttng_complete_sessions
+               return
+               ;;
        esac
 }
 
-- 
1.7.1


_______________________________________________
lttng-dev mailing list
[email protected]
http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev

Reply via email to