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
authy: init at 1.8.3 #113307
authy: init at 1.8.3 #113307
Conversation
26bdedc
to
2bc137e
Compare
Result of 1 package built:2 suggestions:
|
2bc137e
to
0377299
Compare
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 1 package built:
The following issues got detected with the above build packages. authy: gcc-unwrapped is a build tool so it likely goes to Near pkgs/applications/misc/authy/default.nix:51:10:
See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/build-tools-in-build-inputs.md |
8f2059f
to
1c74cfb
Compare
1c74cfb
to
8e372cd
Compare
Motivation for this change
Package Authy. Closes #112697
This is my first init package. I'd like some feedback on how things can be simplified. Adding a few things that I personally had difficulty with and/or learnt through the process:
buildInputs
in thelet
block due to whichautoPatchelfHook
could not find the required libraries.patchelf
manually andautoPatchelfHook
will take care of finding the required libraries based onbuildInputs
.authy
binary, we could run the electron app directly. So wrapped the app withelectron
spotify
helped as a reference forsnap
apps and am glad that it had good comments around it.I had support from quite a few helpful people over at the NixOS Discord which made this a bit easier as they also debugged it to help out in getting this sorted.
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)