this is a great start but I think we would like to have this use a flag so user can build with OPENCV=ON. Right now this would just appear as a bonus if people have both but if for example someone wants OpenCV integration and they don't have OpenCV installed we won't warn them about it - makes sense?

Have a look at the way we do it for python and python modular