Skip to content

ConfigStep: disable status.submoduleSummary when using GVFS protocol#383

Merged
derrickstolee merged 1 commit intomicrosoft:masterfrom
derrickstolee:status-submodule-summary
Jun 4, 2020
Merged

ConfigStep: disable status.submoduleSummary when using GVFS protocol#383
derrickstolee merged 1 commit intomicrosoft:masterfrom
derrickstolee:status-submodule-summary

Conversation

@derrickstolee
Copy link
Contributor

A VFS for Git user noticed extremely slow "git status" times, and the
root cause was that they had set status.submoduleSummary=true in their
global config. This causes two "git submodule" subprocesses, which are
very slow. The GVFS protocol does not allow submodules, so disable the
setting when using that protocol.

I investigating fixing this in Git proper, but there is no efficient
way to detect that we can ignore this setting since there could be
an untracked submodule.

A VFS for Git user noticed extremely slow "git status" times, and the
root cause was that they had set status.submoduleSummary=true in their
global config. This causes two "git submodule" subprocesses, which are
very slow. The GVFS protocol does not allow submodules, so disable the
setting when using that protocol.

I investigating fixing this in Git proper, but there is no efficient
way to detect that we can ignore this setting since there could be
an untracked submodule.

Signed-off-by: Derrick Stolee <dstolee@microsoft.com>
@derrickstolee derrickstolee marked this pull request as ready for review June 3, 2020 15:22
@jeffhostetler
Copy link
Contributor

are we setting this as a "git config --global ..." value? not trying to redesign gvfs, but would it be better to set this (and the ahead/behind value) as a repo-local config setting when the repo is cloned, so that Git behaves as expected on normal repos??

@jeffhostetler
Copy link
Contributor

i'm not criticizing the choice here, just I can't follow the large picture from the context available here.

@derrickstolee
Copy link
Contributor Author

are we setting this as a "git config --global ..." value? not trying to redesign gvfs, but would it be better to set this (and the ahead/behind value) as a repo-local config setting when the repo is cloned, so that Git behaves as expected on normal repos??

We are setting it locally to the repo. The experience before this is that a user has it set globally, so it applies locally. We are just trying to disable it locally, overriding their (possibly present) global value.

@jeffhostetler
Copy link
Contributor

ok. i think i mis-read the comment. we're setting it false locally to override any global setting that they may have.

@derrickstolee derrickstolee merged commit 318aaf1 into microsoft:master Jun 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants