-
Notifications
You must be signed in to change notification settings - Fork 33
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
xidlehook's pulse module segfaulted #56
Comments
Not much to go on, but I suppose some of the unsafe code in the pulse module is doing this. |
yes I think so too, sometime pulse breaks xidlehook even i added --no audio param, should we create an issue for pulseaudio? |
No, it's probably not pulseaudio's fault - It's likely my usage of it is wrong. You should not be seeing it if you don't add |
Hi! How does latest master (e26137e) work for you? I was able to reproduce the issue and I think I fixed it |
I've build the latest master and testing it as we speak:
|
Seems to work fine on my side; will you release a new version so we can notify the AUR packager? |
I've build the latest version and test it, it works fine for me |
Unfortunately, this version also dumps core, but strangely enough there's little info:
|
These messages seem to contradict each other @LordChaos73. What is the status of the git version, is it working and ready to be released or not? The lack of stack trace could be because you ran |
I just cloned the repo from github and built the binary with
|
I noticed the master branch was updated, I've rebuild the binary and I'm testing it again. |
Try building without |
I've done that and I'm testing the new binary as we speak. Will keep you updated. |
|
Did you build without |
Yes, I build it with |
I don't know... Not sure how to gather more information here. Do you have pulseaudio up and running? |
Yes, pulseaudio is up-and-running on my system:
|
I did some more testing, when playing audio (spotify) the screen is locked regardless, so it seems xidlehook it isn't detecting it. I haven't had a coredump for a while now. I'm running the binary from the master, not the AUR one. |
Disregard my previous post; the screen got locked because my monitor went to sleep, which killed the HDMI audio. So detecting sound seems to work fine. I haven't had a coredump in the past 20 hours, so it is all good so far. |
Are you running xidlehook normally or via systemd? I've had users have issues with systemd in the past |
xidlehook is started by i3:
|
The current version of master is already much more stable compared to the version in the AUR, I haven't had a coredump in a while. So to me it makes sense to push a new release. |
released :) |
great work! waiting for AUR update |
Unfortunately I still get quite a few coredumps, especially when using the 5.11 kernel. When I was using the 5.10 LTS kernel, I almost had no coredumps from xidlehook. Unfortunately, I have no clue how I can debug this further.
|
an 15 05:14:38 kn systemd-coredump[677499]: Process 238040 (xidlehook) of user 1000 dumped core.
The text was updated successfully, but these errors were encountered: