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

make.bat: quote paths when cloning `tcc` #6209

Merged
merged 1 commit into from Aug 24, 2020
Merged

make.bat: quote paths when cloning `tcc` #6209

merged 1 commit into from Aug 24, 2020

Conversation

@spaceface777
Copy link
Member

@spaceface777 spaceface777 commented Aug 24, 2020

This avoids escaping issues if V is installed on a directory with spaces in its name, and should also fix #6207.

This avoids escaping issues if V is installed on a directory with spaces in it's name, and should also fix #6207.
@medvednikov medvednikov merged commit 60eedc2 into master Aug 24, 2020
32 checks passed
32 checks passed
ubuntu-tcc
Details
ubuntu-tcc
Details
alpine-musl
Details
alpine-musl
Details
macos (macOS-latest)
Details
macos (macOS-latest)
Details
ubuntu
Details
ubuntu
Details
ubuntu-autofree-selfcompile
Details
ubuntu-autofree-selfcompile
Details
ubuntu-musl
Details
ubuntu-musl
Details
ubuntu-llvm-mingw
Details
ubuntu-llvm-mingw
Details
windows-gcc
Details
windows-gcc
Details
windows-msvc
Details
windows-msvc
Details
windows-tcc
Details
windows-tcc
Details
docs-line-len-check
Details
docs-line-len-check
Details
compilable-v-c-and-v-win-c
Details
compilable-v-c-and-v-win-c
Details
ubuntu-c-plus-plus
Details
ubuntu-c-plus-plus
Details
install-modules
Details
install-modules
Details
gitly-compiles
Details
gitly-compiles
Details
Autobahn integrations tests
Details
Autobahn integrations tests
Details
@spaceface777 spaceface777 deleted the quoted-tcc-paths branch Aug 24, 2020
ahgm added a commit to ahgm/v that referenced this pull request Aug 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

2 participants
You can’t perform that action at this time.