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
multimc: fix build with Qt 5.11, fix desktop icon #42432
Conversation
Fixes NixOS#42387. See also NixOS#42351 for information on Qt breakage.
@GrahamcOfBorg build multimc |
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: multimc Partial log (click to expand)
|
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: multimc Partial log (click to expand)
|
Failure on x86_64-linux (full log) Attempted: multimc Partial log (click to expand)
|
@GrahamcOfBorg build multimc |
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: multimc Partial log (click to expand)
|
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: multimc Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: multimc Partial log (click to expand)
|
Motivation for this change
MultiMC build has been broken since the upgrade to Qt 5.11 (see #42387). As mentioned in that issue, the desktop icon is also broken.
MultiMC had a patch upstream to fix build with Qt 5.11, but it hasn't hit a stable release yet, and applying the patch on top of 0.6.2 fails (see, again, discussion in #42387).
Things done
develop
branch, where the Qt 5.11 fix has been applied.MultiMC
→multimc
to fix the desktop icon.sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)