-
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
error: link modifiers combination +bundle,+whole-archive
is unstable when generating rlibs
#519
Comments
So looking at https://doc.rust-lang.org/rustc/command-line-arguments.html#-l-link-the-generated-crate-to-a-native-library Then we are normally building to This from initial investigation seems to have broken via a newer rustc ? |
Manually setting But we need to check that
@Be-ing ^^ any opinions ? |
Otherwise the build fails with link modifiers combination `+bundle,+whole-archive` is unstable when generating rlibs Closes KDAB#519
Otherwise the build fails with the following error with rustc 1.69 link modifiers combination `+bundle,+whole-archive` is unstable when generating rlibs Closes KDAB#519
Otherwise the build fails with the following error with rustc 1.69 link modifiers combination `+bundle,+whole-archive` is unstable when generating rlibs Closes KDAB#519
Otherwise the build fails with the following error with rustc 1.69 link modifiers combination `+bundle,+whole-archive` is unstable when generating rlibs Closes #519
Otherwise the build fails with the following error with rustc 1.69 link modifiers combination `+bundle,+whole-archive` is unstable when generating rlibs Closes KDAB#519
This is caused by a regression in rustc 1.69: rust-lang/rust#110912 Reopening this until the upstream issue is fixed |
+bundle,+whole-archive
is unstable when generating rlibs
Otherwise the build fails with the following error with rustc 1.69 link modifiers combination `+bundle,+whole-archive` is unstable when generating rlibs Closes #519
Upstream fix: rust-lang/rust#110917 |
Fix was merged upstream and will be in Rust 1.71 and maybe 1.70. |
@Be-ing do we want to leave this open as a reminder to remove the workaround ? And also do we want to remove the workaround at some point ? How long should we wait for people to move on from 1.69 to 1.70+ ? Or is it even possible to have a cfg flag for a specific rustc version ? |
The workaround is harmless AFAIK, so we might as well leave it forever. |
Cool, just wanted to check :-) |
We could remove the workaround if we raise the MSRV to >= 1.70 in the future (or 1.71 if the fix doesn't make it into 1.70). |
rust-lang/rust#110917 was approved for backporting to 1.70. |
tested with rust from the fedora repo and from rustup.
The text was updated successfully, but these errors were encountered: