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
pythonPackages.yq: update, repackage as library, fix tests #98282
Conversation
One remaining test issue. One test requires a certain exit code dependant on
https://github.com/kislyuk/yq/blob/v2.11.0/test/test.py#L87 Running the tests I get
https://github.com/kislyuk/yq/blob/v2.11.0/test/test.py#L46-L47 So it thinks the test is not executed from a tty. Is that correct for our tests? |
I would just disable it, as making an environment that is a tty I think is more construction than we care for. |
Replaced the |
Result of 3 packages built:
|
toml | ||
]; | ||
|
||
checkPhase = "pytest ./test/test.py"; | ||
# tests fails if stdin is not a tty | ||
checkPhase = "echo | script -c 'pytest ./test/test.py'"; |
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.
Patch no longer required.
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.
disgusting, but beautiful at the same time lol
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.
Unfortunately, the BSD script(1) doesn't have the -c
option causing this to fail on macOS. Probably something like the following is required on macOS:
checkPhase = "echo | script typescript pytest ./test/test.py";
Result of 3 packages built:
|
Motivation for this change
Part of our (@Mic92 and me) plan to package Mozilla TTS.
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)