From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vicente Olivert Riera Date: Thu, 6 Mar 2014 10:34:51 +0000 Subject: [Buildroot] [PATCH] gst1-plugins-bad: add missing opencv ml module dependency In-Reply-To: <20140306113147.2935873b@skate> References: <1394060130-4805-1-git-send-email-s.martin49@gmail.com> <53184691.40008@imgtec.com> <20140306113147.2935873b@skate> Message-ID: <53184F4B.6010903@imgtec.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net On 03/06/2014 10:31 AM, Thomas Petazzoni wrote: > Dear Vicente Olivert Riera, > > On Thu, 6 Mar 2014 09:57:37 +0000, Vicente Olivert Riera wrote: >> On 03/05/2014 10:55 PM, Samuel Martin wrote: >>> gst1-plugins-bad source code does include the old-style global opencv >>> header "opencv/cv.h", which includes headers from the core, imgproc, >>> video, features2d, flann, calib3d, objdetect and legacy modules. >>> >>> However, the legacy module, in addition to headers from some of the >>> previously listed modules, also includes headers from the ml module. >>> >>> Thus, the ml module must be selected to be able to build the legacy >>> one, so the gst-plugins-bad's opencv plugin. >>> >>> Fixes: >>> http://autobuild.buildroot.net/results/d02/d02e1843ef90f4b40c99e3b23eddf376f827d46e/ >>> >>> Signed-off-by: Samuel Martin >>> --- >>> package/gstreamer1/gst1-plugins-bad/Config.in | 1 + >>> 1 file changed, 1 insertion(+) >>> >>> diff --git a/package/gstreamer1/gst1-plugins-bad/Config.in b/package/gstreamer1/gst1-plugins-bad/Config.in >>> index 0685322..2c5c290 100644 >>> --- a/package/gstreamer1/gst1-plugins-bad/Config.in >>> +++ b/package/gstreamer1/gst1-plugins-bad/Config.in >>> @@ -434,6 +434,7 @@ config BR2_PACKAGE_GST1_PLUGINS_BAD_PLUGIN_OPENCV >>> select BR2_PACKAGE_OPENCV_LIB_FLANN >>> select BR2_PACKAGE_OPENCV_LIB_IMGPROC >>> select BR2_PACKAGE_OPENCV_LIB_LEGACY >>> + select BR2_PACKAGE_OPENCV_LIB_ML >>> select BR2_PACKAGE_OPENCV_LIB_OBJDETECT >>> select BR2_PACKAGE_OPENCV_LIB_VIDEO >>> help >>> >> >> Maybe your BR tree wasn't up to date in the moment to write this patch. >> Look at this: >> >> http://git.buildroot.net/buildroot/commit/?id=3324f4e8b69ab49783a233b27c9b2edeecedb5ce > > And? Samuel is just adding one more line here, in addition to the lines > you added in the commit you're pointing at. > > Thomas > You are right, sorry. -- Vincent