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
nixos/services.imwheel: sleep 3s before restarting #95986
Conversation
dc44a3b
to
dcaa2d2
Compare
cc @infinisil |
What's the error when starting without this delay? |
There is no error. Even |
No logs in |
@infinisil no, only successes. This is the log from several reboots.
|
Hm well I guess if we can't figure out the actual cause, this is the next best thing. |
Oof and I'm also just noticing that this package hasn't seen an update in 16 years! Is there nothing more maintained out there? |
I'm not much happy about this either. It always feels deeply wrong to fix synchronization issues with sleep. The only reason why I was brave enough to open this PR was that I've seen Yes the package is old as hell. I'm not a maintainer of the package itself. That's being said I also used it several years ago on Arch (https://wiki.archlinux.org/index.php/IMWheel). I'm not really sure about alternatives but we can definitely look into it. Anyway in case we would find something neater I would still be in favor of merging this fix for folks out there and then adding some deprecation notice with recommendation of switching to alternative. |
Motivation for this change
Imwheel service required restarting (
systemctl; --user restart imwheel.service
) in order to start working. Probably because it was racing with some other services initialization.Things done
Delay imwheel start by 3s (value taken from redshift) to prevent the issue.
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)