-
Notifications
You must be signed in to change notification settings - Fork 80
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
weird symlinking error on windows? #1120
Comments
contents of directory:
|
Symlinks are "fun" on Windows and are a privileged action on Windows, have you tried https://doc.rust-lang.org/stable/std/os/windows/fs/fn.symlink_dir.html#limitations |
worked! thank you |
The symlinks can cause other things to break on Windows as well, such as the compiler not being able to find include files inside the cxx-qt-lib/ folder. I would also prefer not to need to include a section about enabling Developer Mode in the README of every project I use cxx-qt for. @ahayzen-kdab, if I were to create a PR for cxx-qt-build to copy header files rather than symlinking their directories, would that be a welcome change? I also noticed a comment in its source code: "TODO: If it's neither unix nor windows, we should probably just deep-copy the dependency headers into our own include directory." It seems like symlinking is more trouble than it's worth. |
Are symlinks really that broken on Windows? :-( Or is it a separate bug that include files are not working, it might just need to be passing a resolved path or something. At the moment I'd prefer to at least keep symlinking on Linux/UNIX systems, maybe on Windows we can either deep copy or detect when symlinks aren't going to work and fallback. We should also note that we are about to refactor the build system code during the 0.8 cycle, so something to consider @LeonMatthesKDAB |
@jnbooth Of course not having to enable developer mode would be good. |
hi! im getting an error while building my project on windows, but it builds fine on linux.
the directory exists, and the error isnt that detailed, so i dont really know what to do.
here's my build.rs:
thank you!
The text was updated successfully, but these errors were encountered: