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 clang-cl treating file as a /U arg. #670

Closed
wants to merge 1 commit into from

Conversation

roblabla
Copy link
Contributor

@roblabla roblabla commented Mar 31, 2022

On MacOS, paths will likely start with /Users, but clang-cl treats /U
as an argument, which leads to the source being mistaken for a flag. To
avoid this, we can pass -- before the source to ensure argument parsing
is disabled.

Fixes #513

On MacOS, paths will likely start with /Users, but clang-cl treats /U
as an argument, which leads to the source being mistaken for a flag. To
avoid this, we can pass -- before the source to ensure argument parsing
is disabled.
@roblabla
Copy link
Contributor Author

roblabla commented Mar 31, 2022

This was already previously proposed in #514

I can confirm this works as expected.

@m-ou-se
Copy link
Member

m-ou-se commented May 11, 2022

#514 has been merged.

@m-ou-se m-ou-se closed this May 11, 2022
# 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.

Use -- to separate flags and options from input filenames
2 participants