Re: [PATCH][meta-oe] opencv: fix packaging and install

[Martin Jansa <martin.jansa@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 2455 bytes --]

On Mon, Oct 10, 2016 at 05:31:05PM +0100, Ross Burton wrote:
> The OpenCV*.cmake files and the static library should be in the opencv-dev
> package, not opencv-apps.

Why not PN-staticdev?

> There is no need to specify files for PN-dbg as they're automatically found, so
> remove the FILES_PN-dbg assignment.
> 
> Conditionalise the installation of the example and tutorial binaries so it
> doesn't abort if samples are disabled.
> 
> Signed-off-by: Ross Burton <ross.burton@intel.com>
> ---
>  meta-oe/recipes-support/opencv/opencv_3.1.bb | 11 ++++++-----
>  1 file changed, 6 insertions(+), 5 deletions(-)
> 
> diff --git a/meta-oe/recipes-support/opencv/opencv_3.1.bb b/meta-oe/recipes-support/opencv/opencv_3.1.bb
> index da11316..2136b07 100644
> --- a/meta-oe/recipes-support/opencv/opencv_3.1.bb
> +++ b/meta-oe/recipes-support/opencv/opencv_3.1.bb
> @@ -116,15 +116,14 @@ PACKAGES_DYNAMIC += "^libopencv-.*"
>  
>  FILES_${PN} = ""
>  FILES_${PN}-apps = "${bindir}/* ${datadir}/OpenCV"
> -FILES_${PN}-dbg += "${libdir}/.debug"
> -FILES_${PN}-dev = "${includedir} ${libdir}/pkgconfig"
> +FILES_${PN}-dev = "${includedir} ${libdir}/pkgconfig ${datadir}/OpenCV/*.cmake ${datadir}/OpenCV/3rdparty/lib/*.a"
>  FILES_${PN}-doc = "${datadir}/OpenCV/doc"
>  FILES_${PN}-java = "${datadir}/OpenCV/java"
>  FILES_${PN}-java-dbg = "${datadir}/OpenCV/java/.debug/"
>  FILES_${PN}-samples = "${datadir}/OpenCV/samples/"
>  FILES_${PN}-samples-dbg = "${datadir}/OpenCV/samples/bin/.debug"
>  
> -INSANE_SKIP_${PN}-apps = "staticdev"
> +INSANE_SKIP_${PN}-dev = "staticdev"
>  INSANE_SKIP_${PN}-java = "libdir"
>  INSANE_SKIP_${PN}-java-dbg = "libdir"
>  
> @@ -144,6 +143,8 @@ do_install_append() {
>          rm -rf ${D}/usr/lib
>      fi
>  
> -    install -d ${D}${datadir}/OpenCV/samples/bin/
> -    cp -f bin/*-tutorial-* bin/*-example-* ${D}${datadir}/OpenCV/samples/bin/
> +    if ${@bb.utils.contains("PACKAGECONFIG", "samples", "true", "false", d)}; then
> +        install -d ${D}${datadir}/OpenCV/samples/bin/
> +        cp -f bin/*-tutorial-* bin/*-example-* ${D}${datadir}/OpenCV/samples/bin/
> +    fi
>  }
> -- 
> 2.8.1
> 
> -- 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 169 bytes --]