Skip to content

openmpi: enable c++ bindings#94829

Merged
markuskowa merged 1 commit intoNixOS:masterfrom
matthuszagh:openmpi
Aug 7, 2020
Merged

openmpi: enable c++ bindings#94829
markuskowa merged 1 commit intoNixOS:masterfrom
matthuszagh:openmpi

Conversation

@matthuszagh
Copy link
Contributor

@matthuszagh matthuszagh commented Aug 7, 2020 •
edited
Loading

Motivation for this change

Part of this pr. @jonringer

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.

@matthuszagh matthuszagh marked this pull request as draft August 7, 2020 00:16
@matthuszagh matthuszagh marked this pull request as ready for review August 7, 2020 00:21
@matthuszagh matthuszagh requested a review from markuskowa August 7, 2020 00:21
@ofborg ofborg bot requested a review from 7c6f434c August 7, 2020 00:25
@ofborg ofborg bot added 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. and removed 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels Aug 7, 2020
markuskowa
markuskowa reviewed Aug 7, 2020
View reviewed changes
pkgs/development/libraries/openmpi/default.nix Outdated
Copy link
Member

@markuskowa markuskowa Aug 7, 2020 •
edited
Loading

Choose a reason for hiding this comment

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

We may not need to make it conditional but it can always be enabled. I will run nixpkgs-review on it to see if that affects other packages.

Copy link
Member

@markuskowa markuskowa Aug 7, 2020

Choose a reason for hiding this comment

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

It is OK to enable it per default. Let's remove the conditional here and always enable it on Linux.

Copy link
Contributor Author

@matthuszagh matthuszagh Aug 7, 2020

Choose a reason for hiding this comment

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

Thanks for the review! I've made the change.

@markuskowa markuskowa changed the title Openmpi openmpi: enable c++ bindings Aug 7, 2020
@markuskowa markuskowa self-assigned this Aug 7, 2020
@matthuszagh matthuszagh requested a review from markuskowa August 7, 2020 15:44
@ofborg ofborg bot added 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux. and removed 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. labels Aug 7, 2020
@markuskowa markuskowa merged commit b9bf9c9 into NixOS:master Aug 7, 2020
@matthuszagh matthuszagh deleted the openmpi branch August 7, 2020 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@7c6f434c 7c6f434c Awaiting requested review from 7c6f434c

@markuskowa markuskowa Awaiting requested review from markuskowa

Assignees

@markuskowa markuskowa

Labels

10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@matthuszagh @markuskowa