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.

Sign up for GitHub

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

Jump to bottom

fix sanity check for later versions of Molpro #2499

Merged
merged 1 commit into from
Jul 13, 2021
Merged

fix sanity check for later versions of Molpro #2499

merged 1 commit into from
Jul 13, 2021

Conversation

rungitta
Copy link

(created using eb --new-pr)

@migueldiascosta migueldiascosta added this to the next release (4.4.2?) milestone Jul 13, 2021
@migueldiascosta
Copy link
Member

lgtm - @rungitta can you submit a test report? i.e. running eb on local easyconfig(s) with --include-easyblocks-from-pr=2499 -u should upload the test reports to a gist and post a comment here

@rungitta
Copy link
Author

Test report by @rungitta

Overview of tested easyconfigs (in order)

Build succeeded for 0 out of 0 (1 easyconfigs in total)
bioint02.hpc.uio.no - Linux RHEL 8.4, x86_64, Intel(R) Xeon(R) CPU E5-2690 0 @ 2.90GHz (sandybridge), Python 3.6.8
See https://gist.github.com/a598d04c76c9f65f69eb04a56e41d6fa for a full test report.

@rungitta
Copy link
Author

Test report by @rungitta

Overview of tested easyconfigs (in order)

  • SUCCESS Molpro-mpp-2021.1.0.linux_x86_64_openmp.eb

Build succeeded for 1 out of 1 (1 easyconfigs in total)
bioint02.hpc.uio.no - Linux RHEL 8.4, x86_64, Intel(R) Xeon(R) CPU E5-2690 0 @ 2.90GHz (sandybridge), Python 3.6.8
See https://gist.github.com/c299c513b12505b0c20601f15a77aeee for a full test report.

migueldiascosta
migueldiascosta approved these changes Jul 13, 2021
View reviewed changes
Copy link
Member

@migueldiascosta migueldiascosta left a comment

Choose a reason for hiding this comment

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

lgtm

@migueldiascosta
Copy link
Member

Going in, thanks @rungitta!

@migueldiascosta migueldiascosta merged commit b0a7f95 into easybuilders:develop Jul 13, 2021
@boegel boegel changed the title fixed sanity check for later versions of molpro fixed sanity check for later versions of Molpro Aug 7, 2021
@boegel boegel changed the title fixed sanity check for later versions of Molpro fix sanity check for later versions of Molpro Aug 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@migueldiascosta migueldiascosta migueldiascosta approved these changes

Assignees
No one assigned
Labels
update
Projects
None yet
Milestone
4.4.2
Development

Successfully merging this pull request may close these issues.
2 participants
@rungitta @migueldiascosta