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
python3Packages.{irctokens,ircstates,ircrobots}: init #99521
Conversation
Result of 12 packages built:
|
pytest-mock | ||
]; | ||
|
||
disabledTests = lib.optional (!pythonOlder "3.8") [ |
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.
this is of shape [[string]]
disabledTests = lib.optional (!pythonOlder "3.8") [ | |
disabledTests = lib.optionals (!pythonOlder "3.8") [ |
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.
I always get those wrong. How do you remember which is which?
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.
optionals -> "many" -> an array
optional -> "one" -> singleton
checkInputs = [ | ||
pytest-asyncio | ||
#pytest-trio | ||
#pytest-tornasync | ||
#pytest-twisted | ||
pytestCheckHook | ||
#twisted | ||
]; |
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.
remove if not needed
checkInputs = [ | |
pytest-asyncio | |
#pytest-trio | |
#pytest-tornasync | |
#pytest-twisted | |
pytestCheckHook | |
#twisted | |
]; | |
checkInputs = [ | |
pytest-asyncio | |
pytestCheckHook | |
]; |
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.
Leftovers, had to choose one runtime to run the tests with. Removed!
ircstates | ||
async_stagger | ||
async-timeout | ||
] ++ lib.optional (pythonOlder "3.7") [ |
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.
] ++ lib.optional (pythonOlder "3.7") [ | |
] ++ lib.optionals (pythonOlder "3.7") [ |
0a0c5f4
to
5f9181b
Compare
propagatedBuildInputs = [ | ||
idna | ||
sniffio | ||
] ++ lib.optional (pythonOlder "3.8") [ |
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.
missed one ;)
] ++ lib.optional (pythonOlder "3.8") [ | |
] ++ lib.optionals (pythonOlder "3.8") [ |
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.
Oh, fixed btw.
Result of 12 packages built:
|
Result of 4 packages failed to build:
8 packages built:
|
@SuperSandro2000 Thanks. Your comment needs more build logs to be actionable. |
Sandbox failure:
|
Motivation for this change
I'm writing software depending on these libraries.
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)