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 conduit and axom external project src directory. #39

Merged
merged 3 commits into from
Dec 14, 2020

Conversation

mdavis36
Copy link
Collaborator

@mdavis36 mdavis36 commented Dec 4, 2020

These changes should help with the conduit configure errors some people were seeing.

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 looks safe enough. So has anyone had a chance to test this with the older CMake yet? In particular the 3.10 that comes with Ubuntu was failing before.

@mdavis36
Copy link
Collaborator Author

mdavis36 commented Dec 4, 2020

I tried it with 3.12 as that is the closest on LC. The 3.10 documentation shows the SOURCE_SUBDIR command is available (so this should work as expected). I can give this a go later in a WSL2 build but I don't have that environment set up on any of my systems, so it may be a little.

@jmikeowen
Copy link
Collaborator

Yeah, I actually tried the old way with CMake 3.12 on LC and it worked too. I'll try to test a standard Ubuntu install in WSL2 as well and see if the default 3.10 works now. If so I'll amend the build instructions.

Copy link
Contributor

@MishaZakharchanka MishaZakharchanka left a comment

Choose a reason for hiding this comment

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

These changes work on my end.

Sorry for getting to this late, didn't get a notification for some reason.

@mdavis36 mdavis36 merged commit 0d44c0d into develop Dec 14, 2020
@mdavis36 mdavis36 deleted the bugfix/conduit-axom-build branch December 14, 2020 23: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.

3 participants