Skip to content

Config: disable status.submoduleSummary#1674

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

Config: disable status.submoduleSummary#1674
derrickstolee merged 1 commit intomicrosoft:masterfrom
derrickstolee:status-submodule-summary

Conversation

@derrickstolee
Copy link
Contributor

The GVFS protocol does not support submodules, so disable the
status.submoduleSummary config setting. A user had this enabled
in their global config, but that caused "git status" to be much
slower. When enabled, this runs two "git submodule" subprocesses,
which are very slow.

I investigated disabling these subprocesses in Git when the repo
has no submodules, but it needs to run the processes to check for
untracked submodules. Thus, there is no efficient way to detect
that submodules exist or not in advance.

The GVFS protocol does not support submodules, so disable the
status.submoduleSummary config setting. A user had this enabled
in their global config, but that caused "git status" to be much
slower. When enabled, this runs two "git submodule" subprocesses,
which are very slow.

I investigated disabling these subprocesses in Git when the repo
has no submodules, but it needs to run the processes to check for
untracked submodules. Thus, there is no efficient way to detect
that submodules exist or not in advance.

Signed-off-by: Derrick Stolee <dstolee@microsoft.com>
@derrickstolee derrickstolee marked this pull request as ready for review June 3, 2020 15:22
@derrickstolee derrickstolee merged commit 88588e1 into microsoft:master Jun 4, 2020
derrickstolee added a commit that referenced this pull request Jul 28, 2020
Major Updates
-----------------

* Comes with Git for Windows 2.28.0
* Previous builds were not including fully-signed DLLs. This is now fixed.

Pull Requests
---------------

* #1672: ProductUpgrader: be extra careful about a copy loop
* #1674: Config: disable status.submoduleSummary
* #1675: PreCommandHook: don't lock pack-objects 
* #1676: Update Git to v2.27.0.vfs.1.0 
*  #1679: BuildGVFSForWindows.bat: use nuget on PATH
* #1683: GVFS.SignFiles.csproj: include more DLLs for signing
* #1685: minor path comparison fixup in ProductUpgrader
*  #1682: Update Git to v2.28.0
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