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
calibre: fix build #66229
calibre: fix build #66229
Conversation
Builds and runs for me on NixOS unstable. |
1c51cf7
to
f349b79
Compare
Why are we wrapping in |
It's been wrapped there since initial packaging 10 years ago a4473b8, I'm assuming because https://github.com/kovidgoyal/calibre/tree/master/setup looks a bit extreme. |
I can fix this today 👍 |
I checked and it appears to be wrapped correctly, though I decided to make it a little cleaner #66483 |
Please do not use a custom builder function unless really necessary. I know it's documented as the way with Qt5, but we should avoid it when possible. |
Did you mean to comment that on my PR? (context confused) |
Motivation for this change
Makes calibre compile again, fixing #65313 No real testing done, but it starts again for me, and the viewer works.
Fix maybe technically uncool, splitting qtWrapperArgs, but i don't see how to get PYTHONPATH properly interpolated otherwise.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)cc @ToxicFrog