Code review comment for lp:~seb128/unity-control-center/use-new-gstreamer

Revision history for this message
Iain Lane (laney) wrote :

On Wed, Feb 10, 2016 at 04:34:56PM -0000, Sebastien Bacher wrote:
> You have been requested to review the proposed merge of lp:~seb128/unity-control-center/use-new-gstreamer into lp:unity-control-center.
>
> For more details, see:
> https://code.launchpad.net/~seb128/unity-control-center/use-new-gstreamer/+merge/285455
>
> Update the build-depends to the current gstreamer version

Hum, I guess you thought I might say this, but...

I grepped the source code and it looks like gstreamer isn't actually
used directly, but via cheese - is it necessary for u-c-c to have this
in BD, configure.ac and #include in um-user-accounts.c or should
libcheese give us what we need?

If it is, then this branch is ok I think.

--
Iain Lane [ <email address hidden> ]
Debian Developer [ <email address hidden> ]
Ubuntu Developer [ <email address hidden> ]

« Back to merge proposal