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
chromium: Switch from PipeWire 0.2 to the current 0.3.x releases #114979
Conversation
This should make it work more reliable and support additional features like window sharing. See [0] for more details. [0]: https://jgrulich.cz/2020/12/18/webrtc-chromium-updates-in-2020/
88d987c
to
36e5fe0
Compare
I could successfully use screen sharing but I also realized that this should break |
@@ -161,7 +161,7 @@ in stdenv.mkDerivation { | |||
|
|||
buildCommand = let | |||
browserBinary = "${chromiumWV}/libexec/chromium/chromium"; | |||
libPath = lib.makeLibraryPath [ libva pipewire_0_2 ]; |
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.
If we do not have any other use for pipewire_0_2 it should 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.
From a quick git-grep it looks like Mutter still requires it:
pkgs/desktops/gnome-3/core/mutter/3.34/default.nix:, pipewire_0_2
pkgs/desktops/gnome-3/core/mutter/3.34/default.nix: pipewire_0_2 # TODO: backport pipewire 0.3 support
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.
cc @jtojnar is this still the case?
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.
Yup, the old version of Mutter used by Pantheon uses 0.2. The current version used by GNOME uses 0.3, obviously.
With #115781 merged this is now ready :) |
This should hopefully make it work more reliable and support additional
features like window sharing. See 0 for more details.
Motivation for this change
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)