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

Fixing cmake python build with mpicxx #15

Merged
merged 4 commits into from
Jun 1, 2020

Conversation

mdavis36
Copy link
Collaborator

Fixing bug when building python with mpicxx, python tries to add intel flags even when underlying compiler is not icpc.

@mdavis36 mdavis36 force-pushed the feature/cmake-build-time-tpls branch from b4af9b5 to eb891db Compare May 15, 2020 13:50
@mdavis36 mdavis36 marked this pull request as ready for review June 1, 2020 16:44
@mdavis36 mdavis36 requested a review from jmikeowen June 1, 2020 16:44
Copy link
Collaborator

@jmikeowen jmikeowen left a comment

Choose a reason for hiding this comment

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

This all looks good to me. Any reason not to go ahead and merge to master?

@mdavis36 mdavis36 merged commit ee34e1c into master Jun 1, 2020
@mdavis36 mdavis36 deleted the feature/cmake-build-time-tpls branch June 15, 2020 20:09
jmikeowen pushed a commit that referenced this pull request Aug 14, 2020
* commit 'c2de9176089ed4dd2e17053dee2a1ca219c52db4':
  Changing from hard-wired python to CMake defined PYTHON_EXE
  Fixes for the new python source install function.
  Adding a central cmake function for installing Spheral python source files, and moved byte-compiling the results to that method.  This currently results redundant compileall calls, but seems to work.
  Adding a few new install tasks: byte compile all the installed Spheral python; make a "spheral" symlink to the python executable.
  Working on add pycompileall for Spheral python, and a spheral symlink.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants