Music Lens

Merge lp://qastaging/~3v1n0/unity-lens-music/preview-icon-fix into lp://qastaging/unity-lens-music

  1. preview-icon-fix
  2. Merge into trunk
Proposed by Marco Trevisan (Treviño)
Status: Merged
Approved by: Brandon Schaefer
Approved revision: 150
Merged at revision: 152
Proposed branch: lp://qastaging/~3v1n0/unity-lens-music/preview-icon-fix
Merge into: lp://qastaging/unity-lens-music
Diff against target: 54 lines (+4/-5)
3 files modified
m4/gcov.m4 (+1/-1)
src/musicstore-scope.vala (+2/-3)
src/rhythmbox-collection.vala (+1/-1)
To merge this branch: bzr merge lp://qastaging/~3v1n0/unity-lens-music/preview-icon-fix
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Brandon Schaefer (community) Approve
Review via email: mp+225906@code.qastaging.launchpad.net

Commit message

Rhythmbox: Use proper unity icons paths for preview image

Description of the change

Unity lens music points to the old icons location, thus we don't get the proper results in unity previews.

To post a comment you must log in.
Revision history for this message
Brandon Schaefer (brandontschaefer) wrote :

LGTM

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:150
http://jenkins.qa.ubuntu.com/job/unity-lens-music-ci/19/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-lens-music-utopic-amd64-ci/1/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-lens-music-utopic-armhf-ci/1/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-lens-music-utopic-i386-ci/1/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity-lens-music-ci/19/rebuild

review: Needs Fixing (continuous-integration)
lp://qastaging/~3v1n0/unity-lens-music/preview-icon-fix updated
151. By Marco Trevisan (Treviño)

Add support for lcov 1.11

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:151
http://jenkins.qa.ubuntu.com/job/unity-lens-music-ci/20/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-lens-music-utopic-amd64-ci/2/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-lens-music-utopic-armhf-ci/2/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-lens-music-utopic-i386-ci/2/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity-lens-music-ci/20/rebuild

review: Needs Fixing (continuous-integration)
lp://qastaging/~3v1n0/unity-lens-music/preview-icon-fix updated
152. By Marco Trevisan (Treviño)

MusicStore: remove ambiguity with GLib.Notification

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:152
http://jenkins.qa.ubuntu.com/job/unity-lens-music-ci/21/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-lens-music-utopic-amd64-ci/3
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-lens-music-utopic-armhf-ci/3
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-lens-music-utopic-i386-ci/3

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity-lens-music-ci/21/rebuild

review: Approve (continuous-integration)

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