-
Notifications
You must be signed in to change notification settings - Fork 189
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
Update installation instruction for mac #3732
Conversation
@mkuron can you maybe have a look at the command change, I don't understand that ports stuff... |
Codecov Report
@@ Coverage Diff @@
## python #3732 +/- ##
======================================
- Coverage 89% 89% -1%
======================================
Files 548 548
Lines 24455 24431 -24
======================================
- Hits 21779 21756 -23
+ Misses 2676 2675 -1
Continue to review full report at Codecov.
|
Cannot confirm this. The documentation was correct as-is. Maybe he needs to update his MacPorts. I believe that was changed a few months ago.
|
Even if the documentation is correct in some cases, it obviously does not cover all macport environments. |
I'd rather have it not reference fortran if not needed... |
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.
Okay, he is right. I seem to have had an OpenMPI package without Fortran bindings.
$ port install openmpi-default +gcc10
$ port select --list mpi
Available versions for mpi:
none
openmpi-mp-fortran (active)
The +gcc10
variant is the default, so Fortran bindings will be installed by default.
So the updated version is correct, yes? |
as reported by C. Holm