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
calls: 0.1.5 -> 0.2.0 with init of callaudiod and feedbackd #107681
Conversation
d383e21
to
2c762bf
Compare
2c762bf
to
45ec818
Compare
Result of 3 packages built:
|
45ec818
to
3f994ec
Compare
3f994ec
to
070354c
Compare
Updated calls to 0.2.0 and fixed name and version for feedbackd to match the naming rules. Callaudiod cannot be updated, because calls depends on 0.0.4 this should change in the next calls version. |
Result of 3 packages built:
|
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.
one minor nitpick
Package builds and runs. Cant test the actual functionality. One thing i noticed is that when i close it, the process still runs until i kill it with CTRL+C.
|
0ef9c7a
to
5bf1872
Compare
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.
LGTM
Yeah the description thing makes sense to me, just curious does nixpkgs have any specific guidelines for descriptions? I usually just copy directly from upstream.
I think thats intended, it is supposed to be also be a daemon, so it doesn't kill on application close. Its so it can receive phone calls. |
That makes sense most of the time. We have a few simple rules. https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md#submitting-changes
i see. good |
5bf1872
to
30e3315
Compare
This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch). Result of 3 packages built:
|
Co-authored-by: davidak <davidak@users.noreply.github.com>
30e3315
to
2455293
Compare
Updated feedbackd to 2021-01-25 |
This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch). Result of 3 packages built:
|
Thanks for merging! |
Motivation for this change
To update calls and add its new dependencies: callaudiod and feedbackd.
The feedbackd package is from @masipcat's #94820, I just bumped the version. I can remove it if that PR gets merged first, but its necessary for this version of calls to build right.
Also callaudiod and feedbackd should probably be setup as modules at some point, possibly as a part of #88767. I might try my hand at it sometime.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)