-
Notifications
You must be signed in to change notification settings - Fork 77
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
qml_minimal not compiling on Windows #625
Comments
Thanks for taking to time to report this issue and help make CXX-Qt better. Are you able to provide the contents of your rust file (i assume I wonder if what has happened is have you taken the Also if you are able to provide any debug information for #450 that would be much appreciated as there are many different possible combinations of setups 😅 |
Thank you for your help, it's much appreciated. I'll gladly help for #450 :D Your first assumption was good : I was at the latest commit of the main branch. So that's exactly what happened. I made the assumption that the main branch was stable, forgot that you were still in 0.x.x, meaning unstable . My bad, sorry. Now, based on the qml_minimal on tag 0.5.3, the error was passed, but another appeared.
|
That's a fun error :-) sounds like we might need to set However I would also note that before you were using Could you try using MSVC2019 again ? |
No problem, FYI this new API we are writing for 0.6 we are hoping to use as the base to start stabilising for 1.0 :-) (as seen in #555 ) |
Sorry, I'll be able to test it this evening |
I started again from scratch:
Interesting:
Compile args gererated by Qt Creator: compile_args.txt Is it worth to investagate on 0.5.3 ? Do you want me to test it on the latest from main branch ? |
This is on Windows right ? This is likely due to the corrosion bug in combination with Rust 1.71 see Try bumping your corrosion version to 0.4.2 as in #609 |
Yes, Windows 10 (22H2). That was exactly the solution for this bug. Now it runs fine. Thank you ! For fun, I was testing out from the latest main 0af59b , with Cargo.toml deps pointing to the path of the corresponding crates in the local cxx-qt repo. Qt 6.5.2 official installer with Qt Creator 11:
|
MinGW debug looks like it needs bigobj enabling, MSCV debug looks like a more interesting failure to investigate. |
I'll find out what "bigobj" means tonight :D . I'm at your disposition for the other failure. Maybe we can close this ticket as resolved and open a new one dedicated to this MSCV failure ? |
We might need to set the equivalent of Note we already have some settings in cxx-qt-build and cxx-qt-lib, so these might just need adding there to make things happier. // MSVC
builder.flag_if_supported("/std:c++17");
builder.flag_if_supported("/Zc:__cplusplus");
builder.flag_if_supported("/permissive-");
// GCC + Clang
builder.flag_if_supported("-std=c++17"); https://learn.microsoft.com/en-us/cpp/build/reference/bigobj-increase-number-of-sections-in-dot-obj-file?view=msvc-170 You could open a new issue or just reuse this one for compile issues on Windows. |
We will continue with this issue since the title is still valid. With your agreement, I'll implement & test the flags for MinGW this weekend. |
Cool thanks! |
Hello, I tested |
Note you would need to adjust the builder in both cxx-qt-lib/build.rs and cxx-qt-build/src/lib.rs. But as you say maybe this is not the issue :-) |
Done, tested successfully. PR #644 created |
Looks like this issue (for Windows) was solved between the corrosion bump and adding bigobj args for MinGW. Please open a new issue if there are still issues! |
Hello,
I'm trying out this nice crate. Already encountered issue #450 on MacOS (still investigating) and now trying it on Windows 10. I followed step by step your book, then I tried qml_minimal directly. Nothing compile.
Any idea ?
Regards,
Cyril
Steps to reproduce:
Expected behavior:
Current behavior:
The text was updated successfully, but these errors were encountered: