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
keepassxc: 2.6.2 -> 2.6.3 #109800
keepassxc: 2.6.2 -> 2.6.3 #109800
Conversation
Result of 1 package built:
|
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 1 package failed to build and already failed to build on hydra master:
The following issues got detected with the above build packages. keepassxc: Please consider this feature to be alpha. A substituteInPlace with an unused --replace got detected:
Please check the offending substituteInPlace for typos or changes in source. Near pkgs/applications/misc/keepassx/community.nix:95:3:
See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/build-tools-in-build-inputs.md Near pkgs/applications/misc/keepassx/community.nix:85:3:
See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/missing-phase-hooks.md Near pkgs/applications/misc/keepassx/community.nix:127:5:
See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/unclear-gpl.md |
I fixed the issues detected by nixpkgs-hammering and can reproduce the darwin failure locally when using a sandboxed build. It seems like this is the same error as #97194. |
Are these fixes necessary for the version to progress? If not, can you separate the commits? |
It is fine to put them in the version update. Otherwise we would accumulate way to many commits in this repo. |
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 1 package built:
|
I created a backport pr in #110921. |
Motivation for this change
Version 2.6.3 was released a while ago.
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)