Shouldn't sysconfdir be /etc? Otherwise your configuration is read only.

We currently aren't modifying/generatating this config file with a NixOS module.

Even if the NixOS module does not support configuration, users should still be able to use environment.etc.[…] option. Since patching it like fwupd is quite simple, I suggest we do that instead.

Even if the NixOS module does not support configuration, users should still be able to use environment.etc.[…] option. Since patching it like fwupd is quite simple, I suggest we do that instead.

Yes we can do that, but we'd also need to add

environment.etc."geoclue/geoclue.conf".source = "${pkgs.geoclue2}/etc/geoclue/geoclue.conf";

to the NixOS module.

Prior to the update to geoclue it was just reading the bundled config within its own prefix.

I actually wasn't going for an improvement like that, just restore how it was.

@jtojnar I integrated your change