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.

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

lazygit --debug --> Potential Deadlock #4002

Open
1Jo1 opened this issue Oct 21, 2024 · 3 comments
Open

lazygit --debug --> Potential Deadlock #4002

1Jo1 opened this issue Oct 21, 2024 · 3 comments
Labels
bug Something isn't working

Comments

@1Jo1
Copy link

1Jo1 commented Oct 21, 2024

Describe the bug
when I execute lazygit --debug:

tenet@jobackend:  lazygit --debug
POTENTIAL DEADLOCK: Recursive locking:
current goroutine 2 lock 0xc0002a33d0
github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:504 helpers.(*RefreshHelper).refreshBranches ??? <<<<<
github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:503 helpers.(*RefreshHelper).refreshBranches ???
github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:273 helpers.(*RefreshHelper).refreshReflogAndBranches ???
github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:129 helpers.(*RefreshHelper).Refresh.func2.2 ???
github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:107 helpers.(*RefreshHelper).Refresh.func2.1.1 ???
github.com/jesseduffield/[email protected]/gui.go:702 gocui.(*Gui).onWorkerAux ???
github.com/jesseduffield/[email protected]/task.go:22 gocui.(*TaskImpl).Done ???

Previous place where the lock was grabbed (same goroutine)
github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:322 helpers.(*RefreshHelper).refreshCommitsWithLimit ??? <<<<<
github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:321 helpers.(*RefreshHelper).refreshCommitsWithLimit ???
github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:277 helpers.(*RefreshHelper).refreshCommitsAndCommitFiles ???
github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:107 helpers.(*RefreshHelper).Refresh.func2.1.1 ???
github.com/jesseduffield/[email protected]/gui.go:702 gocui.(*Gui).onWorkerAux ???
github.com/jesseduffield/[email protected]/task.go:22 gocui.(*TaskImpl).Done ???

Other goroutines holding locks:
goroutine 2 lock 0xc0002a33c0
github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:442 helpers.(*RefreshHelper).refreshBranches ??? <<<<<
github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:441 helpers.(*RefreshHelper).refreshBranches ???
github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:273 helpers.(*RefreshHelper).refreshReflogAndBranches ???
github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:129 helpers.(*RefreshHelper).Refresh.func2.2 ???
github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:107 helpers.(*RefreshHelper).Refresh.func2.1.1 ???
github.com/jesseduffield/[email protected]/gui.go:702 gocui.(*Gui).onWorkerAux ???
github.com/jesseduffield/[email protected]/task.go:22 gocui.(*TaskImpl).Done ???

goroutine 2 lock 0xc0002a33b8
github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:512 helpers.(*RefreshHelper).refreshFilesAndSubmodules ??? <<<<<
github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:511 helpers.(*RefreshHelper).refreshFilesAndSubmodules ???
github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:154 helpers.(*RefreshHelper).Refresh.func2.8 ???
github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:107 helpers.(*RefreshHelper).Refresh.func2.1.1 ???
github.com/jesseduffield/[email protected]/gui.go:702 gocui.(*Gui).onWorkerAux ???
github.com/jesseduffield/[email protected]/task.go:22 gocui.(*TaskImpl).Done ???

To Reproduce
see you above

Version info:
commit=v0.44.1, build date=2024-09-18T13:19:48Z, build source=binaryRelease, version=0.44.1, os=linux, arch=amd64, git version=2.47.0

@1Jo1 1Jo1 added the bug Something isn't working label Oct 21, 2024
@1Jo1
Copy link
Author

1Jo1 commented Oct 21, 2024

same issue with the newest lazygit version

tenet@jobackend   fix-husky lazygit --version
commit=095eb130e, build date=20241021.092648, build source=unknown, version=v0.44.1.095eb130e, os=linux, arch=amd64, git version=2.47.0
tenet@jobackend   fix-husky lazygit --debug
POTENTIAL DEADLOCK: Recursive locking:
current goroutine 2 lock 0xc00011d0b0
~/.cache/yay/lazygit-git/src/src/github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:505 helpers.(*RefreshHelper).refreshBranches ??? <<<<<
~/.cache/yay/lazygit-git/src/src/github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:504 helpers.(*RefreshHelper).refreshBranches ???
~/.cache/yay/lazygit-git/src/src/github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:273 helpers.(*RefreshHelper).refreshReflogAndBranches ???
~/.cache/yay/lazygit-git/src/src/github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:129 helpers.(*RefreshHelper).Refresh.func2.2 ???
~/.cache/yay/lazygit-git/src/src/github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:107 helpers.(*RefreshHelper).Refresh.func2.1.1 ???
~/.cache/yay/lazygit-git/src/src/github.com/jesseduffield/lazygit/vendor/github.com/jesseduffield/gocui/gui.go:702 gocui.(*Gui).onWorkerAux ???
~/.cache/yay/lazygit-git/src/src/github.com/jesseduffield/lazygit/vendor/github.com/jesseduffield/gocui/task.go:22 gocui.(*TaskImpl).Done ???

Previous place where the lock was grabbed (same goroutine)
~/.cache/yay/lazygit-git/src/src/github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:322 helpers.(*RefreshHelper).refreshCommitsWithLimit ??? <<<<<
~/.cache/yay/lazygit-git/src/src/github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:321 helpers.(*RefreshHelper).refreshCommitsWithLimit ???
~/.cache/yay/lazygit-git/src/src/github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:277 helpers.(*RefreshHelper).refreshCommitsAndCommitFiles ???
~/.cache/yay/lazygit-git/src/src/github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:107 helpers.(*RefreshHelper).Refresh.func2.1.1 ???
~/.cache/yay/lazygit-git/src/src/github.com/jesseduffield/lazygit/vendor/github.com/jesseduffield/gocui/gui.go:702 gocui.(*Gui).onWorkerAux ???
~/.cache/yay/lazygit-git/src/src/github.com/jesseduffield/lazygit/vendor/github.com/jesseduffield/gocui/task.go:22 gocui.(*TaskImpl).Done ???

Other goroutines holding locks:
goroutine 2 lock 0xc00011d0a0
~/.cache/yay/lazygit-git/src/src/github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:447 helpers.(*RefreshHelper).refreshBranches ??? <<<<<
~/.cache/yay/lazygit-git/src/src/github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:446 helpers.(*RefreshHelper).refreshBranches ???
~/.cache/yay/lazygit-git/src/src/github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:273 helpers.(*RefreshHelper).refreshReflogAndBranches ???
~/.cache/yay/lazygit-git/src/src/github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:129 helpers.(*RefreshHelper).Refresh.func2.2 ???
~/.cache/yay/lazygit-git/src/src/github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:107 helpers.(*RefreshHelper).Refresh.func2.1.1 ???
~/.cache/yay/lazygit-git/src/src/github.com/jesseduffield/lazygit/vendor/github.com/jesseduffield/gocui/gui.go:702 gocui.(*Gui).onWorkerAux ???
~/.cache/yay/lazygit-git/src/src/github.com/jesseduffield/lazygit/vendor/github.com/jesseduffield/gocui/task.go:22 gocui.(*TaskImpl).Done ???

goroutine 2 lock 0xc00011d098
~/.cache/yay/lazygit-git/src/src/github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:513 helpers.(*RefreshHelper).refreshFilesAndSubmodules ??? <<<<<
~/.cache/yay/lazygit-git/src/src/github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:512 helpers.(*RefreshHelper).refreshFilesAndSubmodules ???
~/.cache/yay/lazygit-git/src/src/github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:154 helpers.(*RefreshHelper).Refresh.func2.8 ???
~/.cache/yay/lazygit-git/src/src/github.com/jesseduffield/lazygit/pkg/gui/controllers/helpers/refresh_helper.go:107 helpers.(*RefreshHelper).Refresh.func2.1.1 ???
~/.cache/yay/lazygit-git/src/src/github.com/jesseduffield/lazygit/vendor/github.com/jesseduffield/gocui/gui.go:702 gocui.(*Gui).onWorkerAux ???
~/.cache/yay/lazygit-git/src/src/github.com/jesseduffield/lazygit/vendor/github.com/jesseduffield/gocui/task.go:22 gocui.(*TaskImpl).Done ???

@willparsons
Copy link

This seems to only occur when using go 1.23, just tested 1.22 and it works fine.

I'm not really familiar with the changes in 1.23 so I'm not sure why this version doesn't work.

@1Jo1
Copy link
Author

1Jo1 commented Nov 1, 2024

same for me :) maybe some breaking changes in 1.23

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

No branches or pull requests

2 participants