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.moderngl_window: init at 1.2.0 #67399
pythonPackages.moderngl_window: init at 1.2.0 #67399
Conversation
@GrahamcOfBorg build python37Packages.moderngl_window |
a6153f2
to
a6a1ba1
Compare
@GrahamcOfBorg build pythonPackages.moderngl_window python3Packages.moderngl_window |
@mmahut can this PR be merged? |
Sorry for late reply - please, do you mind splitting the package upgrade and the new package into two separate commits? |
a6a1ba1
to
2db6316
Compare
No problem, i've split them up. |
do you mind rebasing on top of master? there's been a lot of changes to python-modules since then
|
2db6316
to
83a0319
Compare
Done, rebased it on top of master. |
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.
nix-review
passes on NixOS (failures are related to other packages)
diff LGTM
[26 built (7 failed), 44 copied (927.0 MiB), 533.4 MiB DL]
error: build of '/nix/store/y3m0m0jwwx9nlm5321ay4cs96ykdrhbx-env.drv' failed
https://github.com/NixOS/nixpkgs/pull/67399
4 package failed to build:
python27Packages.baselines python37Packages.baselines python37Packages.rl-coach python38Packages.baselines
14 package were build:
printrun python27Packages.gym python27Packages.pyglet python27Packages.pytmx python37Packages.gym python37Packages.moderngl-window python37Packages.pyglet python37Packages.pytmx python37Packages.roboschool python38Packages.gym python38Packages.moderngl-window python38Packages.pyglet python38Packages.pytmx python38Packages.roboschool
@ofborg build printrun python27Packages.gym python27Packages.pyglet python27Packages.pytmx python37Packages.gym python37Packages.moderngl-window python37Packages.pyglet python37Packages.pytmx python37Packages.roboschool python38Packages.gym python38Packages.moderngl-window python38Packages.pyglet python38Packages.pytmx python38Packages.roboschool |
Motivation for this change
Examples of
pythonPackages.moderngl
use this package. When upgrading the package wanted to run the tests.This PR also upgrades
pythonPackages.pyglet
to1.4.2
because moderngl_window requires it.Tested this package (and the changes to
pyglet
) indirectly with examples of ModernGL.Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @