* [Buildroot] [PATCH v4 1/1] package/freeswitch: select OpenCV modules needed for mod_cv
@ 2016-04-19 17:00 Bernd Kuhls
2016-04-19 19:05 ` Thomas Petazzoni
2016-04-20 11:12 ` Samuel Martin
0 siblings, 2 replies; 4+ messages in thread
From: Bernd Kuhls @ 2016-04-19 17:00 UTC (permalink / raw)
To: buildroot
Fixes
http://autobuild.buildroot.net/results/ffd/ffd572d7bb68d8ed46ce76b2af824503c587449f/
http://autobuild.buildroot.net/results/e53/e53e356451aa4541a76e26722fd71051896e774b/
http://autobuild.buildroot.net/results/564/5649ff1b50a1eaafbd0e1a329bff87a65ce18da0/
http://autobuild.buildroot.net/results/c60/c60f97c6aafb1a94cb4ce344953598eae8b4a912/
http://autobuild.buildroot.net/results/c1a/c1ac61f9a77635962babdb6bfb31dbecb7f4d439/
Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
v4: add more suboptions (Samuel)
v3: add more suboptions
v2: select the required suboption instead of checking for its existance
in freeswitch.mk (Thomas)
package/freeswitch/Config.in | 11 +++++++++++
1 file changed, 11 insertions(+)
diff --git a/package/freeswitch/Config.in b/package/freeswitch/Config.in
index 1350b2b..cef3fd4 100644
--- a/package/freeswitch/Config.in
+++ b/package/freeswitch/Config.in
@@ -7,6 +7,17 @@ menuconfig BR2_PACKAGE_FREESWITCH
depends on BR2_USE_WCHAR # libuuid
select BR2_PACKAGE_LIBCURL
select BR2_PACKAGE_JPEG
+ # src/mod/applications/mod_cv/mod_cv.cpp includes cv.h
+ select BR2_PACKAGE_OPENCV_LIB_CALIB3D if BR2_PACKAGE_OPENCV
+ select BR2_PACKAGE_OPENCV_LIB_CONTRIB if BR2_PACKAGE_OPENCV
+ select BR2_PACKAGE_OPENCV_LIB_FEATURES2D if BR2_PACKAGE_OPENCV
+ select BR2_PACKAGE_OPENCV_LIB_FLANN if BR2_PACKAGE_OPENCV
+ select BR2_PACKAGE_OPENCV_LIB_IMGPROC if BR2_PACKAGE_OPENCV
+ select BR2_PACKAGE_OPENCV_LIB_LEGACY if BR2_PACKAGE_OPENCV
+ select BR2_PACKAGE_OPENCV_LIB_OBJDETECT if BR2_PACKAGE_OPENCV
+ select BR2_PACKAGE_OPENCV_LIB_VIDEO if BR2_PACKAGE_OPENCV
+ # src/mod/applications/mod_cv/mod_cv.cpp includes highgui.h
+ select BR2_PACKAGE_OPENCV_LIB_HIGHGUI if BR2_PACKAGE_OPENCV
select BR2_PACKAGE_OPENSSL
select BR2_PACKAGE_PCRE
select BR2_PACKAGE_SPEEX
--
2.8.0.rc3
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH v4 1/1] package/freeswitch: select OpenCV modules needed for mod_cv
2016-04-19 17:00 [Buildroot] [PATCH v4 1/1] package/freeswitch: select OpenCV modules needed for mod_cv Bernd Kuhls
@ 2016-04-19 19:05 ` Thomas Petazzoni
2016-04-19 20:08 ` Bernd Kuhls
2016-04-20 11:12 ` Samuel Martin
1 sibling, 1 reply; 4+ messages in thread
From: Thomas Petazzoni @ 2016-04-19 19:05 UTC (permalink / raw)
To: buildroot
Hello,
On Tue, 19 Apr 2016 19:00:45 +0200, Bernd Kuhls wrote:
> diff --git a/package/freeswitch/Config.in b/package/freeswitch/Config.in
> index 1350b2b..cef3fd4 100644
> --- a/package/freeswitch/Config.in
> +++ b/package/freeswitch/Config.in
> @@ -7,6 +7,17 @@ menuconfig BR2_PACKAGE_FREESWITCH
> depends on BR2_USE_WCHAR # libuuid
> select BR2_PACKAGE_LIBCURL
> select BR2_PACKAGE_JPEG
> + # src/mod/applications/mod_cv/mod_cv.cpp includes cv.h
> + select BR2_PACKAGE_OPENCV_LIB_CALIB3D if BR2_PACKAGE_OPENCV
> + select BR2_PACKAGE_OPENCV_LIB_CONTRIB if BR2_PACKAGE_OPENCV
> + select BR2_PACKAGE_OPENCV_LIB_FEATURES2D if BR2_PACKAGE_OPENCV
> + select BR2_PACKAGE_OPENCV_LIB_FLANN if BR2_PACKAGE_OPENCV
> + select BR2_PACKAGE_OPENCV_LIB_IMGPROC if BR2_PACKAGE_OPENCV
> + select BR2_PACKAGE_OPENCV_LIB_LEGACY if BR2_PACKAGE_OPENCV
> + select BR2_PACKAGE_OPENCV_LIB_OBJDETECT if BR2_PACKAGE_OPENCV
> + select BR2_PACKAGE_OPENCV_LIB_VIDEO if BR2_PACKAGE_OPENCV
> + # src/mod/applications/mod_cv/mod_cv.cpp includes highgui.h
> + select BR2_PACKAGE_OPENCV_LIB_HIGHGUI if BR2_PACKAGE_OPENCV
Now that inter-module dependencies are properly handled in opencv.mk,
why do we have to do all of this?
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH v4 1/1] package/freeswitch: select OpenCV modules needed for mod_cv
2016-04-19 19:05 ` Thomas Petazzoni
@ 2016-04-19 20:08 ` Bernd Kuhls
0 siblings, 0 replies; 4+ messages in thread
From: Bernd Kuhls @ 2016-04-19 20:08 UTC (permalink / raw)
To: buildroot
[posted and mailed]
Hi Thomas,
Thomas Petazzoni
<thomas.petazzoni@free-electrons.com> wrote
in news:20160419210522.1df89766 at free-electrons.com:
> Now that inter-module dependencies are properly handled in opencv.mk,
> why do we have to do all of this?
Hi Thomas,
because I interpreted Samuels mail[1] in a way that he wants to see all
opencv modules mentioned in freeswitch's Config.in.
My patch enables all opencv modules included in
freeswitch/src/mod/applications/mod_cv/mod_cv.cpp, regardless of the code
in opencv/Config.in.
If you prefer a minimal solution you can commit v3:
http://patchwork.ozlabs.org/patch/611882/
Regards, Bernd
[1] http://lists.busybox.net/pipermail/buildroot/2016-April/159041.html
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH v4 1/1] package/freeswitch: select OpenCV modules needed for mod_cv
2016-04-19 17:00 [Buildroot] [PATCH v4 1/1] package/freeswitch: select OpenCV modules needed for mod_cv Bernd Kuhls
2016-04-19 19:05 ` Thomas Petazzoni
@ 2016-04-20 11:12 ` Samuel Martin
1 sibling, 0 replies; 4+ messages in thread
From: Samuel Martin @ 2016-04-20 11:12 UTC (permalink / raw)
To: buildroot
Bernd, all,
On Tue, Apr 19, 2016 at 7:00 PM, Bernd Kuhls <bernd.kuhls@t-online.de> wrote:
> Fixes
> http://autobuild.buildroot.net/results/ffd/ffd572d7bb68d8ed46ce76b2af824503c587449f/
> http://autobuild.buildroot.net/results/e53/e53e356451aa4541a76e26722fd71051896e774b/
> http://autobuild.buildroot.net/results/564/5649ff1b50a1eaafbd0e1a329bff87a65ce18da0/
> http://autobuild.buildroot.net/results/c60/c60f97c6aafb1a94cb4ce344953598eae8b4a912/
> http://autobuild.buildroot.net/results/c1a/c1ac61f9a77635962babdb6bfb31dbecb7f4d439/
>
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
> v4: add more suboptions (Samuel)
> v3: add more suboptions
> v2: select the required suboption instead of checking for its existance
> in freeswitch.mk (Thomas)
>
> package/freeswitch/Config.in | 11 +++++++++++
> 1 file changed, 11 insertions(+)
>
> diff --git a/package/freeswitch/Config.in b/package/freeswitch/Config.in
> index 1350b2b..cef3fd4 100644
> --- a/package/freeswitch/Config.in
> +++ b/package/freeswitch/Config.in
> @@ -7,6 +7,17 @@ menuconfig BR2_PACKAGE_FREESWITCH
> depends on BR2_USE_WCHAR # libuuid
> select BR2_PACKAGE_LIBCURL
> select BR2_PACKAGE_JPEG
> + # src/mod/applications/mod_cv/mod_cv.cpp includes cv.h
and cvaux.h and cxmisc.h.
cvaux.h also pulls the photo module, which does not get selected
(either directly here or after via another opencv module in the
current state of the master).
Side note: freeswitch mixes deprecated legacy[1] and opencv2-style[2]
header include statement. Maybe just removing the legacy headers is
enough to not pull a (almost) full opencv, and reduce this select
list? but that's another question for freswitch upstream ;)
> + select BR2_PACKAGE_OPENCV_LIB_CALIB3D if BR2_PACKAGE_OPENCV
> + select BR2_PACKAGE_OPENCV_LIB_CONTRIB if BR2_PACKAGE_OPENCV
hum... where does the contrib module come from?
> + select BR2_PACKAGE_OPENCV_LIB_FEATURES2D if BR2_PACKAGE_OPENCV
> + select BR2_PACKAGE_OPENCV_LIB_FLANN if BR2_PACKAGE_OPENCV
> + select BR2_PACKAGE_OPENCV_LIB_IMGPROC if BR2_PACKAGE_OPENCV
> + select BR2_PACKAGE_OPENCV_LIB_LEGACY if BR2_PACKAGE_OPENCV
> + select BR2_PACKAGE_OPENCV_LIB_OBJDETECT if BR2_PACKAGE_OPENCV
> + select BR2_PACKAGE_OPENCV_LIB_VIDEO if BR2_PACKAGE_OPENCV
> + # src/mod/applications/mod_cv/mod_cv.cpp includes highgui.h
> + select BR2_PACKAGE_OPENCV_LIB_HIGHGUI if BR2_PACKAGE_OPENCV
> select BR2_PACKAGE_OPENSSL
> select BR2_PACKAGE_PCRE
> select BR2_PACKAGE_SPEEX
> --
> 2.8.0.rc3
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
[1] https://freeswitch.org/stash/projects/FS/repos/freeswitch/browse/src/mod/applications/mod_cv/mod_cv.cpp#43
[2] https://freeswitch.org/stash/projects/FS/repos/freeswitch/browse/src/mod/applications/mod_cv/mod_cv.cpp#34
Regards,
--
Samuel
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2016-04-20 11:12 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-04-19 17:00 [Buildroot] [PATCH v4 1/1] package/freeswitch: select OpenCV modules needed for mod_cv Bernd Kuhls
2016-04-19 19:05 ` Thomas Petazzoni
2016-04-19 20:08 ` Bernd Kuhls
2016-04-20 11:12 ` Samuel Martin
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox