<div dir="ltr">Yeah I'm happy with that!<br clear="all"><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><br style="color:rgb(34,34,34)"><span style="color:rgb(34,34,34)">Is mise le meas/Regards,</span><br style="color:rgb(34,34,34)"><br style="color:rgb(34,34,34)"><span style="color:rgb(34,34,34)">Eric Curtin</span><br><div><span style="color:rgb(34,34,34)"><br></span></div></div></div></div><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, 24 Jun 2022 at 13:47, Kieran Bingham <<a href="mailto:kieran.bingham@ideasonboard.com">kieran.bingham@ideasonboard.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Quoting Eric Curtin (2022-06-23 19:45:39)<br>
> I think this ones an easy review if someone gets the chance.<br>
> <br>
> Is mise le meas/Regards,<br>
> <br>
> Eric Curtin<br>
> <br>
> <br>
> On Wed, 22 Jun 2022 at 15:38, Eric Curtin <<a href="mailto:ecurtin@redhat.com" target="_blank">ecurtin@redhat.com</a>> wrote:<br>
> ><br>
> > KMS sink requires some DRM libraries, SDL sink requires some SDL<br>
> > libraries.<br>
> ><br>
> > Suggested-by: Pavel Machek <<a href="mailto:pavel@ucw.cz" target="_blank">pavel@ucw.cz</a>><br>
> > Signed-off-by: Eric Curtin <<a href="mailto:ecurtin@redhat.com" target="_blank">ecurtin@redhat.com</a>><br>
> > ---<br>
> > Changes in v2:<br>
> ><br>
> > - Updated commit message/<br>
> > ---<br>
> >  README.rst | 8 +++++++-<br>
> >  1 file changed, 7 insertions(+), 1 deletion(-)<br>
> ><br>
> > diff --git a/README.rst b/README.rst<br>
> > index 05ebcec4..28c79b3f 100644<br>
> > --- a/README.rst<br>
> > +++ b/README.rst<br>
> > @@ -87,7 +87,13 @@ for gstreamer: [optional]<br>
> >          libgstreamer1.0-dev libgstreamer-plugins-base1.0-dev<br>
> ><br>
> >  for cam: [optional]<br>
> > -        libevent-dev<br>
> > +        libevent-dev libdrm-dev libsdl2-dev libsdl2-image-dev<br>
<br>
Should we wrap optional dependencies in [ ] ?<br>
Cam is optional (hence the [optional]) but now the optional component<br>
has optional dependencies!?<br>
<br>
> > +<br>
> > +        If you would like to use KMS sink libdrm-dev is an optional dependancy,<br>
> > +        if you would like to use SDL sink with pixel formats such as YUYV,<br>
> > +        libsdl2-dev is an optional dependancy, if you would like to use SDL<br>
> > +        sink with pixel formats such as MJPEG, libsdl2-image-dev is an<br>
> > +        optional dependancy.<br>
<br>
This is quite some terse reading.<br>
<br>
In an effort to make it easier I'll have a go:<br>
<br>
        libevent-dev is required to support cam, however the following<br>
        optional dependencies bring more functionality to the cam test<br>
        tool:<br>
<br>
        libdrm-dev: Enables the KMS sink<br>
        libsdl2-dev: Enables the SDL sink<br>
        libsdl2-image-dev: Supports MJPEG on the SDL sink<br>
<br>
<br>
> ><br>
> >  for qcam: [optional]<br>
> >          qtbase5-dev libqt5core5a libqt5gui5 libqt5widgets5 qttools5-dev-tools libtiff-dev<br>
> > --<br>
> > 2.35.3<br>
> ><br>
><br>
<br>
</blockquote></div>