-
-
Notifications
You must be signed in to change notification settings - Fork 15.3k
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
haskellPackages.dbus: multiple addresses patch #68529
Conversation
Can you justify this patch a bit? I'm still not convinced that applications should survive this change trivially. And if they should, it sounds more like an upstream issue. :) |
I'm not certain either, so I opened rblaze/haskell-dbus#37 :) |
I can't log into |
That is likely. You could start an xterm or i3 session and run it manually, then you should see some related error message. Or apply the patch locally and don't forget |
so far I tried to use gnome + xorg and tried to run xmonad from terminal there. I can confirm I see error related to dbus. When I rollback to the generation where xmonad works for me and try to run it under gnome I don't see this error but expected |
Now I can fully confirm that this turboMaCk/Dotfiles#19 resolved all issues for me. |
This sounds like an issue that upstream should make a decision about -- not we. |
Looks like they will, or they would also accept this patch, see linked issue above. |
Once they have accepted this patch, I'll be perfectly happy to apply it to our package, too. |
The PR was accepted and merged, haskell-dbus v1.2.11 was published. We should use that. |
We have |
Closing, this will be solved by an upstream update with this change instead. |
Motivation for this change
xmonad breaks for me because of #68498. This reintroduces an old patch removed in 87ec7bb (While multiple addresses was upstreamed as
parseAddresses
, that function is not actually used for the env variables, so the issue remains).Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @