Merge lp://staging/~timo-jyrinki/qtgrilo/fix_lp1287180 into lp://staging/qtgrilo

Proposed by Timo Jyrinki
Status: Merged
Approved by: Timo Jyrinki
Approved revision: 15
Merged at revision: 15
Proposed branch: lp://staging/~timo-jyrinki/qtgrilo/fix_lp1287180
Merge into: lp://staging/qtgrilo
Diff against target: 89 lines (+27/-1)
6 files modified
debian/changelog (+19/-0)
debian/control (+0/-1)
declarative/grilodatasource.h (+2/-0)
declarative/grilomedia.h (+2/-0)
declarative/griloplugin.h (+2/-0)
declarative/griloregistry.h (+2/-0)
To merge this branch: bzr merge lp://staging/~timo-jyrinki/qtgrilo/fix_lp1287180
Reviewer Review Type Date Requested Status
Victor Thompson Approve
Review via email: mp+209222@code.staging.launchpad.net

Commit message

Prevent 'signals' from GDBus from clashing with Qt's 'signals' (LP: #1287180)

Description of the change

Obviously a hack, and I'm not familiar with qtgrilo.

To post a comment you must log in.
13. By Timo Jyrinki

Add comments to the ugly hacks.

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

A hack is very acceptable in this case--since we'll be moving away from qtgrilo shortly. Thanks Timo!

review: Approve
14. By Timo Jyrinki

Sync debian/ with what is in the archives already.

15. By Timo Jyrinki

Add changelog entry for 0.0.20130610-0ubuntu3

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

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches