mediaplayer-app

Merge lp://qastaging/~bfiller/mediaplayer-app/fix-138769 into lp://qastaging/mediaplayer-app

  1. fix-138769
  2. Merge into trunk
Proposed by Bill Filler
Status: Merged
Approved by: Bill Filler
Approved revision: 361
Merged at revision: 362
Proposed branch: lp://qastaging/~bfiller/mediaplayer-app/fix-138769
Merge into: lp://qastaging/mediaplayer-app
Diff against target: 8 lines (+1/-0)
1 file modified
data/mediaplayer-app.desktop.in.in (+1/-0)
To merge this branch: bzr merge lp://qastaging/~bfiller/mediaplayer-app/fix-138769
Reviewer Review Type Date Requested Status
Ted Gould (community) Needs Information
PS Jenkins bot continuous-integration Approve
Review via email: mp+263596@code.qastaging.launchpad.net

Commit message

use OnlyShowIn=Old to hide from Apps scope

Description of the change

use OnlyShowIn=Old to hide from Apps scope

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:361
http://jenkins.qa.ubuntu.com/job/mediaplayer-app-ci/270/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/3353/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mediaplayer-app-vivid-amd64-ci/28
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mediaplayer-app-vivid-armhf-ci/28
        deb: http://jenkins.qa.ubuntu.com/job/mediaplayer-app-vivid-armhf-ci/28/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mediaplayer-app-vivid-i386-ci/28
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-vivid-mako/2800/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/3351
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/3351/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/21617

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/mediaplayer-app-ci/270/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:361
http://jenkins.qa.ubuntu.com/job/mediaplayer-app-ci/271/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/3355
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mediaplayer-app-vivid-amd64-ci/29
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mediaplayer-app-vivid-armhf-ci/29
        deb: http://jenkins.qa.ubuntu.com/job/mediaplayer-app-vivid-armhf-ci/29/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mediaplayer-app-vivid-i386-ci/29
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-vivid-mako/2801
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/3353
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/3353/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/21620

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/mediaplayer-app-ci/271/rebuild

review: Approve (continuous-integration)
Revision history for this message
Ted Gould (ted) wrote :

Can we use the URL overlay branch instead? It solves the problem and doesn't add all the additional issues of adding the media player to launcher and other places that lead to user confusion. I think that's a much stronger solution and it is just waiting for review and a media-hub fix.

review: Needs Information
Revision history for this message
Bill Filler (bfiller) wrote :

After discussion with Pat, the consensus was to go with this solution for ota5 as it's been requested by the customer for a while. While it's not perfect, it's better than what we have today and if the user pins the mediaplayer to the launcher then they will get the same error message they get today.

We should get the url-dispatcher overlay and required media-hub work into a silo and tested and can switch to this solution when it's ready.

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