<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body><table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>Bug ID</th>
<td><a class="bz_bug_link
bz_status_NEW "
title="NEW - [GTK] Remove support for geoclue1 and unnecessary dependency on libgeoclue"
href="https://bugs.webkit.org/show_bug.cgi?id=164205">164205</a>
</td>
</tr>
<tr>
<th>Summary</th>
<td>[GTK] Remove support for geoclue1 and unnecessary dependency on libgeoclue
</td>
</tr>
<tr>
<th>Classification</th>
<td>Unclassified
</td>
</tr>
<tr>
<th>Product</th>
<td>WebKit
</td>
</tr>
<tr>
<th>Version</th>
<td>Other
</td>
</tr>
<tr>
<th>Hardware</th>
<td>PC
</td>
</tr>
<tr>
<th>OS</th>
<td>Linux
</td>
</tr>
<tr>
<th>Status</th>
<td>NEW
</td>
</tr>
<tr>
<th>Severity</th>
<td>Normal
</td>
</tr>
<tr>
<th>Priority</th>
<td>P2
</td>
</tr>
<tr>
<th>Component</th>
<td>WebKit Gtk
</td>
</tr>
<tr>
<th>Assignee</th>
<td>webkit-unassigned@lists.webkit.org
</td>
</tr>
<tr>
<th>Reporter</th>
<td>mcatanzaro@igalia.com
</td>
</tr>
<tr>
<th>CC</th>
<td>bugs-noreply@webkitgtk.org
</td>
</tr>
<tr>
<th>Depends on</th>
<td>164170
</td>
</tr></table>
<p>
<div>
<pre><a class="bz_bug_link
bz_status_NEW "
title="NEW - [GTK] Build break by missing geoclue-2.0."
href="show_bug.cgi?id=164170">Bug #164170</a> 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.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>