@GrahamcOfBorg eval

Things that are also changed:

blueman: withPulseAudio ? config.pulseaudio or stdenv.isLinux instead of withPulseAudio = config.pulseaudio or true;
wxgtk: withMesa ? libGLSupported instead of withMesa = lib.elem stdenv.hostPlatform.system lib.platforms.mesaPlatforms;
mpv: stdenv.isLinux instead of !stdenv.isDarwin;

All of this is probably okay, but I wanted to mention it

libGLSupported is defined to be precisely the mesaPlatforms check.

(The two other things seem to be changes; good ones, in my opinion).

@dasJ

wxgtk

libGLSupported is defined as lib.elem stdenv.hostPlatform.system lib.platforms.mesaPlatforms in all-packages.nix :)

blueman, mpv

Yes, these are fixes for systems not built by hydra and ofborg. Outside Linux (and emulation of Linux by FreeBSD and Illumos) people don't normally run PulseAudio and only Linux (and emulations) has ALSA. But these are a noop for stuff hydra builds.

Thanks!

@oxij As I said I think the changes are probably okay, but I wanted to point that out to a potential merger

@dasJ yes, thanks for attentive checking — and I think it was only natural for @oxij to put the reasoning on record if the changes were mentioned.