qtubuntu-sensors

qtubuntu-sensors fails to build against Qt 5.2 - part 2

Bug #1271415 reported by Timo Jyrinki
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
qtubuntu-sensors
Fix Released
Critical
Albert Astals Cid
qtubuntu-sensors (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

After bug #1271034 gets fixed, the next problem is:

error: cannot allocate an object of abstract type 'core::GeoPositionInfoSourceFactory'

https://launchpad.net/~canonical-qt5-edgers/+archive/qt5-beta2/+sourcepub/3840133/+listing-archive-extra (ppa:canonical-qt5-edgers/qt5-beta2)

A fix should be made so that the package builds both against Qt 5.0 and Qt 5.2.

Tags: qt5.2

Related branches

lp:~aacid/qtubuntu-sensors/newvirtual
PS Jenkins bot: Approve (continuous-integration)
Timo Jyrinki: Approve
Diff: 29 lines (+12/-0)
2 files modified
plugins/position/core_geo_position_info_source_factory.cpp (+9/-0)
plugins/position/core_geo_position_info_source_factory.h (+3/-0)
lp:ubuntu/trusty-proposed/qtubuntu-sensors
Revision history for this message
Albert Astals Cid (aacid) wrote :

https://code.launchpad.net/~aacid/qtubuntu-sensors/newvirtual/+merge/202811 should be a fix until we decide if we need to implement the area monitor or not (fwiw i checked upstream and the BB10 plugin does not)

Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote :

Confirming it fixes the build problem. I filed bug #1271886 for the next problem which is some failing tests still.

Changed in qtubuntu-sensors:
assignee: nobody → Albert Astals Cid (aacid)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:qtubuntu-sensors at revision 48, scheduled for release in qtubuntu-sensors, milestone Unknown

Changed in qtubuntu-sensors:
status: New → Fix Committed
Pat McGowan (pat-mcgowan)
Changed in qtubuntu-sensors (Ubuntu):
status: New → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package qtubuntu-sensors - 0.6+14.04.20140213-0ubuntu1

---------------
qtubuntu-sensors (0.6+14.04.20140213-0ubuntu1) trusty; urgency=low

  [ Martin Pitt ]
  * Add integration tests for the QtSensor API → qtubuntu-sensors →
    platform-api (with simulated sensor backend).
  * Fix crash if there are no available sensors. (LP: #1272294)

  [ Albert Astals ]
  * Make it compile with Qt 5.2 (LP: #1271415). (LP: #1271415)
  * Q_ASSERT -> EXPECT_EQ Since sometimes Q_ASSERT can be compiled out
    and thus the test would fail. (LP: #1271886)
 -- Ubuntu daily release <email address hidden> Thu, 13 Feb 2014 01:44:53 +0000

Changed in qtubuntu-sensors (Ubuntu):
status: Fix Committed → Fix Released
Albert Astals Cid (aacid)
Changed in qtubuntu-sensors:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.