Skip to content
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

kmon: fix checkPhase #99482

Closed
wants to merge 1 commit into from
Closed

kmon: fix checkPhase #99482

wants to merge 1 commit into from

Conversation

@jmpunkt
Copy link
Contributor

@jmpunkt jmpunkt commented Oct 3, 2020

Kmon contains tests which parse the given arguments. The argument
parser used in the test is the actual argument parser used by the Kmon
binary. By default Nix provides the --test-threads argument. This
argument can not be parsed by the Kmon parser, thus the test fails.

Motivation for this change

Building the package fails due to the checkPhase. By overwriting the checkPhase the problem is solved and the package builds properly.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.
Kmon contains tests which parse the given arguments. The argument
parser used in the test is the actual argument parser used by the Kmon
binary. By default Nix provides the `--test-threads` argument. This
argument can not be parsed by the Kmon parser, thus the test fails.
@mdevlamynck
Copy link
Contributor

@mdevlamynck mdevlamynck commented Oct 3, 2020

For information, it should be fixed in the next version if my PR is accepted orhun/kmon#23.

fadenb
fadenb approved these changes Oct 3, 2020
misuzu
misuzu approved these changes Oct 3, 2020
SuperSandro2000 added a commit to SuperSandro2000/nixpkgs that referenced this issue Oct 10, 2020
SuperSandro2000 added a commit to SuperSandro2000/nixpkgs that referenced this issue Oct 10, 2020
@SuperSandro2000 SuperSandro2000 mentioned this pull request Oct 10, 2020
10 tasks
@jmpunkt jmpunkt deleted the kmon-fix-test-phase branch Oct 12, 2020
p-h added a commit to p-h/nixpkgs that referenced this issue Oct 16, 2020
Closes NixOS#99482

(cherry picked from commit 2412389)
@p-h p-h mentioned this pull request Oct 16, 2020
10 tasks
jonringer added a commit that referenced this issue Oct 17, 2020
Closes #99482

(cherry picked from commit 2412389)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants