From: Martin Jansa <martin.jansa@gmail.com>
To: openembedded-devel@lists.openembedded.org
Subject: Re: [meta-oe][PATCH] opencv: Upgrade to 2.4.11
Date: Wed, 4 Mar 2015 23:03:48 +0100 [thread overview]
Message-ID: <20150304220348.GK2337@jama> (raw)
In-Reply-To: <1425457252-8056-1-git-send-email-mike.looijmans@topic.nl>
[-- Attachment #1: Type: text/plain, Size: 4133 bytes --]
On Wed, Mar 04, 2015 at 09:20:52AM +0100, Mike Looijmans wrote:
> Upgrade OpenCV to the 2.4.11 release.
>
> Remove the opencv-fix-pkgconfig-generation patch which has been integrated upstream,
> be it in modified form.
> ---
> .../opencv/opencv-fix-pkgconfig-generation.patch | 44 ----------------------
> meta-oe/recipes-support/opencv/opencv_2.4.bb | 8 ++--
> 2 files changed, 3 insertions(+), 49 deletions(-)
> delete mode 100644 meta-oe/recipes-support/opencv/opencv/opencv-fix-pkgconfig-generation.patch
>
> diff --git a/meta-oe/recipes-support/opencv/opencv/opencv-fix-pkgconfig-generation.patch b/meta-oe/recipes-support/opencv/opencv/opencv-fix-pkgconfig-generation.patch
> deleted file mode 100644
> index d352778..0000000
> --- a/meta-oe/recipes-support/opencv/opencv/opencv-fix-pkgconfig-generation.patch
> +++ /dev/null
> @@ -1,44 +0,0 @@
> -Fix pkg-config generation
> -
> -Replace absolute library path with library name spec and library search
> -path option.
> -
> -The fix has been provided by Ray Rashif (code.opencv.org/issues/1925)
> -
> -Upstream-Status: Pending
> -
> -diff -Nbaur OpenCV-2.4.3.orig/cmake/OpenCVGenPkgconfig.cmake OpenCV-2.4.3/cmake/OpenCVGenPkgconfig.cmake
> ---- OpenCV-2.4.3.orig/cmake/OpenCVGenPkgconfig.cmake 2012-11-04 08:40:14.243505926 +0000
> -+++ OpenCV-2.4.3/cmake/OpenCVGenPkgconfig.cmake 2012-11-04 08:40:42.286649120 +0000
> -@@ -10,7 +10,7 @@
> - # -------------------------------------------------------------------------------------------
> - set(prefix "${CMAKE_INSTALL_PREFIX}")
> - set(exec_prefix "\${prefix}")
> --set(libdir "") #TODO: need link paths for OpenCV_EXTRA_COMPONENTS
> -+set(libdir "\${prefix}/${OPENCV_LIB_INSTALL_PATH}")
> - set(includedir "\${prefix}/${OPENCV_INCLUDE_INSTALL_PATH}")
> - set(VERSION ${OPENCV_VERSION})
> -
> -@@ -36,10 +36,11 @@
> - ocv_list_reverse(OpenCV_EXTRA_COMPONENTS)
> -
> - #build the list of components
> --set(OpenCV_LIB_COMPONENTS_ "")
> -+set(OpenCV_LIB_COMPONENTS_ "-L\${libdir}")
> - foreach(CVLib ${OpenCV_LIB_COMPONENTS})
> - get_target_property(libpath ${CVLib} LOCATION_${CMAKE_BUILD_TYPE})
> - get_filename_component(libname "${libpath}" NAME)
> -+ get_filename_component(lname "${libpath}" NAME_WE)
> -
> - if(INSTALL_TO_MANGLED_PATHS)
> - set(libname "${libname}.${OPENCV_VERSION}")
> -@@ -52,7 +53,8 @@
> - set(installDir "${OPENCV_LIB_INSTALL_PATH}")
> - endif()
> -
> -- set(OpenCV_LIB_COMPONENTS_ "${OpenCV_LIB_COMPONENTS_} \${exec_prefix}/${installDir}/${libname}")
> -+ string(REPLACE "libopencv" "-lopencv" lname "${lname}")
> -+ set(OpenCV_LIB_COMPONENTS_ "${OpenCV_LIB_COMPONENTS_} ${lname}")
> - endforeach()
> -
> - # add extra dependencies required for OpenCV
> diff --git a/meta-oe/recipes-support/opencv/opencv_2.4.bb b/meta-oe/recipes-support/opencv/opencv_2.4.bb
> index 63d7c8b..e57f9a6 100644
> --- a/meta-oe/recipes-support/opencv/opencv_2.4.bb
> +++ b/meta-oe/recipes-support/opencv/opencv_2.4.bb
> @@ -9,12 +9,10 @@ ARM_INSTRUCTION_SET = "arm"
>
> DEPENDS = "python-numpy libtool swig swig-native python bzip2 zlib glib-2.0"
>
> -SRCREV = "df8e28283f09825cca0c2902160b7abebcfe1b64"
> -SRC_URI = "git://github.com/Itseez/opencv.git;branch=2.4 \
> - file://opencv-fix-pkgconfig-generation.patch \
> -"
> +SRCREV = "2c9547e3147779001811d01936aed38f560929fc"
> +SRC_URI = "git://github.com/Itseez/opencv.git;branch=2.4"
>
> -PV = "2.4.9+git${SRCPV}"
> +PV = "2.4.11+git${SRCPV}"
Please Fix this issue first:
WARNING: QA Issue: libopencv-highgui rdepends on jasper, but it isn't a
build dependency? [build-deps]
WARNING: QA Issue: libopencv-highgui rdepends on libdc1394, but it isn't
a build dependency? [build-deps]
> S = "${WORKDIR}/git"
>
> --
> 1.9.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: 188 bytes --]
next prev parent reply other threads:[~2015-03-04 22:03 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-03-04 8:20 [meta-oe][PATCH] opencv: Upgrade to 2.4.11 Mike Looijmans
2015-03-04 22:03 ` Martin Jansa [this message]
2015-03-05 7:33 ` Mike Looijmans
2015-03-05 8:21 ` Martin Jansa
2015-03-05 10:24 ` [meta-oe][PATCH v2] " Mike Looijmans
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20150304220348.GK2337@jama \
--to=martin.jansa@gmail.com \
--cc=openembedded-devel@lists.openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox