From: "Lessley Dennington via GitGitGadget" <gitgitgadget@gmail.com>
To: git@vger.kernel.org
Cc: stolee@gmail.com, gitster@pobox.com,
johannes.schindelin@gmail.com, "Elijah Newren" <newren@gmail.com>,
"Taylor Blau" <me@ttaylorr.com>,
"SZEDER Gábor" <szeder.dev@gmail.com>,
"Lessley Dennington" <lessleydennington@gmail.com>
Subject: [PATCH v6 0/3] completion: sparse-checkout updates
Date: Fri, 04 Feb 2022 03:26:35 +0000 [thread overview]
Message-ID: <pull.1108.v6.git.1643945198.gitgitgadget@gmail.com> (raw)
In-Reply-To: <pull.1108.v5.git.1643921091.gitgitgadget@gmail.com>
This change updates custom tab completion for the sparse-checkout command.
Specifically, it corrects the following issues with the current method:
1. git sparse-checkout <TAB> results in an incomplete list of subcommands
(it is missing reapply and add).
2. Options for subcommands are not tab-completable.
3. git sparse-checkout set <TAB> and git sparse-checkout add <TAB> show
both file names and directory names. While this may be a less surprising
behavior for non-cone mode, we want to only show directories in cone
mode.
The first commit in this series is an intermediate step that fixes issues 1
and 2 above and introduces a simple fix for issue 3 with some performance
and unusual character-related caveats. The next commit adds a new
__gitcomp_directories method that fixes the performance-related caveat from
the first commit by completing just a single level of directories. The final
commit modifies __gitcomp_directories to handle unusual characters in
directory names.
Changes since V5
================
* Fix incorrect conditional that was causing failure of non-cone mode test
(and causing 'seen' CI to fail).
* Remove __git_comp_directories indentation changes between the second and
third commits.
Changes since V4
================
* Clarify that completing on files and directories is a less-surprising
behavior in non-cone mode (rather than definitely the correct behavior)
in commit message and cover letter.
* Add explicit --no-cone to non-cone mode sparse-checkout completion test.
* Replace use of __git_complete_index_file with a new version of
__gitcomp_directories that is able to handle unusual characters.
* Ensure that users running git sparse-checkout set --cone get cone
mode-style completion whether or not they already have the cone mode
config set.
* Refactor creation of sparse-checkout test repo into a sparse-checkout
setup test.
Changes since V3
================
* Update commit descriptions to begin with completion instead of
sparse-checkout.
* Remove explicit completion of --help, as this is inconsistent with the
rest of the completion script.
* Use __gitcomp_builtin to auto-complete subcommand options (and remove
option completion test from previous iterations, as it is no longer
necessary).
* Use __git_complete_index_file helper function in place of __gitcomp
"$(git ls-tree -d -r HEAD --name-only).
* If in cone-mode sparse checkout, only show directory completions. If in
non-cone mode sparse checkout show both files and directories.
Changes since V2
================
* Change use of $prev to $subcommand in _git_sparse_checkout() method in
git-completion.bash.
* State explicitly that directory completion applies in both cone and
non-cone mode in 'sparse-checkout: custom tab completion' commit
* Add new patch with __gitcomp_directories method to improve performance by
only outputting directories at the current level.
Changes since V1
================
* Rebase onto en/sparse-checkout-set.
* Add subcommand options (including --no-cone) for set and reapply.
* Extend 'sparse-checkout completes subcommand options' test to validate
new set/reapply subcommand options.
* No longer set index.sparse to false in 'sparse-checkout completes
directory names' test.
* Refactor creation of sparse-checkout repo into a setup test.
Thanks, Lessley
Lessley Dennington (3):
completion: address sparse-checkout issues
completion: improve sparse-checkout cone mode directory completion
completion: handle unusual characters for sparse-checkout
contrib/completion/git-completion.bash | 44 ++++++++--
t/t9902-completion.sh | 117 +++++++++++++++++++++++++
2 files changed, 153 insertions(+), 8 deletions(-)
base-commit: dfac9b609f86cd4f6ce896df9e1172d2a02cde48
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-1108%2Fldennington%2Fsparse-checkout-bash-completion-v6
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-1108/ldennington/sparse-checkout-bash-completion-v6
Pull-Request: https://github.com/gitgitgadget/git/pull/1108
Range-diff vs v5:
1: a1c46c763fd ! 1: 23390000dd3 completion: address sparse-checkout issues
@@ contrib/completion/git-completion.bash: _git_sparse_checkout ()
+ __gitcomp_builtin sparse-checkout_$subcommand "" "--"
;;
+ set,*|add,*)
-+ if [ $(__git config core.sparseCheckoutCone) ] ||
-+ [ "$(__git_find_on_cmdline --cone)" ]; then
++ if [ "$(__git config core.sparseCheckoutCone)" == "true" ] ||
++ [ -n "$(__git_find_on_cmdline --cone)" ]; then
+ __gitcomp "$(git ls-tree -d -r HEAD --name-only)"
+ fi
esac
2: 5b8d53402ae ! 2: 10a190ce6a1 completion: improve sparse-checkout cone mode directory completion
@@ contrib/completion/git-completion.bash: _git_show_branch ()
+__gitcomp_directories ()
+{
-+ local _tmp_dir _tmp_completions
++ local _tmp_dir _tmp_completions
+
-+ # Get the directory of the current token; this differs from dirname
-+ # in that it keeps up to the final trailing slash. If no slash found
-+ # that's fine too.
-+ [[ "$cur" =~ .*/ ]]
-+ _tmp_dir=$BASH_REMATCH
++ # Get the directory of the current token; this differs from dirname
++ # in that it keeps up to the final trailing slash. If no slash found
++ # that's fine too.
++ [[ "$cur" =~ .*/ ]]
++ _tmp_dir=$BASH_REMATCH
+
-+ # Find possible directory completions, adding trailing '/' characters
-+ _tmp_completions="$(git ls-tree -d --name-only HEAD $_tmp_dir |
-+ sed -e s%$%/%)"
++ # Find possible directory completions, adding trailing '/' characters
++ _tmp_completions="$(git ls-tree -d --name-only HEAD $_tmp_dir |
++ sed -e s%$%/%)"
+
-+ if [[ -n "$_tmp_completions" ]]; then
-+ # There were some directory completions, so find ones that
-+ # start with "$cur", the current token, and put those in COMPREPLY
-+ local i=0 c IFS=$' \t\n'
-+ for c in $_tmp_completions; do
-+ if [[ $c == "$cur"* ]]; then
-+ COMPREPLY+=("$c")
-+ fi
-+ done
-+ elif [[ "$cur" =~ /$ ]]; then
-+ # No possible further completions any deeper, so assume we're at
-+ # a leaf directory and just consider it complete
-+ __gitcomp_direct_append "$cur "
-+ fi
++ if [[ -n "$_tmp_completions" ]]; then
++ # There were some directory completions, so find ones that
++ # start with "$cur", the current token, and put those in COMPREPLY
++ local i=0 c IFS=$' \t\n'
++ for c in $_tmp_completions; do
++ if [[ $c == "$cur"* ]]; then
++ COMPREPLY+=("$c")
++ fi
++ done
++ elif [[ "$cur" =~ /$ ]]; then
++ # No possible further completions any deeper, so assume we're at
++ # a leaf directory and just consider it complete
++ __gitcomp_direct_append "$cur "
++ fi
+}
+
_git_sparse_checkout ()
@@ contrib/completion/git-completion.bash: _git_show_branch ()
local subcommands="list init set disable add reapply"
@@ contrib/completion/git-completion.bash: _git_sparse_checkout ()
set,*|add,*)
- if [ $(__git config core.sparseCheckoutCone) ] ||
- [ "$(__git_find_on_cmdline --cone)" ]; then
+ if [ "$(__git config core.sparseCheckoutCone)" == "true" ] ||
+ [ -n "$(__git_find_on_cmdline --cone)" ]; then
- __gitcomp "$(git ls-tree -d -r HEAD --name-only)"
+ __gitcomp_directories
fi
3: ddf5e583dd7 ! 3: e805dbb58cc completion: handle unusual characters for sparse-checkout
@@ contrib/completion/git-completion.bash: _git_show_branch ()
__gitcomp_directories ()
{
-- local _tmp_dir _tmp_completions
--
-- # Get the directory of the current token; this differs from dirname
-- # in that it keeps up to the final trailing slash. If no slash found
-- # that's fine too.
-- [[ "$cur" =~ .*/ ]]
-- _tmp_dir=$BASH_REMATCH
--
-- # Find possible directory completions, adding trailing '/' characters
-- _tmp_completions="$(git ls-tree -d --name-only HEAD $_tmp_dir |
-- sed -e s%$%/%)"
--
-- if [[ -n "$_tmp_completions" ]]; then
-- # There were some directory completions, so find ones that
-- # start with "$cur", the current token, and put those in COMPREPLY
-- local i=0 c IFS=$' \t\n'
-- for c in $_tmp_completions; do
-- if [[ $c == "$cur"* ]]; then
-- COMPREPLY+=("$c")
-- fi
-- done
-- elif [[ "$cur" =~ /$ ]]; then
-- # No possible further completions any deeper, so assume we're at
-- # a leaf directory and just consider it complete
-- __gitcomp_direct_append "$cur "
-- fi
+- local _tmp_dir _tmp_completions
+ local _tmp_dir _tmp_completions _found=0
-+
-+ # Get the directory of the current token; this differs from dirname
-+ # in that it keeps up to the final trailing slash. If no slash found
-+ # that's fine too.
-+ [[ "$cur" =~ .*/ ]]
-+ _tmp_dir=$BASH_REMATCH
-+
+
+ # Get the directory of the current token; this differs from dirname
+ # in that it keeps up to the final trailing slash. If no slash found
+@@ contrib/completion/git-completion.bash: __gitcomp_directories ()
+ [[ "$cur" =~ .*/ ]]
+ _tmp_dir=$BASH_REMATCH
+
+- # Find possible directory completions, adding trailing '/' characters
+- _tmp_completions="$(git ls-tree -d --name-only HEAD $_tmp_dir |
+- sed -e s%$%/%)"
+-
+- if [[ -n "$_tmp_completions" ]]; then
+- # There were some directory completions, so find ones that
+- # start with "$cur", the current token, and put those in COMPREPLY
+- local i=0 c IFS=$' \t\n'
+- for c in $_tmp_completions; do
+ # Find possible directory completions, adding trailing '/' characters,
+ # de-quoting, and handling unusual characters.
+ while IFS= read -r -d $'\0' c ; do
+ # If there are directory completions, find ones that start
+ # with "$cur", the current token, and put those in COMPREPLY
-+ if [[ $c == "$cur"* ]]; then
+ if [[ $c == "$cur"* ]]; then
+- COMPREPLY+=("$c")
+ COMPREPLY+=("$c/")
+ _found=1
-+ fi
+ fi
+- done
+- elif [[ "$cur" =~ /$ ]]; then
+ done < <(git ls-tree -z -d --name-only HEAD $_tmp_dir)
+
+ if [[ $_found == 0 ]] && [[ "$cur" =~ /$ ]]; then
-+ # No possible further completions any deeper, so assume we're at
-+ # a leaf directory and just consider it complete
-+ __gitcomp_direct_append "$cur "
-+ fi
- }
-
- _git_sparse_checkout ()
+ # No possible further completions any deeper, so assume we're at
+ # a leaf directory and just consider it complete
+ __gitcomp_direct_append "$cur "
## t/t9902-completion.sh ##
@@ t/t9902-completion.sh: test_expect_success 'cone mode sparse-checkout completes directory names' '
--
gitgitgadget
next prev parent reply other threads:[~2022-02-04 3:26 UTC|newest]
Thread overview: 95+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-12-30 0:32 [PATCH 0/2] sparse checkout: custom bash completion updates Lessley Dennington via GitGitGadget
2021-12-30 0:32 ` [PATCH 1/2] sparse-checkout: custom tab completion tests Lessley Dennington via GitGitGadget
2021-12-30 13:43 ` Derrick Stolee
2021-12-30 16:19 ` Lessley Dennington
2021-12-30 17:43 ` Derrick Stolee
2021-12-31 19:27 ` Elijah Newren
2022-01-04 19:19 ` Lessley Dennington
2021-12-30 0:32 ` [PATCH 2/2] sparse-checkout: custom tab completion Lessley Dennington via GitGitGadget
2021-12-30 13:50 ` Derrick Stolee
2021-12-30 16:24 ` Lessley Dennington
2021-12-30 19:26 ` [PATCH v2 0/2] sparse checkout: custom bash completion updates Lessley Dennington via GitGitGadget
2021-12-30 19:26 ` [PATCH v2 1/2] sparse-checkout: custom tab completion tests Lessley Dennington via GitGitGadget
2021-12-31 20:03 ` Elijah Newren
2021-12-31 22:20 ` Junio C Hamano
2021-12-31 22:25 ` Elijah Newren
2022-01-04 19:25 ` Lessley Dennington
2022-01-04 20:25 ` Elijah Newren
2022-01-05 14:05 ` Lessley Dennington
2022-01-04 19:24 ` Lessley Dennington
2021-12-30 19:26 ` [PATCH v2 2/2] sparse-checkout: custom tab completion Lessley Dennington via GitGitGadget
2021-12-31 22:52 ` Elijah Newren
2022-01-04 19:41 ` Lessley Dennington
2022-01-04 20:42 ` Elijah Newren
2022-01-05 20:20 ` Lessley Dennington
2022-01-05 22:55 ` Elijah Newren
2022-01-10 18:59 ` [PATCH v3 0/3] sparse checkout: custom bash completion updates Lessley Dennington via GitGitGadget
2022-01-10 18:59 ` [PATCH v3 1/3] sparse-checkout: custom tab completion tests Lessley Dennington via GitGitGadget
2022-01-10 18:59 ` [PATCH v3 2/3] sparse-checkout: custom tab completion Lessley Dennington via GitGitGadget
2022-01-15 9:57 ` SZEDER Gábor
2022-01-16 1:03 ` Elijah Newren
2022-01-16 22:13 ` Junio C Hamano
2022-01-17 18:14 ` Elijah Newren
2022-01-17 19:40 ` Junio C Hamano
2022-01-18 17:56 ` Lessley Dennington
2022-01-22 1:07 ` Lessley Dennington
2022-01-22 1:08 ` Lessley Dennington
2022-01-22 2:11 ` Lessley Dennington
2022-01-18 21:02 ` SZEDER Gábor
2022-01-18 21:43 ` Elijah Newren
2022-01-18 17:59 ` Lessley Dennington
2022-01-18 22:22 ` SZEDER Gábor
2022-01-18 23:30 ` Elijah Newren
2022-01-10 18:59 ` [PATCH v3 3/3] sparse-checkout: limit tab completion to a single level Lessley Dennington via GitGitGadget
2022-01-12 23:43 ` Lessley Dennington
2022-01-13 0:00 ` Junio C Hamano
2022-01-13 0:38 ` Elijah Newren
2022-01-13 19:02 ` Lessley Dennington
2022-01-10 20:38 ` [PATCH v3 0/3] sparse checkout: custom bash completion updates Elijah Newren
2022-01-11 17:17 ` Lessley Dennington
2022-01-11 19:45 ` Taylor Blau
2022-01-12 18:35 ` Lessley Dennington
2022-01-27 21:21 ` [PATCH v4 0/3] completion: sparse-checkout updates Lessley Dennington via GitGitGadget
2022-01-27 21:21 ` [PATCH v4 1/3] completion: add sparse-checkout tests Lessley Dennington via GitGitGadget
2022-01-28 0:08 ` Elijah Newren
2022-01-28 1:56 ` Junio C Hamano
2022-01-28 2:04 ` Elijah Newren
2022-01-27 21:21 ` [PATCH v4 2/3] completion: sparse-checkout updates Lessley Dennington via GitGitGadget
2022-01-28 1:21 ` Elijah Newren
2022-01-31 20:03 ` Lessley Dennington
2022-01-31 21:37 ` Elijah Newren
2022-01-27 21:21 ` [PATCH v4 3/3] completion: ensure cone mode completion with multiple <TAB>s Lessley Dennington via GitGitGadget
2022-01-28 1:53 ` Elijah Newren
2022-02-03 20:44 ` [PATCH v5 0/3] completion: sparse-checkout updates Lessley Dennington via GitGitGadget
2022-02-03 20:44 ` [PATCH v5 1/3] completion: address sparse-checkout issues Lessley Dennington via GitGitGadget
2022-02-03 23:52 ` Elijah Newren
2022-02-04 0:34 ` Lessley Dennington
2022-02-03 20:44 ` [PATCH v5 2/3] completion: improve sparse-checkout cone mode directory completion Lessley Dennington via GitGitGadget
2022-02-03 20:44 ` [PATCH v5 3/3] completion: handle unusual characters for sparse-checkout Lessley Dennington via GitGitGadget
2022-02-03 23:58 ` Elijah Newren
2022-02-04 0:37 ` Lessley Dennington
2022-02-04 4:25 ` Ævar Arnfjörð Bjarmason
2022-02-04 21:55 ` Junio C Hamano
2022-02-03 21:48 ` [PATCH v5 0/3] completion: sparse-checkout updates Junio C Hamano
2022-02-03 22:17 ` Lessley Dennington
2022-02-03 23:28 ` Junio C Hamano
2022-02-03 23:59 ` Lessley Dennington
2022-02-04 2:43 ` Lessley Dennington
2022-02-04 3:28 ` Lessley Dennington
2022-02-04 4:21 ` Ævar Arnfjörð Bjarmason
2022-02-04 3:26 ` Lessley Dennington via GitGitGadget [this message]
2022-02-04 3:26 ` [PATCH v6 1/3] completion: address sparse-checkout issues Lessley Dennington via GitGitGadget
2022-02-04 3:26 ` [PATCH v6 2/3] completion: improve sparse-checkout cone mode directory completion Lessley Dennington via GitGitGadget
2022-02-04 3:26 ` [PATCH v6 3/3] completion: handle unusual characters for sparse-checkout Lessley Dennington via GitGitGadget
2022-02-04 6:05 ` [PATCH v6 0/3] completion: sparse-checkout updates Elijah Newren
2022-02-04 17:04 ` Junio C Hamano
2022-02-04 17:55 ` Elijah Newren
2022-02-04 19:54 ` Junio C Hamano
2022-02-04 20:01 ` Elijah Newren
2022-02-04 21:47 ` Junio C Hamano
2022-02-07 17:31 ` [PATCH v7 " Lessley Dennington via GitGitGadget
2022-02-07 17:31 ` [PATCH v7 1/3] completion: address sparse-checkout issues Lessley Dennington via GitGitGadget
2022-02-07 17:31 ` [PATCH v7 2/3] completion: improve sparse-checkout cone mode directory completion Lessley Dennington via GitGitGadget
2022-02-07 17:31 ` [PATCH v7 3/3] completion: handle unusual characters for sparse-checkout Lessley Dennington via GitGitGadget
2022-04-06 9:42 ` Adam Dinwoodie
2022-02-08 4:16 ` [PATCH v7 0/3] completion: sparse-checkout updates Elijah Newren
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=pull.1108.v6.git.1643945198.gitgitgadget@gmail.com \
--to=gitgitgadget@gmail.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=johannes.schindelin@gmail.com \
--cc=lessleydennington@gmail.com \
--cc=me@ttaylorr.com \
--cc=newren@gmail.com \
--cc=stolee@gmail.com \
--cc=szeder.dev@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).