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
robotframework: Add unit tests and optional dependencies #96873
base: master
Are you sure you want to change the base?
Conversation
fd92a4a
to
f739f59
Compare
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/mktime-out-of-range-during-python-unit-tests/8848/1 |
21be274
to
ef4b5da
Compare
Can you please fix the merge conflict? |
65b9793
to
8e75ac1
Compare
So sorry for the delay. Does this look good? |
This is a semi-automatic executed nixpkgs-review. If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC. Result of 30 packages failed to build and are new build failures:
I think it fails due to modeled failing which is fixed in #108084 |
Result of 30 packages failed to build:
|
I marked this as stale due to inactivity. → More info |
Fix commit message? |
Oh, and rebase on master to fix conflicts (related: 738e8eb). |
@bjornfor , How's that? |
{ lib, fetchFromGitHub, buildPythonPackage, jsonschema }: | ||
{ lib, fetchFromGitHub, buildPythonPackage, | ||
tkinter, xmlschema, docutils, pygments, pyyaml, enum34, enum-compat, pillow, lxml, jsonschema, | ||
python }: |
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.
'python' is unused, can be removed.
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 think tkinter is used for the Dialog library (or something), what are the other deps for? (Perhaps say something about that in the commit message.)
{ lib, fetchFromGitHub, buildPythonPackage, | ||
tkinter, xmlschema, docutils, pygments, pyyaml, enum34, enum-compat, pillow, lxml, jsonschema, | ||
python }: |
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, fetchFromGitHub, buildPythonPackage, | |
tkinter, xmlschema, docutils, pygments, pyyaml, enum34, enum-compat, pillow, lxml, jsonschema, | |
python }: | |
{ lib, fetchFromGitHub, buildPythonPackage, | |
, tkinter, xmlschema, docutils, pygments, pyyaml, enum34, enum-compat, pillow, lxml, jsonschema, | |
, python }: |
@@ -11,6 +13,8 @@ buildPythonPackage rec { | |||
sha256 = "1wqz7szbq2g3kkm7frwik4jb5m7517306sz8nxx8hxaw4n6y1i5d"; | |||
}; | |||
|
|||
propagatedBuildInputs = [ tkinter xmlschema docutils pygments pyyaml enum34 enum-compat pillow lxml ]; |
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 gate enum43 like lib.optional (pythonOlder "3.4") enum34
.
I marked this as stale due to inactivity. → More info |
Motivation for this change
The
robot
executable was failing to run my tests, because it needed a dependency called "tkinter." Simply adding it here fixed the issue.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)