Skip to content

gitui triggers terminal activity all the time, even when it's inactive #764

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.

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

Closed
ghost opened this issue Jun 3, 2021 · 8 comments
Closed
Labels
question Further information is requested
Milestone

Comments

@ghost
Copy link

ghost commented Jun 3, 2021

Describe the bug
When I have gitui running in an iTerm tab, gitui triggers activity in the tab when I switch to another tab, despite the program being idle. I imagine it triggers for other terminal emulators, but I'm not in a position to easily test this.

To Reproduce
Steps to reproduce the behavior:

  1. Open an iTerm2 window with some tabs configured to show terminal activity
  2. Open gitui in a tab
  3. Switch to another tab
  4. Observe activity in gitui's tab despite the program being idle

Expected behavior
I would expect an idle program to not trigger activity in my terminal emulator.

Context (please complete the following information):

  • macOS 11.4
  • GitUI Version 0.16
  • Rust version: 1.52.1

Additional context
This happens in very large repos (e.g. a full clone of rust-lang/rust) and medium size repos (a work project), but not some nascent personal projects with just a few commits.

@ghost ghost mentioned this issue Jun 3, 2021
@extrawurst
Copy link
Collaborator

duplicate of #523

@extrawurst extrawurst added the duplicate This issue or pull request already exists label Jun 3, 2021
@ghost
Copy link
Author

ghost commented Jun 3, 2021 via email

@extrawurst
Copy link
Collaborator

I’m not sure this is the case

what does that mean, do you think you observe a different issue than #523 ?

@ghost
Copy link
Author

ghost commented Jun 3, 2021

Yes. My terminal is configured to flag activity on output, not based on whether the application in the terminal is on the CPU. If I delay output to the terminal (e.g. something like sleep 5; echo hi), and switch to another tab, in 5 seconds my terminal will flag for activity just like gitui does when it's inactive.

@extrawurst
Copy link
Collaborator

configured to flag activity

how does this work? will this be triggered if anything is updated on the terminal text buffer?

@extrawurst
Copy link
Collaborator

extrawurst commented Jun 3, 2021

are you referring to this iterm2 graphic?

Screenshot 2021-06-03 at 18 42 26

I checked and this the same behaviour as tig, top, and zenith do. I assume this is triggered by any textual changes to the terminal. we could add a --quiet argument that does not show the spinner indicating ongoing background activity.

@extrawurst extrawurst added enhancement New feature or request good first issue Good for newcomers question Further information is requested and removed duplicate This issue or pull request already exists enhancement New feature or request good first issue Good for newcomers labels Jun 3, 2021
@ghost
Copy link
Author

ghost commented Jun 5, 2021

Hello!

It's the new-output indicator, not the activity indicator, my apologies. It's right below the activity indicator in the preferences and looks like this:

image

@extrawurst
Copy link
Collaborator

@anahata0108 I found the reason why that indicator lighted up all the time, this is fixed and now aligned exactly with the 2 second interval of us polling for changes on the file system. this means now it is a dup of #1 and will be better once we have that implemented and do not need to poll anymore.

@extrawurst extrawurst added this to the v0.16.1 milestone Jun 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

1 participant