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
python38Packages.coloredlogs: disable tests on darwin #96037
python38Packages.coloredlogs: disable tests on darwin #96037
Conversation
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.
Please follow CONTRIBUTING.md and manual#submitting-changes-making-patches and squash the fix-up commits.
This can be done with:
git reset HEAD~1 # move fix-up commits into unstaged
git add -- pkgs/ # move changes into staged
git commit --amend --no-edit # add changes to previous commit
git push ... ... --force # modify current PR branch
a172866
to
18d1a80
Compare
@jonringer done. |
@tobim Please fix the merge conflict. |
d605d30
to
5cf8a80
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 19 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.
diff LGTM
leave darwin verification to @SuperSandro2000
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 10 packages failed to build and are new build failures:
5 packages built:
I think I needed to kill something in that nixpkgs-review because it hang for hours. |
Fixing the build on darwin / py38 by turning
capturer
andverboselogs
intocheckInputs
and disabling the test for this combo.This can be reverted after xolox/python-capturer#10 is fixed.
See
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)