On Fri, 2019-10-25 at 19:59 +0200, Martin Jansa wrote: > I think MYRIAD is already enabled by default: > https://github.com/opencv/dldt/blob/2019/inference-engine/cmake/features.cmake#L48 > when not disabled together with VPU: > https://github.com/opencv/dldt/blob/2019/inference-engine/cmake/check_features.cmake#L42 > > so I'm not sure if we need 2 PACKAGECONFIGs for that.
Thanks, I agree. I somehow missed that it's ON by default. Will send a separate patch with just the installation of udev rules for myriad based on vpu PACKAGECONFIG. > > Also using 8 spaces for indentation is inconsistent with everything > else, oe-core is using tabs for shell tasks and 4 spaces for python > task, most other layers are using 4 spaces everywhere (worst cases > combine both in the same task - e.g. do_install_ptest_base_prepend in > this recipe). I have been doing it this way in meta-intel and might also have done it for some oe-core recipes as well in the past. I don't have any preference though and will change. Thanks, Anuj -- _______________________________________________ meta-intel mailing list [email protected] https://lists.yoctoproject.org/listinfo/meta-intel
