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
gst-plugins-good: Add optional qt5Support #110608
Conversation
@@ -102,6 +104,13 @@ stdenv.mkDerivation rec { | |||
] ++ optionals gtkSupport [ | |||
# for gtksink | |||
gtk3 | |||
] ++ optionals qt5Support (with qt5; [ |
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 don't think you want to input qt5 directly but use libforQt5 in callPackages.
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.
Are there docs on that?
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'm not sure it's really needed... libsForQt5.callPackage
ensures all the dependencies are using the same Qt version, which is significant for a Qt application but this is a library and I don't think any other dependency depends on Qt.
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.
relevant documentation https://nixos.org/manual/nixpkgs/stable/#sec-language-qt
So, I've investigated a bit the I think |
I'd like to get this merged before the other nheko PR. Can you drop the qtrelease thing so we can go on? |
Whoops, I thought I already did that. Here you go. |
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.
So, enabling qt5Support
gives a 780M increase to the closure size, so it's definitely better to keep it disabled by default. I also tested that gst-plugins-good and nheko build.
Motivation for this change
nheko video calling (Drafted in PR #110595) needs qmlgl, so I put a switch into gst-plugins-good to enable it.
I put another switch in for
lreleaseSupport
, because I don't think it's necessary in most cases but I would be happy to see it when I needed it while working on something.While working on it, I saw that pulse was disabled twice in the same block, so I also fixed that in another commit.
Thanks @rnhmjoj for the help,
ping @matthewbauer
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review pr 110608"
./result/bin/
)nix path-info -S
before and after)