* [PATCH] command-list.txt: remove 'sparse-index' from main help
@ 2021-10-24 17:07 SZEDER Gábor
2021-10-24 17:27 ` Martin Ågren
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: SZEDER Gábor @ 2021-10-24 17:07 UTC (permalink / raw)
To: git; +Cc: SZEDER Gábor
Ever since 'git sparse-checkout' was introduced [1] it is included in
'git --help' in the section "work on the current change" along with
the commands 'add', 'mv', 'restore', and 'rm'. It clearly doesn't
belong to that group, moreover it can't be considered such a common
command to belong to 'git --help' in the first place, so remove it
from there.
[1] 94c0956b60 (sparse-checkout: create builtin with 'list'
subcommand, 2019-11-21)
Signed-off-by: SZEDER Gábor <szeder.dev@gmail.com>
---
command-list.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/command-list.txt b/command-list.txt
index a289f09ed6..eb9cee8dee 100644
--- a/command-list.txt
+++ b/command-list.txt
@@ -168,7 +168,7 @@ git-show-index plumbinginterrogators
git-show-ref plumbinginterrogators
git-sh-i18n purehelpers
git-sh-setup purehelpers
-git-sparse-checkout mainporcelain worktree
+git-sparse-checkout mainporcelain
git-stash mainporcelain
git-stage complete
git-status mainporcelain info
--
2.33.1.1176.g03b4fe3d2b
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] command-list.txt: remove 'sparse-index' from main help
2021-10-24 17:07 [PATCH] command-list.txt: remove 'sparse-index' from main help SZEDER Gábor
@ 2021-10-24 17:27 ` Martin Ågren
2021-10-25 15:33 ` Derrick Stolee
2021-10-25 16:10 ` Junio C Hamano
2 siblings, 0 replies; 4+ messages in thread
From: Martin Ågren @ 2021-10-24 17:27 UTC (permalink / raw)
To: SZEDER Gábor; +Cc: Git Mailing List
On Sun, 24 Oct 2021 at 19:08, SZEDER Gábor <szeder.dev@gmail.com> wrote:
>
> Ever since 'git sparse-checkout' was introduced [1] it is included in
> 'git --help' in the section "work on the current change" along with
> the commands 'add', 'mv', 'restore', and 'rm'. It clearly doesn't
> belong to that group, moreover it can't be considered such a common
> command to belong to 'git --help' in the first place, so remove it
> from there.
I actually spotted this in `git --help` just the other day and reacted
along the same line. So for what it's worth, this patch makes sense to
me.
Incidentally, this drops quite some indentation in `git --help` (because
"sparse-checkout" is by far the longest subcommand listed there). After
this patch, we're back within 80 columns again.
Martin
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] command-list.txt: remove 'sparse-index' from main help
2021-10-24 17:07 [PATCH] command-list.txt: remove 'sparse-index' from main help SZEDER Gábor
2021-10-24 17:27 ` Martin Ågren
@ 2021-10-25 15:33 ` Derrick Stolee
2021-10-25 16:10 ` Junio C Hamano
2 siblings, 0 replies; 4+ messages in thread
From: Derrick Stolee @ 2021-10-25 15:33 UTC (permalink / raw)
To: SZEDER Gábor, git
On 10/24/2021 1:07 PM, SZEDER Gábor wrote:
> Ever since 'git sparse-checkout' was introduced [1] it is included in
> 'git --help' in the section "work on the current change" along with
> the commands 'add', 'mv', 'restore', and 'rm'. It clearly doesn't
> belong to that group, moreover it can't be considered such a common
> command to belong to 'git --help' in the first place, so remove it
> from there.
>
> [1] 94c0956b60 (sparse-checkout: create builtin with 'list'
> subcommand, 2019-11-21)
> -git-sparse-checkout mainporcelain worktree
> +git-sparse-checkout mainporcelain
Thanks! I didn't realize what a pain it would be to include the
'worktree' grouping here.
-Stolee
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] command-list.txt: remove 'sparse-index' from main help
2021-10-24 17:07 [PATCH] command-list.txt: remove 'sparse-index' from main help SZEDER Gábor
2021-10-24 17:27 ` Martin Ågren
2021-10-25 15:33 ` Derrick Stolee
@ 2021-10-25 16:10 ` Junio C Hamano
2 siblings, 0 replies; 4+ messages in thread
From: Junio C Hamano @ 2021-10-25 16:10 UTC (permalink / raw)
To: Derrick Stolee; +Cc: git, SZEDER Gábor
SZEDER Gábor <szeder.dev@gmail.com> writes:
> Ever since 'git sparse-checkout' was introduced [1] it is included in
> 'git --help' in the section "work on the current change" along with
> the commands 'add', 'mv', 'restore', and 'rm'. It clearly doesn't
> belong to that group, moreover it can't be considered such a common
> command to belong to 'git --help' in the first place, so remove it
> from there.
>
> [1] 94c0956b60 (sparse-checkout: create builtin with 'list'
> subcommand, 2019-11-21)
>
> Signed-off-by: SZEDER Gábor <szeder.dev@gmail.com>
> ---
> command-list.txt | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Redirecting to Stolee just in case for an Ack.
I tend to agree that this is a housekeeping command for a local
repository to control what are and what are not materialized in the
working tree. It does not directly affect the history produced from
there, similar to "git gc" is, so in that sense, I do not think we
would miss it in the "list of usual commands".
If I had to have it in some category, I do agree with the current
code that it belongs to the "worktree" family, though.
Thanks, will queue.
> diff --git a/command-list.txt b/command-list.txt
> index a289f09ed6..eb9cee8dee 100644
> --- a/command-list.txt
> +++ b/command-list.txt
> @@ -168,7 +168,7 @@ git-show-index plumbinginterrogators
> git-show-ref plumbinginterrogators
> git-sh-i18n purehelpers
> git-sh-setup purehelpers
> -git-sparse-checkout mainporcelain worktree
> +git-sparse-checkout mainporcelain
> git-stash mainporcelain
> git-stage complete
> git-status mainporcelain info
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-10-25 16:10 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-24 17:07 [PATCH] command-list.txt: remove 'sparse-index' from main help SZEDER Gábor
2021-10-24 17:27 ` Martin Ågren
2021-10-25 15:33 ` Derrick Stolee
2021-10-25 16:10 ` Junio C Hamano
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).