Skip to content

Conversation

@jsjtxietian
Copy link

The current doc about building with clang and ninja on windows contains a minor formatting issue, this pr improves this part's readability.

See the current version:
image

Should be more clear now:
image

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Nov 2, 2025
@ehoogeveen-medweb
Copy link

To ensure that paths containing whitespace are correctly preserved with set, you probably want to wrap the whole thing in double quotes:

set "LIB=C:\Program Files (x86)\Windows Kits\10\Lib\10.0.22621.0\um\x64;C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.41.34120\lib\x64\uwp;C:\Program Files (x86)\Windows Kits\10\Lib\10.0.22621.0\ucrt\x64"

Though note that set is only for the current session. To set the environment variable permanently you'd need setx:

setx LIB "C:\Program Files (x86)\Windows Kits\10\Lib\10.0.22621.0\um\x64;C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.41.34120\lib\x64\uwp;C:\Program Files (x86)\Windows Kits\10\Lib\10.0.22621.0\ucrt\x64"

@jsjtxietian
Copy link
Author

To ensure that paths containing whitespace are correctly preserved with set, you probably want to wrap the whole thing in double quotes:

set "LIB=C:\Program Files (x86)\Windows Kits\10\Lib\10.0.22621.0\um\x64;C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.41.34120\lib\x64\uwp;C:\Program Files (x86)\Windows Kits\10\Lib\10.0.22621.0\ucrt\x64"

I agree although the version without double quotes works too in developer Command Prompt For VS. I only addressed format issue in this PR, did not touch the content. cc @Srihari-mcw

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants