-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[bug] Path to compiler with spaces when building boost #15070
Comments
Hi @sogartar Yes, I think this belongs to boost, in this case, it might be workarounded in the Conan recipe in conan-center-index. So I am transferring this issue to the In any case, please note that in Windows, it is always problematic for many tools to have spaces in paths or with other special characters. I always avoid that installing things in different places than |
You can try to replace these lines in boost recipe: if is_msvc(self):
contents += f' "{cxx_fwd_slahes}"'
else:
contents += f' {cxx_fwd_slahes}' by if cxx_fwd_slahes:
contents += f' "{cxx_fwd_slahes}"' Moreover, pay attention that variables defined in |
Hi @sogartar, @memsharded and @SpaceIm , I'm running almost into the same issue, when I used the gcc compiler, it could detect it but I got this error: Then I modified the "user-config.jam" to use msvc like this:
when I run conan install, I get this error:
The issue is, it overwrites the user-config.jam and it doesn't detect the msvc that I added. How can I fix that please? |
This ticket was solved and closed, so if there is any new issue it would be necessary to create a new ticket with updated details:
Thanks! |
Environment details
Steps to reproduce
I am not sure if this error is in Conan or in Boost Jam.
My compiler directory is
C:/Program Files/LLVM/bin
. It has a space in its path.I have this profile
When I do
I get this error
The interesting line is
It seems that somehow the path got treated as two string.
In
C:\.conan\5d11a4\1\src\tools\build\user-config.jam
I have this:The compiler path is not quoted.
Logs
No response
The text was updated successfully, but these errors were encountered: