-
Notifications
You must be signed in to change notification settings - Fork 720
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
fix LAMMPS 3Mar2020 easyconfigs using intel toolchain on AMD CPUs by patching out hardcoded -xHost #11577
Conversation
Test report by @boegel |
|
|
@boegel I do not have any idea how could I fix this |
@hajgato My home dir was close to being full, that may be the problem... I'll try again. |
@boegelbot please test @ generoso |
@boegel: Request for testing this PR well received on generoso PR test command '
Test results coming soon (I hope)... - notification for comment with ID 717460098 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
Adding configopts = "-DBUILD_DOC=off " solves the problem. I have the feeling that this problem specifically relates to our system. |
Test report by @boegel |
My bad, didn't look at the patch hard enough |
@hajgato That |
@boegelbot please test @ generoso |
@boegel: Request for testing this PR well received on generoso PR test command '
Test results coming soon (I hope)... - notification for comment with ID 912537738 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
Test report by @boegel |
Test report by @boegel |
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.
lgtm
Going in, thanks @hajgato! |
No description provided.