Media Hub

Merge lp://staging/~phablet-team/media-hub/gst-1.5 into lp://staging/media-hub

  1. gst-1.5
  2. Merge into trunk
Proposed by Jim Hodapp
Status: Merged
Approved by: Jim Hodapp
Approved revision: 147
Merged at revision: 149
Proposed branch: lp://staging/~phablet-team/media-hub/gst-1.5
Merge into: lp://staging/media-hub
Prerequisite: lp://staging/~phablet-team/media-hub/fix-black-video-bugs
Diff against target: 62 lines (+19/-5)
2 files modified
debian/changelog (+7/-0)
src/core/media/gstreamer/playbin.cpp (+12/-5)
To merge this branch: bzr merge lp://staging/~phablet-team/media-hub/gst-1.5
Reviewer Review Type Date Requested Status
Jim Hodapp (community) code Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+265413@code.staging.launchpad.net

This proposal supersedes a proposal from 2015-07-21.

Commit message

Set gstreamer context directly instead of using gobject to communicate with the video sink.

Description of the change

Set gstreamer context directly instead of using gobject to communicate with the video sink.

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

PASSED: Continuous integration, rev:147
http://jenkins.qa.ubuntu.com/job/media-hub-ci/364/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/media-hub-vivid-amd64-ci/204
    SUCCESS: http://jenkins.qa.ubuntu.com/job/media-hub-vivid-armhf-ci/204
        deb: http://jenkins.qa.ubuntu.com/job/media-hub-vivid-armhf-ci/204/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/media-hub-vivid-i386-ci/204

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/media-hub-ci/364/rebuild

review: Approve (continuous-integration)
Revision history for this message
Jim Hodapp (jhodapp) wrote :

Just one thing to fix.

review: Needs Fixing (code)
Revision history for this message
Alfonso Sanchez-Beato (alfonsosanchezbeato) wrote :

See comment below.

Revision history for this message
Jim Hodapp (jhodapp) :
review: Approve (code)

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

to all changes: