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
sc-controller: fix missing icons (librsvg) #98116
Conversation
Uhm, I didn't write that line (@orivej did) so I'm not sure on why it's needed. Removing it may have some unintended side effect... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have not investigated this, but your change looks OK since 0c2ede9#diff-036410e9211b4336186fc613f7200b12
Today I switched to 20.09: I noticed the problem but this commit is not fixing it. Also, It's more serious than just aesthetics: various menus are not working because the code loading the icons is throwing exceptions. I think the application is not wrapped correctly. |
@rnhmjoj No it's not working anymore. But at the time I did the patch it was working: I tested the exact same build with and without this line and it fixed the issue. Sorry to have submitted a wrong patch :(. At the same time I also wrapped sc-controller's AppImage (which was not trivial), and it's still working fine. Here is the package if it can help: |
Specifically I had to patch |
I run a bisect and it seems the bad commit was |
ping @matthewbauer |
Motivation for this change
Fix missing SVG icons apparently due to redundant declaration/loading of
librsvg
.Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)