pulseeffects: fix functionality for non-GNOME environments

[Ma27]

Motivation for this change

Until now pkgs.pulseeffects only worked on GNOME-like environments and didn't work in different desktop environments (in my case i3wm as window manager and a simple X session init script).

Icons now work and settings can be persisted.

Things done

• Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
• Built on platform(s)
  • NixOS
  • macOS
  • other Linux distributions
• Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
• Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
• Tested execution of all binary files (usually in ./result/bin/)
• Determined the impact on package closure size (by running nix path-info -S before and after)
• Assured whether relevant documentation is up to date
• Fits CONTRIBUTING.md.

---

[flokli]

@Ma27 do we need this, or is wrapGAppsHook good enough (due to 670744e)

[jtojnar]

DConf GSettings module requires a DConf service. In my experience, this is achieved by adding dconf to services.dbus.packages (which has been recently done in #46058), environment.systemPackages should not be necessary,.

[Ma27]

First of all thanks a lot for your fast feedback!

You're right I guess. I removed the systemPackages statement from the dconf module. The change to pulseeffects fixes the problem for me. With having programs.dconf.enable = true I now can use pulseeffects in my desktop environments (and actually change Pulse settings).

[jtojnar]

I am discussing the icons here: #43150 (comment)

[Ma27]

so let's discuss the issue there and merge or close this PR depending on the result?

[peti]

Is this PR still relevant?

[Ma27]

I guess not, if the addition can be that simple :)