mbox series

[0/2] Fix Opencv4 bug found by a user in Mailing list

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

Message

Giulio Benetti Aug. 7, 2022, 10:25 p.m. UTC 
These patches fixes a bug Stefano Cottafavi found and asked help for in
Mailing List:
https://lists.buildroot.org/pipermail/buildroot/2022-August/648616.html

It turns out that opencv4 library objdetect requires library dnn and that
also dnn library requires protobuf. So this patchset select both options
accordingly.

@Stefano, if you can, please give a try to these 2 patches without
enabling the 2 options I've pointed you in Mailing List by hand and report
the result.

Thank you!

Giulio Benetti (2):
  package/opencv4: BR2_PACKAGE_OPENCV4_LIB_DNN requires protobuf
  package/opencv4: BR2_PACKAGE_OPENCV4_LIB_OBJDETECT requires
    BR2_PACKAGE_OPENCV4_LIB_DNN

 package/opencv4/Config.in | 2 ++
 1 file changed, 2 insertions(+)

Comments

Giulio Benetti Aug. 13, 2022, 2:15 p.m. UTC | #1 
Hi Stefano,

I add ML in Cc,

On 13/08/22 16:06, Stefano Cottafavi wrote:
> Hi Giulio,
> sorry for the late reply but have been distracted by more pressing issues.

No problem

> Today I manually applied the patches and they worked like a charm.
> In the menuconfig dependencies are selected automatically as expected
> and the modules are built fine and exported to the sdk.

Thank you for testing.

> Thanks a lot for the support.

You're welcome :-)

> ciao

Ciao

Best regards