mirror of
https://github.com/JakeStanger/ironbar.git
synced 2026-07-11 08:15:21 +01:00
[GH-ISSUE #166] Crash when tray is enabled #2893
Labels
No labels
A:Build
A:CI
A:Client
A:Config
A:Core
A:Documentation
A:Documentation
A:IPC
A:Testing
A:UX/UI
blocked
BREAKING CHANGE
duplicate
good first issue
GTK4
help wanted
invalid
M:Battery
M:Battery
M:Bindmode
M:Bluetooth
M:Brightness
M:Cairo
M:Clipboard
M:Clock
M:Clock
M:Custom
M:Focused
M:Keyboard
M:Launcher
M:Menu
M:Music
M:Music
M:Music
M:Network Manager
M:Notifications
M:SysInfo
M:Tray
M:Volume
M:Workspaces
partially resolved
P:Critical
P:High
P:Low
P:Medium
pull-request
T:Bug
T:Bug
T:Core Enhancement
T:Module Enhancement
T:New Module
T:Question
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
JakeStanger/ironbar#2893
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @TheSunCat on GitHub (May 27, 2023).
Original GitHub issue: https://github.com/JakeStanger/ironbar/issues/166
Describe the bug
If I enable the tray module, ironbar does not start.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
ironbar should start
System information:
Configuration
Config
Additional context
Crash log (with backtrace enabled):
tray_crash.txt
@JakeStanger commented on GitHub (May 27, 2023):
Interesting, I've not seen that before, but I've just tried it on my barebones Arch/Hyprland install and encountered the same thing.
I'm not able to properly test at the moment, but I should be able to over the next couple of days. In the meantime it might be worth checking for any dependencies that are required for the service (I'm not aware of any other than dbus), or seeing if it occurs in different distros/environments.
If the problem is replicable, it will likely be an issue with the stray library since that handles the dbus connection.
@TheSunCat commented on GitHub (May 27, 2023):
Thanks for looking into this. I am not familiar with dbus but I've had issues before with Discord being unable to open links due to a missing dbus environment variable. I don't remember what it was as it is not an issue in WebCord, but perhaps ironbar needs this, too?
@JakeStanger commented on GitHub (May 28, 2023):
I think that env var is possibly related to XDG, which would be separate to this. I've just updated Ironbar on my main machine and I'm now encountering the same thing, so it would seem something under the hood has changed.
@yavko commented on GitHub (May 29, 2023):
I'm also experiencing this!!

@JakeStanger commented on GitHub (May 29, 2023):
I've managed to create a minimal repro, and it seems
strayis crashing as soon as it tries to connect to DBus.@TheSunCat are you using the stable Hyprland package or
-git?@TheSunCat commented on GitHub (May 29, 2023):
I build Hyprland manually from git to add my own (unrelated) bugfixes.
@JakeStanger commented on GitHub (Jun 1, 2023):
I think from previous discussions oknozor is struggling to find time to maintain
stray. Work's wearing me out at the moment, but I'm hoping to have a look into the issue myself this weekend.@JakeStanger commented on GitHub (Jun 17, 2023):
I've opened https://github.com/oknozor/stray/pull/7 with some fixes that seem to coax the tray back into life.
I have just opened #193 here, which builds using the PR above. I'll merge that shortly. Because the fix relies on patching this crate, it will be necessary to install by means other than
cargo install, even if the version there gets bumped.Once the PR in stray is merged, I will be able to bump the package properly. I will leave this issue open until that's resolved.
@yavko FYI it looks like the build is also failing on Nix due to the patch. How easy it that to work around?
@yavko commented on GitHub (Jun 17, 2023):
Nix is failing because the checksum is being removed from the lockfile, this is probably due to using the patch option

The checksum however can be added manually to the package, as mentioned in the error message in CI
@JakeStanger commented on GitHub (Jun 17, 2023):
Cool, was able to add a hash into the
default.nixwhich sorted it. Merged!