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

[WIN] Preprocessing fails on paths with unicode characters #2

Closed
facchinm opened this issue Oct 12, 2017 · 3 comments
Closed

[WIN] Preprocessing fails on paths with unicode characters #2

facchinm opened this issue Oct 12, 2017 · 3 comments
Assignees

Comments

@facchinm
Copy link
Member

As reported by @gril002 on arduino/Arduino#6816 , the preprocessor fails if launched from a path containing unicode characters.
The error message is

LLVM ERROR: Cannot chdir into "C:\Users\Gašper Gril

This call doesn't fails

"C:\Users\IEUser\Desktop\arduino-PR-beta1.9-BUILD-41\tools-builder\arduino-preprocessor\0.1.3/arduino-preprocessor" "C:\Users\GAPERG~1\AppData\Local\Temp\arduino_build_861144\preproc\ctags_target_for_gcc_minus_e.cpp" "" -- -std=gnu++11

This does

"C:\Users\Gašper Gril\Desktop\arduino-PR-beta1.9-BUILD-41\tools-builder\arduino-preprocessor\0.1.3/arduino-preprocessor" "C:\Users\GAPERG~1\AppData\Local\Temp\arduino_build_779924\preproc\ctags_target_for_gcc_minus_e.cpp" "" -- -std=gnu++11

The builder seems to mangle the path in strange ways but gcc is unaffected so I believe the bug is not there.

Unable to reproduce on Linux, testing on Mac later today

@facchinm
Copy link
Member Author

facchinm commented Oct 12, 2017

@facchinm
Copy link
Member Author

Does only affect Windows, Linux and Mac are unaffected as far as I have tested. The patch could be quite complicated, requiring to recompile the whole clang project

@facchinm facchinm self-assigned this Oct 12, 2017
facchinm added a commit to facchinm/arduino-builder that referenced this issue Oct 13, 2017
facchinm added a commit to facchinm/arduino-builder that referenced this issue Jan 10, 2018
cmaglie pushed a commit to cmaglie/arduino-builder that referenced this issue Jan 10, 2018
facchinm added a commit to facchinm/arduino-builder that referenced this issue Jan 10, 2018
cmaglie pushed a commit to cmaglie/arduino-builder that referenced this issue Jan 10, 2018
cmaglie pushed a commit to cmaglie/arduino-builder that referenced this issue Jan 11, 2018
@facchinm
Copy link
Member Author

facchinm commented Oct 3, 2018

Closing as "fixed" by arduino/arduino-builder@7de76a9

@facchinm facchinm closed this as completed Oct 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant