[libcamera-devel] [PATCH] Documentation: guides: application-developer: update pkgconfig name

Laurent Pinchart laurent.pinchart at ideasonboard.com
Tue Jul 20 11:26:09 CEST 2021


Hi Paul,

Thank you for the patch.

On Tue, Jul 20, 2021 at 01:16:09PM +0900, Paul Elder wrote:
> The pkgconfig name was changed from camera to libcamera, but the
> application developer guide wasn't updated. Update it to refer to the
> new name.
> 
> Bug: https://bugs.libcamera.org/show_bug.cgi?id=65
> Fixes: ec7afef6 ("libcamera: Rename libcamera pkg-config generation")
> Signed-off-by: Paul Elder <paul.elder at ideasonboard.com>
> ---
>  Documentation/guides/application-developer.rst | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/Documentation/guides/application-developer.rst b/Documentation/guides/application-developer.rst
> index e3430f03..9c7b1bb0 100644
> --- a/Documentation/guides/application-developer.rst
> +++ b/Documentation/guides/application-developer.rst
> @@ -596,12 +596,12 @@ has been installed in your system.
>  
>     export PKG_CONFIG_PATH=/usr/local/lib/pkgconfig/
>  
> -Verify that ``pkg-config`` can identify the ``camera`` library with
> +Verify that ``pkg-config`` can identify the ``libcamera`` library with
>  
>  .. code:: shell
>  
> -   $ pkg-config --libs --cflags camera
> -     -I/usr/local/include/libcamera -L/usr/local/lib -lcamera
> +   $ pkg-config --libs --cflags libcamera
> +     -I/usr/local/include/libcamera -L/usr/local/lib -lcamera -lcamera-base
>  
>  ``meson`` can alternatively use ``cmake`` to locate packages, please refer to
>  the ``meson`` documentation if you prefer to use it in place of ``pkgconfig``
> @@ -621,7 +621,7 @@ accordingly. In this example, the application file has been named
>  
>     simpler_cam = executable('simple-cam',
>         'simple-cam.cpp',
> -       dependencies: dependency('camera', required : true))
> +       dependencies: dependency('libcamera', required : true))
>  
>  The ``dependencies`` line instructs meson to ask ``pkgconfig`` (or ``cmake``) to
>  locate the ``camera`` library, (libcamera without the lib prefix) which the test

This should be updated too. There may be other occurences I have missed.

-- 
Regards,

Laurent Pinchart


More information about the libcamera-devel mailing list