diff mbox series

[2/2] package/opencv4: BR2_PACKAGE_OPENCV4_LIB_OBJDETECT requires BR2_PACKAGE_OPENCV4_LIB_DNN

Message ID 20220807222529.1501004-3-giulio.benetti@benettiengineering.com
State Accepted
Headers show
Series Fix Opencv4 bug found by a user in Mailing list | expand

Commit Message

Giulio Benetti Aug. 7, 2022, 10:25 p.m. UTC
Opencv4 library objdetect needs library dnn, otherwise it throws:
```
-- Module opencv_objdetect disabled because opencv_dnn dependency can't be resolved!
```
So let's "select BR2_PACKAGE_OPENCV4_LIB_DNN" if
BR2_PACKAGE_OPENCV4_LIB_OBJDETECT is enabled.

Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
---
 package/opencv4/Config.in | 1 +
 1 file changed, 1 insertion(+)

Comments

Thomas Petazzoni Aug. 15, 2022, 12:35 p.m. UTC | #1
On Mon,  8 Aug 2022 00:25:29 +0200
Giulio Benetti <giulio.benetti@benettiengineering.com> wrote:

> Opencv4 library objdetect needs library dnn, otherwise it throws:
> ```
> -- Module opencv_objdetect disabled because opencv_dnn dependency can't be resolved!
> ```
> So let's "select BR2_PACKAGE_OPENCV4_LIB_DNN" if
> BR2_PACKAGE_OPENCV4_LIB_OBJDETECT is enabled.
> 
> Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
> ---
>  package/opencv4/Config.in | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/package/opencv4/Config.in b/package/opencv4/Config.in
> index bdeddcc089..4991b4252b 100644
> --- a/package/opencv4/Config.in
> +++ b/package/opencv4/Config.in
> @@ -155,6 +155,7 @@ config BR2_PACKAGE_OPENCV4_LIB_OBJDETECT
>  	select BR2_PACKAGE_OPENCV4_LIB_CALIB3D
>  	select BR2_PACKAGE_OPENCV4_LIB_IMGPROC
>  	select BR2_PACKAGE_OPENCV4_LIB_ML
> +	select BR2_PACKAGE_OPENCV4_LIB_DNN

Alphabetic ordering was not correct, and it also lacked the propagation
of the "depends on" of select BR2_PACKAGE_OPENCV4_LIB_DNN.

The final patch is at:

  https://git.buildroot.org/buildroot/commit/?id=e229943785534e0884c62c7346a8c6df8d565e1e

Best regards,

Thomas
Giulio Benetti Aug. 15, 2022, 12:45 p.m. UTC | #2
> Il giorno 15 ago 2022, alle ore 14:35, Thomas Petazzoni <thomas.petazzoni@bootlin.com> ha scritto:
> 
> On Mon,  8 Aug 2022 00:25:29 +0200
> Giulio Benetti <giulio.benetti@benettiengineering.com> wrote:
> 
>> Opencv4 library objdetect needs library dnn, otherwise it throws:
>> ```
>> -- Module opencv_objdetect disabled because opencv_dnn dependency can't be resolved!
>> ```
>> So let's "select BR2_PACKAGE_OPENCV4_LIB_DNN" if
>> BR2_PACKAGE_OPENCV4_LIB_OBJDETECT is enabled.
>> 
>> Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
>> ---
>> package/opencv4/Config.in | 1 +
>> 1 file changed, 1 insertion(+)
>> 
>> diff --git a/package/opencv4/Config.in b/package/opencv4/Config.in
>> index bdeddcc089..4991b4252b 100644
>> --- a/package/opencv4/Config.in
>> +++ b/package/opencv4/Config.in
>> @@ -155,6 +155,7 @@ config BR2_PACKAGE_OPENCV4_LIB_OBJDETECT
>>    select BR2_PACKAGE_OPENCV4_LIB_CALIB3D
>>    select BR2_PACKAGE_OPENCV4_LIB_IMGPROC
>>    select BR2_PACKAGE_OPENCV4_LIB_ML
>> +    select BR2_PACKAGE_OPENCV4_LIB_DNN
> 
> Alphabetic ordering was not correct, and it also lacked the propagation
> of the "depends on" of select BR2_PACKAGE_OPENCV4_LIB_DNN.

Oh no, again the dependencies in Config.in!
I still keep forgetting them :-/

> 
> The final patch is at:
> 
>  https://git.buildroot.org/buildroot/commit/?id=e229943785534e0884c62c7346a8c6df8d565e1e

Thank you Thomas, you’re too kind.

Best regards
Giulio

> 
> Best regards,
> 
> Thomas
> -- 
> Thomas Petazzoni, co-owner and CEO, Bootlin
> Embedded Linux and Kernel engineering and training
> https://bootlin.com
Peter Korsgaard Sept. 16, 2022, 12:22 p.m. UTC | #3
>>>>> "Giulio" == Giulio Benetti <giulio.benetti@benettiengineering.com> writes:

 > Opencv4 library objdetect needs library dnn, otherwise it throws:
 > ```
 > -- Module opencv_objdetect disabled because opencv_dnn dependency can't be resolved!
 > ```
 > So let's "select BR2_PACKAGE_OPENCV4_LIB_DNN" if
 > BR2_PACKAGE_OPENCV4_LIB_OBJDETECT is enabled.

 > Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>

Committed to 2022.05.x and 2022.02.x, thanks.
diff mbox series

Patch

diff --git a/package/opencv4/Config.in b/package/opencv4/Config.in
index bdeddcc089..4991b4252b 100644
--- a/package/opencv4/Config.in
+++ b/package/opencv4/Config.in
@@ -155,6 +155,7 @@  config BR2_PACKAGE_OPENCV4_LIB_OBJDETECT
 	select BR2_PACKAGE_OPENCV4_LIB_CALIB3D
 	select BR2_PACKAGE_OPENCV4_LIB_IMGPROC
 	select BR2_PACKAGE_OPENCV4_LIB_ML
+	select BR2_PACKAGE_OPENCV4_LIB_DNN
 	help
 	  Include opencv_objdetect (object detection) module into the
 	  OpenCV build.