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

Fix our legion config to build against our hdf5 #90

Merged
merged 1 commit into from
Nov 25, 2019

Conversation

cferenba
Copy link
Member

@cferenba cferenba commented Nov 8, 2019

This modifies the configuration for legion to correctly find our hdf5 build (if requested).

@ktsai7 @ipdemes : please review. @opensdh : FYI.

@cferenba
Copy link
Member Author

As I suspected, something has changed in the environment in the past few months, such that the HEAD of master passed CI on Aug 9, but doesn't pass now (see #89 and #91).

What to do?

@cmsquared
Copy link
Member

cmsquared commented Nov 12, 2019

Is this one of the repos whose CI has been moved to gitlab?

@cferenba
Copy link
Member Author

Is this one of the repo's whose CI has been moved to gitlab?

I looked on gitlab and didn't see it. But even if it's there, there's no reason why this CI shouldn't pass.

@cmsquared
Copy link
Member

Ok. I didn't know if something had changed to accommodate gitlab that was incompatible with travis.

@cferenba
Copy link
Member Author

Good suggestion, but I don't think that's the issue.

@ktsai7
Copy link
Contributor

ktsai7 commented Nov 25, 2019

@cferenba retry?

@cferenba
Copy link
Member Author

@ktsai7 : looks like the build reran and passed, thanks!

@ktsai7 @ipdemes @tuxfan : would one of you please review and merge?

@ipdemes
Copy link

ipdemes commented Nov 25, 2019

@cferenba :did you have to do this changes because of some kind of issues with the FindHDF5 from cmake?

@cferenba
Copy link
Member Author

@ipdemes : yes, the FindHDF5 from cmake wasn't behaving as expected.

@ipdemes
Copy link

ipdemes commented Nov 25, 2019

@cferenba I see, I have had some similar experience myself. Looks good to me.

@ktsai7 ktsai7 requested a review from ipdemes November 25, 2019 20:42
@ipdemes ipdemes merged commit 6e0f5af into master Nov 25, 2019
@ktsai7 ktsai7 deleted the fix/legion-with-hdf5 branch November 26, 2019 17:27
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.

4 participants