[mer][qtlocation] Fix again run-time warning when using Geoclue positioning plugin. Fixes JB#48424

See merge request !17

Previous fix was broken because of a mistake when backporting commits from upstream.

Fix location satelliteinfo

See merge request mer-core/qtlocation!14

[mer][qtlocation] Fix QGeoSatelliteInfo connect/disconnect undefined slots from geoclue side. Contributes JB#41029

Quites from docs QGeoSatelliteInfoSource::updateInterval:
"If the update interval is not set (or is set to 0) the source will provide
updates as often as necessary."

"If the requested interval is less than the minimumUpdateInterval(),
the minimum interval is used instead."

Ensure geoclue gets installed. JB#41029

See merge request !13

Earlier dropping of run-time and build-time geoclue dependencies from
qt5-plugin-position-geoclue has caused geoclue to no longer get installed
on the rootfs.

Fixing this by adding explicit rpm requirement.
Signed-off-by: Simonas Leleiva <simonas.leleiva@jollamobile.com>

[packaging] Cleanup libqtposition_geoclue.so. Fixes JB#40423

See merge request !12

Get rid of build- and run-time dependencies on libgeoclue

See merge request !11

Re-implement Geoclue plugin using only Qt DBus dropping both the
build-time and run-time dependency on libgeoclue. Allowing the Geoclue
plugin to be build on all platforms that support Qt DBus.

The priority of the Geoclue plugin has been lowered slightly so that
the native position plugin, if available, has precedence.

[ChangeLog][QtPositioning][Position] The Geoclue plugin has been
re-implemented using Qt DBus.

Task-number: QTBUG-40702
Change-Id: Ia06d089bfb46c10769ccffd765c044c361a9b484
Reviewed-by: Alex Blasche <alexander.blasche@theqtcompany.com>

Upstream commit SHA1: a3cb74a7

Move call to g_type_init() to the constructor and only conditionally
call it. g_type_init() has been deprecated since glib 2.35.

Change-Id: I425dd7883eef404229e924003fe86924b5f8bb09
Reviewed-by: Alex Blasche <alexander.blasche@digia.com>

Upstream commit SHA1: 22affd32

[ChangeLog][QtPositioning][QGeoPositionInfoSource] Report errors when
failing to connect to Geoclue.

Task-number: QTBUG-40425
Change-Id: If3fa0929b724aa70a1fed01b5951f5bc9bd5adad
Reviewed-by: Alex Blasche <alexander.blasche@digia.com>

Upstream commit SHA1: 7d9173dd

Revert "[QtLocation] Use Qt event loop to emit signals from GLib callbacks. Contributes to JB#34945"

This reverts commit 55009ecc.

Revert "[qtlocation] Start Geoclue client position interface asynchronously. Contributes to JB#38358"

This reverts commit 3020dff9.

[qtlocation] Start Geoclue client position interface asynchronously. Contributes to JB#38358

See merge request !10

Previously, the geoclue client position interface was started via
an asynchronous method call.  This one could stall for multiple
seconds if the dbus call required the geoclue service to be started.

This commit replaces it with an asynchronous call which doesn't stall
the GUI thread.

Contributes to JB#38358

[QtLocation] Register needed metatypes. Contributes to JB#36989

The invokable methods require QList<int> Q_ARG params, so that type
needs to be registered as a metatype.

Contributes to JB#36989

See merge request !7

The invokable methods require QList<int> Q_ARG params, so that type
needs to be registered as a metatype.

Contributes to JB#36989

[QtLocation] Make invoked methods invokable. Contributes to JB#36989



See merge request !6

A previous commit 55009ecc added an invalid metaobject invocation
of methods which were not a slot or Q_INVOKABLE.
This commit makes those methods private slots.

Contributes to JB#36989

[QtLocation] Rename Nokia geoservices plugin to HERE and port to new APIs. Contributes to JB#36270



See merge request !5

This commit renames the old "nokia" plugin to "here" and ports it
from the Nokia GeoCoding API to the new HERE GeoCoding API.
It also updates the documentation, examples and tests to refer
to the HERE plugin instead of the Nokia one, and updates the
usage terms according to HERE's terms and conditions.

Contributes to JB#36270

Change-Id: I0ff4814d1615226140144a6185abc793f35ebab0
Reviewed-by: Simon Hausmann <simon.hausmann@digia.com>

This fixes the current build break in dev too.

Change-Id: Id35f4411c228e4365042ab370c7c505f611cb694
Reviewed-by: Simon Hausmann <simon.hausmann@digia.com>
Reviewed-by: Michal Klocek <michal.klocek@digia.com>

The value type wrapper code was completely removed. Instead
QGeoShape & derived classes and QGeoCoordinate use Q_GADGET
to advertise their meta data which in turn allows QtQML
to dynamically discover the types when needed.

This implies that the above C++ classes become larger as the meta
data is permanently added. On the positive side all QML value type
wrapper were removed which reduces the QML plugin size. At the
same time it becomes significantly easier to use those types in
other QML plugins such as the QtLocation one.

Unfortunately this change comes with an unavoidable behavior change
as well. Previously is was possible to set a width and height
on an invalid QML rectangle. In the case of an invalid rect it was
made valid (setting center to 0,0) before setting the height or
width. The QGeoRectangle class explicitly aborts calls to
setHeight() and setWidth() in case of an invalid rect. From
now on the C++ class behavior is used in QML too. This was
the reason for the change to tst_map_routing.qml in this patch.
Incidentally the documentation always referred to the QGeoRectangle
documentation and thus always stated the now new behavior.

Documentation for the value types and the relevant behavior changes
will be re-added in a later commit.

Each Gadget must be registered
using QMetaType::registerComparators<T>() to enable == and !=
operators in QML. Unfortunately QMetaType requires the operator<
which does not make any sense for the value types touched by
this patch. Right now, a warning is printed. Attempts will
be made to remove the operator< requirement in QMetaType.
Otherwise the warning has to be removed before the release
as they are printed very often.

[ChangeLog][QtPositioning][Important Behavior Changes] The QML
rectangle type ignores setting of the height and width if the
rectangle is invalid. Previously the property setter changed
the rectangle center to QGeoCoordinate(0,0) before setting the
height and width of a previously invalid rectangle.

Change-Id: Iab98ede58066ce03b2b4e0ce03ed85ba7ecb1020
Reviewed-by: Simon Hausmann <simon.hausmann@digia.com>

Conflicts:
	src/imports/location/qdeclarativegeoroute.cpp
	src/imports/location/qdeclarativegeoroutemodel.cpp
	src/imports/location/qdeclarativegeoroutesegment.cpp
	src/imports/location/qdeclarativepolygonmapitem.cpp
	src/imports/location/qdeclarativepolylinemapitem.cpp
	src/imports/positioning/locationvaluetypeprovider.cpp
	src/imports/positioning/locationvaluetypeprovider.h
	src/imports/positioning/positioning.cpp
	src/imports/positioning/qdeclarativecoordinate.cpp
	src/imports/positioning/qdeclarativecoordinate_p.h
	src/imports/positioning/qdeclarativegeocircle.cpp
	src/imports/positioning/qdeclarativegeocircle.h
	src/imports/positioning/qdeclarativegeorectangle.cpp
	src/imports/positioning/qdeclarativegeorectangle.h
	src/imports/positioning/qdeclarativegeoshape.cpp
	src/imports/positioning/qdeclarativegeoshape.h
	src/positioning/qgeocircle.cpp
	src/positioning/qgeocircle.h
	src/positioning/qgeocoordinate.h
	src/positioning/qgeorectangle.cpp
	src/positioning/qgeorectangle.h
	src/positioning/qgeoshape.cpp
	src/positioning/qgeoshape.h
	tests/auto/declarative_core/tst_map_routing.qml
	tests/auto/qgeocoordinate/tst_qgeocoordinate.cpp

[QtLocation] Use Qt event loop to emit signals from GLib callbacks. Contributes to JB#34945

This commit ensures that we use Qt::QueuedConnection to emit signals
(or invoke functions which emit signals) within codepaths which are
entered from the glib event loop, to avoid QTBUG-18434.

Contributes to JB#34945

See merge request !4

This commit ensures that we use Qt::QueuedConnection to emit signals
(or invoke functions which emit signals) within codepaths which are
entered from the glib event loop, to avoid QTBUG-18434.

Contributes to JB#34945

[mer] Ensure timestamps use C locale. Contributes to MER#1536

Timestamps must be generated using the C locale, so they do not contain
 digits other than Latin.

See merge request !3

 Timestamps must be generated using the C locale, so they do not contain
 digits other than Latin.

[mer] Always update route node when a new scene graph node is created. Fixes MER#1149

Change-Id: Ib89646cb804f925fedc668f845ac33cf916a7721