python3Packages.ovito: 3.0.0 -> 3.3.1#102250
Conversation
jonringer
left a comment
There was a problem hiding this comment.
I would much rather have the python interpreter parameterized
in all-packages, i would would just do python = python3;
|
do you mind squashing? Please follow CONTRIBUTING.md and manual#submitting-changes-making-patches and squash the fix-up commits. This can be done without However, |
I am squashing it at the moment. Easiest thing how to accept your suggestions in my local branch is to accept them via Github UI into remote branch and then update my local branch. |
What do you mean by this? This changeset does not touch all-packages ... I guess you meant this, right? (in - ovito = toPythonModule (pkgs.libsForQt5.callPackage ../development/python-modules/ovito { });
+ ovito = toPythonModule (pkgs.libsForQt5.callPackage ../development/python-modules/ovito { inherit python; }); |
|
sorry, for some reason I thought there was a top-level ovito |
|
oh in the original list, it just shows |
Motivation for this change
Version update, convert to Python 3 (part of #101964), fix build
Things done
sandboxinnix.confon non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"./result/bin/)nix path-info -Sbefore and after)