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

build.sh: replace git protocol with https when cloning repos #237

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

MichalHe
Copy link
Contributor

@MichalHe MichalHe commented Dec 16, 2022

When trying to build symbiotic using the build.sh script on a system without z3 installed, the build fails on timeout while trying to clone Z3Prover/z3 repository. According to this GitHub post, the support for git:// protocol has been dropped, explaining the timeouts when cloning z3 since the current repository URL uses the git:// protocol.

This patch replaces all GitHub URLs that relied on the git:// protocol, with ones that use https://. This patch fixes a small typo in README.md. If it is undesired to have this change within one commit, there is no problem in splitting it into multiple commits.

Copy link
Contributor

@lzaoral lzaoral left a comment

Choose a reason for hiding this comment

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

Thanks @MichalHe! Let's wait for #235 to be merged first.

@MichalHe
Copy link
Contributor Author

Sure, no problem.

@lzaoral lzaoral added the bug label Dec 19, 2022
@lzaoral
Copy link
Contributor

lzaoral commented Dec 23, 2022

@MichalHe, sorry for a late reply, can you please rebase this PR as well?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants