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-desktop: init at 1.0.35 #31335
Conversation
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.
It's a shame we both decided to package this on the same day, within 2 hours of each other...
|
||
# Patch signal | ||
patchelf --set-interpreter "$(cat $NIX_CC/nix-support/dynamic-linker)" $out/libexec/signal-desktop | ||
patchelf --set-rpath ${rpath}:$out/libexec $out/libexec/signal-desktop |
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.
Perhaps better to just use one patchelf
invocation
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.
Good point.
xlibs.libxcb | ||
zulu | ||
zulu8 | ||
] + ":${stdenv.cc.cc.lib}/lib64"; |
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.
Quite a few unnecessary imports here, see #31337 for a minimal set
unpackPhase = "true"; | ||
buildCommand = '' | ||
mkdir -p $out | ||
dpkg -x $src $out |
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.
I think this should be in the unpackPhase
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.
Yup, good point.
''; | ||
|
||
meta = { | ||
description = "Signal messenger for the desktop."; |
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.
I think the branding is Signal Desktop or something like that. Better to ask someone who knows?
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.
I used what you had for the description in your PR.
Thank you for the review @NickHu! |
Thanks guys! |
Motivation for this change
The Signal Desktop application was recently released and the community has no derivation for it.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)