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
signal-cli: enable dbus functionality #72829
Conversation
@GrahamcOfBorg build signal-cli |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This all looks fine and I confirmed that it works, at least after I set DBUS_SESSION_BUS_ADDRESS
(not present in my SSH session).
Since you're using and maintaining this more than I am, would you also like to add yourself or a co-author as a maintainer?
Sure! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, I just noticed that dbus_java
and libmatthew_java
are Linux-only, it would be good to keep this working on darwin in the spirit of not breaking things that worked previously.
a5943a1
to
41b2264
Compare
Personally I don't think I'd need to fix the breakage for darwin, as
But as this one is cheap I added some extra logic that hopefully enables the darwin package to still build. |
@GrahamcOfBorg build signal-cli |
pkgs/applications/networking/instant-messengers/signal-cli/default.nix
Outdated
Show resolved
Hide resolved
This closes NixOS#72737 Co-authored-by: Matthew Piziak <matthew-piziak@users.noreply.github.com> Co-authored-by: Kiwi <Kiwi@users.noreply.github.com>
41b2264
to
a39fbee
Compare
@GrahamcOfBorg build signal-cli |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! This looks fine and I confirmed that it still builds on macOS 10.14.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, so with a slightly modified #72948 (with jwoglom/npyscreen@d01ce55) signal-ncurses can utilize signal-cli.
So anybody wants to merge this? I guess this is low hanging fruit, as it got approved by two people. |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/packaging-help-with-dbus-for-intune/24591/2 |
Motivation for this change
Close #72737
Things done
Refined the code from #72737 (comment).
Now the following should work:
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @ivan