Bug #164170 is correct, but adds an unnecessary runtime dependency on libgeoclue when built with geoclue2. Previously, we were conditionally linked to libgeoclue only if using geoclue1, because the pkg-config name in the FindGeoclue2 module was wrong, pointing to the .pc file that provides no cflags or ldflags for the library, only a couple pkg-config variables for finding the D-Bus service. So the GEOCLUE_LIBRARIES CMake variable was previously (incorrectly) empty, and that worked because we don't actually use the library, only the D-Bus interface. Although Ubuntu has until recently been building WebKit with geoclue1, it actually has had support for geoclue2 for years, so we don't need to keep the obsolete geoclue1 stuff around anymore. Removing geoclue1 support means we can get rid of the cruft for linking to libgeoclue.
So I'm wrong, GEOCLUE2_LIBRARIES and GEOCLUE2_INCLUDE_DIRS are separate CMake variables from the Geoclue1 versions, so there's no unnecessary dependency and no particular reason to drive removing Geoclue1 support. We should do it eventually anyway, so I'll leave this bug open.
Created attachment 317740 [details] Patch
Comment on attachment 317740 [details] Patch Clearing flags on attachment: 317740 Committed r220515: <http://trac.webkit.org/changeset/220515>
All reviewed patches have been landed. Closing bug.