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

Waybar crashes on startup because of rfkill #822

Closed
W-joe opened this issue Aug 14, 2020 · 6 comments
Closed

Waybar crashes on startup because of rfkill #822

W-joe opened this issue Aug 14, 2020 · 6 comments
Labels
bug Something isn't working

Comments

@W-joe
Copy link

W-joe commented Aug 14, 2020

I don't know what rfkill is and I don't want to know.
I went by just fine without it and I'm living a happy life.
However since waybar 0.9.3, more specifically from the change log Network: Added Rfkill status #568 @marcplustwo, waybar refuses to start which makes me sad.

@W-joe W-joe mentioned this issue Aug 14, 2020
@Alexays Alexays added the bug Something isn't working label Aug 14, 2020
@W-joe
Copy link
Author

W-joe commented Aug 14, 2020

P.S. this is my first Issue/Pull request on github, so please bear with my mistakes and I'd appreciate it if those were pointed out to me. Thanks.

@Alexays
Copy link
Owner

Alexays commented Aug 14, 2020

Perfect!

@Alexays Alexays closed this as completed Aug 14, 2020
@theduke
Copy link

theduke commented Sep 2, 2020

@Alexays I'm getting the same crash:

terminate called after throwing an instance of 'std::runtime_error'
  what():  Can't open RFKILL control device

A compile time flag is a somewhat odd solution here.

I disabled the network module for now, which does allow Waybar to start.

But that's obviously suboptimal.

@Alexays Alexays reopened this Sep 2, 2020
@W-joe
Copy link
Author

W-joe commented Sep 5, 2020

@theduke not wanting to pay for what you don't use isn't odd at all. Just think any custom Linux kernel that doesn't include unwanted/unneeded functionality.
Or source based Linux distributions where you can just disable unneeded features via use flags or some such.

So from my point of view this 'rfkill optional' works as advertised and, at least for me, perfectly fine.

If you come from a binary distribution's point of view one could argue it's incomplete and, indeed, it does still crash for the very same reasons as before.
But the problem is with the incomplete implementation of #568 by @marcplustwo who doesn't check whether or not rfkill is available prior to using it.

Anyways if you need a quick fix maybe make the author of the feature aware of the issue as he is much more familiar with #568 and the problem domain or look into it yourself and make a pull request because I won't be able to allocate a time slot for this any time soon.

@mcepl
Copy link
Contributor

mcepl commented Mar 14, 2021

I guess a duplicate of this bug is https://bugzilla.suse.com/show_bug.cgi?id=1183499 (with the backtrace attached).

@mcepl
Copy link
Contributor

mcepl commented Mar 14, 2021

And it is probably a duplicate of #994, at least the patch from #1015 fixed it for me.

@Alexays Alexays closed this as completed Apr 16, 2021
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

4 participants