Skip to content
New issue

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

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

Already on GitHub? # to your account

Investigate filewatcher edge cases (appimage, wsl, etc.) #4713

Open
ndom91 opened this issue Aug 16, 2024 · 2 comments
Open

Investigate filewatcher edge cases (appimage, wsl, etc.) #4713

ndom91 opened this issue Aug 16, 2024 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@ndom91
Copy link
Contributor

ndom91 commented Aug 16, 2024

Version

0.12.20

Operating System

Linux

Distribution Method

AppImage (Linux)

Describe the issue

Edge cases:

  1. WSL, see: GitButler fails silently when attempting to add a repository from a network drive on Windows #4953
  2. AppImage:

When running GitButler via the AppImage, the file watcher sometimes does not work as expected. I then have to reload (Ctrl + R) the application to pick up changes.

I can't quite pin down how to reproduce this consistently, but i have noticed this tends to start to happen after the application has been open for a while.

Running inotifywait -m package.json, for example, to monitor and print inotify events on the package.json file shows that MODIFY events are fired as expected when writing to package.json, but GitButler running simultaneously via the AppImage doesn't display any modified files in my default branch.

How to reproduce

No response

Expected behavior

No response

Relevant log output

No response

@ndom91 ndom91 added the bug Something isn't working label Aug 16, 2024
@mtsgrd
Copy link
Contributor

mtsgrd commented Aug 17, 2024

Would you be able to check if e.g. modifying package.json generates any activity whatsoever? I would let it idle for a bit, then tail -f -n 0 on the log file, and save the change.

@ndom91 ndom91 changed the title Filewatcher not always working inside AppImage Investigate filewatcher edge cases (appimage, wsl, etc.) Sep 24, 2024
@ndom91 ndom91 self-assigned this Sep 24, 2024
@ndom91
Copy link
Contributor Author

ndom91 commented Sep 24, 2024

There's another edge case with WSL2 users here.

In that thread, @Byron had an interesting idea to test disk events ourselves on startup and if we don't get them, fallback to polling for example, more details.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants