Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Status takes a long time to load with many submodules #1152

Closed
bentitmus opened this issue Sep 29, 2021 · 0 comments · Fixed by #1155
Closed

Status takes a long time to load with many submodules #1152

bentitmus opened this issue Sep 29, 2021 · 0 comments · Fixed by #1155

Comments

@bentitmus
Copy link

bentitmus commented Sep 29, 2021 •
edited
Loading

The repository I'm using has lots of submodules. The normal git status command takes a long time to run, but if I set diff.ignoreSubmodules = all in my git config it runs quickly. This doesn't work with tig though because tig uses git diff-files which doesn't use the value of diff.ignoreSubmodules = all.

I can reproduce this manually.

  • If I try running git diff-files -z it takes a long time.
  • If I run git diff-files --ignore-submodules -z it is quite a lot faster.

Unfortunately there isn't any option for me to add the --ignore-submodules argument to tig as far as I can tell.
koutcher added a commit to koutcher/tig that referenced this issue Oct 6, 2021
@koutcher
Pass command line args through to diff-files in status view
cf17f20 
This makes it possible to use `tig --ignore-submodules` to speed up
the status view for a repo with many submodules.

Closes jonas#1152
@koutcher koutcher mentioned this issue Oct 6, 2021
Merged
koutcher added a commit that referenced this issue Nov 11, 2021
@koutcher
Pass command line args through to diff-files in status view (#1155)
4885101 
This makes it possible to use `tig --ignore-submodules` to speed up
the status view for a repo with many submodules.

Closes #1152

Reported-by: Benjamin Titmus <ben.titmus@cantab.net>
Reviewed-by: Johannes Altmanninger <aclopte@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Pass command line args through to diff-files in status view koutcher/tig
1 participant
@bentitmus