-
-
Notifications
You must be signed in to change notification settings - Fork 14.4k
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
pipewire: 0.3.63 -> 0.3.63 #207895
pipewire: 0.3.63 -> 0.3.63 #207895
Conversation
@bjornfor interesting. I'm guessing that's what the update script does when you run it. |
Right, looks like the person that last updated pipewire did not use the update script so the module changes were omitted. cc @K900 |
Huh, I was sure I committed those. Anyway, I've been looking at how we can avoid doing the thing we're currently doing with the config files, because it doesn't really work the way you expect anyway... |
Is there any good ways to disable/remove Currently I've ended up reverting this commit in my local checkout of nixpkgs. |
Debian added an option to disable it according to this tweet Imo we should do the same. Otherwise I might switch to windows. |
That is in no way Debian specific and you can already set that option in your Pipewire config file. |
yes I will for now use |
|
ok this is the right thing. Doesn't matter how you create it:
OrginalSo I've read https://gist.github.com/K900/20fb1651d72411e3419d278bb29343ea and https://t.co/AB7Jvf8Lne did:
without success. If someone figures this out let me know. Unfortunately I don't have time to fiddle with this now since I should have been working on different thing so for now the solution is never to turn on sound on the laptop :D |
My time to shine! This is how my pipewire config looks (take out the things you don't want), but this just removes the bell, as it was driving me insane as well... { config, lib, ... }:
let
defaultConfig = lib.importJSON (<nixpkgs/nixos/modules/services/desktops/pipewire/daemon/pipewire.conf.json>);
finalConfig = let
filteredModules = builtins.filter (v: v.name != "libpipewire-module-x11-bell") defaultConfig."context.modules";
in defaultConfig // { "context.modules" = filteredModules; };
in {
# disable sound as it might cause issues
sound.enable = lib.mkForce false;
# required to disable pulseaudio
hardware.pulseaudio.enable = lib.mkForce false;
# rtkit is optional but recommended
security.rtkit.enable = true;
services.pipewire = {
enable = true;
alsa.enable = true;
alsa.support32Bit = true;
pulse.enable = true;
config.pipewire = finalConfig;
};
} |
The drop-in needs to be a JSON file, and the value needs to be set in |
yeah I already updated with solution hour ago #207895 (comment) that should be hopefully fool proofed. |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/mysterious-unstoppable-alert-sounds/26801/3 |
I guess this is part of pulse audio? The docs aren't really clear to me for pipewire and modules and contexts et al https://docs.pipewire.org/page_pulse_module_x11_bell.html - would be nice to have an easy way to set an alert sound when someone isn't using a desktop env like myself but, hey, turning it off works well enough. |
@eyJhb how can I use the spaceship jazz |
Maybe this (untested). { config, lib, modulesPath, ... }:
let
defaultConfig = lib.importJSON (modulesPath + "/services/desktops/pipewire/daemon/pipewire.conf.json");
.... |
This is no longer supported at all. What is your end goal here? |
Oh true, I completely forgot. I don't even use that anymore. This is what I currently have in my configuration environment.etc."pipewire/pipewire.conf.d/99-no-bell.conf".text = ''{"context.properties": {"module.x11.bell": false}}''; |
ah, I see. I probably can get this done with |
I'm still not sure what you're actually trying to do. Are you trying to enable the bell? Because it's on by default. |
disable it for now (which I've done) but in future would like to set it to some custom sound |
Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for pipewire is: "Server and user space API to deal with multimedia pipelines"
meta.homepage for pipewire is: "https://pipewire.org/"
meta.changelog for pipewire is: ""
Updates performed
To inspect upstream changes
Impact
Checks done (click to expand)
passthru.tests
did not passRebuild report (if merged into master) (click to expand)
Instructions to test this update (click to expand)
Either download from Cachix:
(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the
trusted-users
list or you can usesudo
since root is effectively trusted.Or, build yourself:
After you've downloaded or built it, look at the files and if there are any, run the binaries:
Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of
nixpkgs-review
run on x86_64-linux 11 package built:
Maintainer pings
cc @jtojnar @Kranzes for testing.