Skip to content
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

pymol: init at 1.8.4 #23007

Merged
merged 2 commits into from
Feb 22, 2017
Merged

pymol: init at 1.8.4 #23007

merged 2 commits into from
Feb 22, 2017

Conversation

Mounium
Copy link
Contributor

@Mounium Mounium commented Feb 19, 2017

Motivation for this change
Things done
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@Mounium
Copy link
Contributor Author

Mounium commented Feb 19, 2017

Any idea on why have it failed? Maybe relevant: openai/gym#164

let
version = "1.8.4.0";
in
stdenv.mkDerivation {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is also buildPythonApplication in pythonPackages which should automate all the phases you have overriden. Have you tried that?


meta = {
description = "Molecule editor and visualizer";
};
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please include more information in the meta section. At least the license and the homepage should be present.

@Mounium
Copy link
Contributor Author

Mounium commented Feb 21, 2017

@fpletz Applied buildPhytonApplication, and added license, a bit more detailed description as well as the website. Although I still doubt that it will build on Mac.

Note: This version of PyMol does not utilises Python megawidgets, as I could not find this package on nixpkgs. It doesn't cause any problem:

  1. Python megawidgets (Pmw -- only required for the "external" GUI)

sha256 = "0yfj8g5yic9zz6f0bw2n8h6ifvgsn8qvhq84alixsi28wzppn55n";
};

buildInputs = [ python27Packages.numpy glew freeglut libpng libxml2.dev tk freetype ];
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adding libxml2 to buildInputs should automatically pull libxml2.dev as well.

let
version = "1.8.4.0";
in
python27Packages.buildPythonPackage {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

python27Packages.buildPythonApplication is probably wanted here.

@Mic92 Mic92 merged commit eb688ac into NixOS:master Feb 22, 2017
fpletz added a commit that referenced this pull request Feb 22, 2017
@Mounium Mounium deleted the pymol branch February 22, 2017 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants