Bug 352156

Summary:	master build uses qtpositioning always but finds it optionally
Product:	[Applications] marble	Reporter:	Harald Sitter <sitter>
Component:	general	Assignee:	marble-bugs
Status:	RESOLVED DUPLICATE
Severity:	normal	CC:	nienhueser
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Other
OS:	Linux
URL:	https://launchpadlibrarian.net/216058530/buildlog_ubuntu-wily-amd64.marble_4%3A15.08.0%2Bgit20150901.1105%2B15.10-0_BUILDING.txt.gz
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description Harald Sitter 2015-09-02 10:57:52 UTC

In git master there is an optional find_package on qt5positioning

> CMakeLists.txt:   find_package(Qt5Positioning)

and an unprotected include in

> ./src/lib/marble/declarative/Placemark.cpp:#include <QGeoAddress>

ultimately resulting in a build failure if qt5positioning is not found

> ../../../../../src/lib/marble/declarative/Placemark.cpp:14:23: fatal error: QGeoAddress: No such file or directory




Reproducible: Always


Actual Results:  
should either mark qt5positioning REQUIRE in cmake or make the QGeoAddress use conditional to whether positioning was found

Comment 1 Dennis Nienhüser 2015-09-03 19:28:44 UTC

Rex Dieter was working on a patch to improve the optional switch between Qt4 and Qt5 in Marble's cmake files. Once that is in we can mark all Qt modules REQUIRED.

Comment 2 Dennis Nienhüser 2015-09-05 11:52:31 UTC


*** This bug has been marked as a duplicate of bug 352242 ***