Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix finding .git directory in CMake when z3 is a submodule of some other repository #4850

Merged
merged 2 commits into from
Dec 6, 2020

Conversation

SweetVishnya
Copy link
Contributor

Hi!

We use z3 as a submodule in our project. This commit fixes z3 build when its source is located in a submodule.

@aytey
Copy link
Contributor

aytey commented Dec 4, 2020

Are there edge-cases where ${GIT_WORKTREE_DIR}/commondir doesn't exist and Z3 is not a submodule?

Your code has changed the build system to default to a submodule (I guess without confirming it is a submodule) vs. the current behaviour to error-out.

@SweetVishnya
Copy link
Contributor Author

Added verification that z3 is actually a submodule, otherwise I error-out as current behavior.

@NikolajBjorner NikolajBjorner merged commit 0c93c7a into Z3Prover:master Dec 6, 2020
@SweetVishnya SweetVishnya deleted the fix-z3-submodule-build branch December 12, 2020 08:45
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants