[libcamera-devel] [PATCH 2/2] libcamera: Document dependencies for IPA module signing

[Kieran Bingham]

Hi Laurent,

On 15/04/2020 20:37, Laurent Pinchart wrote:
> Document the dependency on libgnutls28-dev and openssl for IPA module
> signing int he README.rst file. While at it, sort the entries
> alphabetically.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> ---
>  README.rst | 9 ++++++---
>  1 file changed, 6 insertions(+), 3 deletions(-)
> 
> diff --git a/README.rst b/README.rst
> index 656fdbfdc94c..da3cc64966b0 100644
> --- a/README.rst
> +++ b/README.rst
> @@ -60,15 +60,18 @@ for libcamera: [required]
>  for device hotplug enumeration: [optional]
>  	pkg-config libudev-dev
>  
> -for qcam: [optional]
> -	qtbase5-dev libqt5core5a libqt5gui5 libqt5widgets5
> -
>  for documentation: [optional]
>  	python3-sphinx doxygen
>  
>  for gstreamer: [optional]
>  	libgstreamer1.0-dev libgstreamer-plugins-base1.0-dev
>  
> +for IPA module signing: [optional]
> +        libgnutls28-dev openssl

Hrmm... Currently this is not [optional], but [required], as the
resulting code paths which get executed otherwise are non-functional :-[

Can we make this [required] until the IPAProxyLinux is implemented
correctly?

Otherwise - the user is presented with completely un-obvious segfaults
on the EventNotifier destructor when it perhaps incorrectly obtains an
invalid EventDispatcher *1


*1: Not fully investigated.

--
Kieran



> +
> +for qcam: [optional]
> +	qtbase5-dev libqt5core5a libqt5gui5 libqt5widgets5
> +
>  Using GStreamer plugin
>  ~~~~~~~~~~~~~~~~~~~~~~
>  
> 

-- 
Regards
--
Kieran