Merged in master, stable-2.10, and stable-2.9.

Thanks!
Jérémie

On Mon, May 28, 2018 at 05:31:48PM -0400, Jonathan Rajotte wrote:
> xmllint cli does not "easily" support namespace.
> 
> One can use the local_name() xpath function and other "trick".
> The simplest trick for bash completion is to ignore the namespace
> altogether.
> 
> Replacing "xmlns" by "ignore" does the job.
> 
> Signed-off-by: Jonathan Rajotte <jonathan.rajotte-jul...@efficios.com>
> ---
>  extras/lttng-bash_completion | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/extras/lttng-bash_completion b/extras/lttng-bash_completion
> index ef1a71ab..c7482c54 100644
> --- a/extras/lttng-bash_completion
> +++ b/extras/lttng-bash_completion
> @@ -18,7 +18,7 @@
>  # Generates COMPREPLY with the existing session names
>  _lttng_complete_sessions() {
>       local sessions
> -     sessions=$(lttng --mi xml list |  xmllint --xpath 
> "//command/output/sessions/session/name" - 2>/dev/null | sed  -e 
> 's/<name>//g' -e $'s/<\/name>/\\n/g')
> +     sessions=$(lttng --mi xml list | sed '2 s/xmlns/ignore/g' | xmllint 
> --xpath "//command/output/sessions/session/name" - 2>/dev/null | sed  -e 
> 's/<name>//g' -e $'s/<\/name>/\\n/g')
>       COMPREPLY=( $(compgen -W "${sessions}" -- $cur) )
>       return
>  }
> @@ -27,7 +27,7 @@ _lttng_complete_sessions() {
>  # Generates COMPREPLY with the available kernel event
>  _lttng_complete_kernel_events() {
>       local kernel_event
> -     kernel_event=$(lttng --mi xml list -k | xmllint --xpath 
> "//command/output/domains/domain[./type = 'KERNEL']/events/event/name" - 
> 2>/dev/null | sed -e "s/<name>//g" -e $"s/<\/name>/\\n/g")
> +     kernel_event=$(lttng --mi xml list -k |sed '2 s/xmlns/ignore/g' | 
> xmllint --xpath "//command/output/domains/domain[./type = 
> 'KERNEL']/events/event/name" - 2>/dev/null | sed -e "s/<name>//g" -e 
> $"s/<\/name>/\\n/g")
>       COMPREPLY=( $(compgen -W "${kernel_event}" -- $cur) )
>       return
>  }
> @@ -35,7 +35,7 @@ _lttng_complete_kernel_events() {
>  # Generates COMPREPLY with the available ust event
>  _lttng_complete_ust_events() {
>       local ust_event
> -     ust_event=$(lttng --mi xml list -u | xmllint --xpath 
> "//command/output/domains/domain[./type = 'UST']/pids/pid/events/event/name" 
> - 2>/dev/null | sed -e "s/<name>//g" -e $"s/<\/name>/\\n/g")
> +     ust_event=$(lttng --mi xml list -u | sed '2 s/xmlns/ignore/g' | xmllint 
> --xpath "//command/output/domains/domain[./type = 
> 'UST']/pids/pid/events/event/name" - 2>/dev/null | sed -e "s/<name>//g" -e 
> $"s/<\/name>/\\n/g")
>       COMPREPLY=( $(compgen -W "${ust_event}" -- $cur) )
>       return
>  }
> @@ -43,7 +43,7 @@ _lttng_complete_ust_events() {
>  # Generates COMPREPLY with the available jul event
>  _lttng_complete_jul_events() {
>       local jul_event
> -     jul_event=$(lttng --mi xml list -j | xmllint --xpath 
> "//command/output/domains/domain[./type = 'JUL']/pids/pid/events/event/name" 
> - 2>/dev/null | sed -e "s/<name>//g" -e $"s/<\/name>/\\n/g")
> +     jul_event=$(lttng --mi xml list -j | sed '2 s/xmlns/ignore/g' | xmllint 
> --xpath "//command/output/domains/domain[./type = 
> 'JUL']/pids/pid/events/event/name" - 2>/dev/null | sed -e "s/<name>//g" -e 
> $"s/<\/name>/\\n/g")
>       COMPREPLY=( $(compgen -W "${jul_event}" -- $cur) )
>       return
>  }
> -- 
> 2.17.0
> 
_______________________________________________
lttng-dev mailing list
lttng-dev@lists.lttng.org
https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev

Reply via email to