Skip to content

Make geoclue an optional dependency

Steven requested to merge stebalien/xdg-desktop-portal:steb/geoclue-optdep into main

It's needed at build-time but only needed at runtime (over dbus) if/when using the geolocation portal. None of the binaries in this package actually link to the geoclue libraries (checked with both namcap and pacman -Qql xdg-desktop-portal | grep '/usr/lib/xdg-' | xargs ldd | grep geo).

From what I can tell, Debian doesn't have a direct dependency either (https://packages.debian.org/experimental/xdg-desktop-portal) although they definitely build against it (https://salsa.debian.org/debian/xdg-desktop-portal/-/blob/9f3741310502dab6e4920ab6f3c0cb405b3a2520/debian/control#L15).

I've also added glib2-devel to the build dependencies as it appeared to be missing (and isn't a part of base and/or base-devel).

Merge request reports