From 620c56e90e0392d85004ee5efb10e2f4e53c4984 Mon Sep 17 00:00:00 2001
From: Ng Zhi An <zhin@google.com>
Date: Tue, 19 May 2020 21:21:09 +0000
Subject: [PATCH] Fix git cl completion

__git_cl_all_commands is a variable, so it should use the variable
parameter, ${}, not a command substitution $().

Change-Id: I81fc2c73afd85cc1f737af00fbc294d933145e22
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2205456
Reviewed-by: Andrii Shyshkalov <tandrii@google.com>
Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>
Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
---
 git_cl_completion.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/git_cl_completion.sh b/git_cl_completion.sh
index 93677f4b61..86de6a5a98 100755
--- a/git_cl_completion.sh
+++ b/git_cl_completion.sh
@@ -30,5 +30,5 @@ __git_cl_compute_all_commands () {
 
 _git_cl () {
   __git_cl_compute_all_commands
-  __gitcomp_nl "$(__git_cl_all_commands)"
+  __gitcomp_nl "${__git_cl_all_commands}"
 }