Yes, that is fine. I suppose there is no way to detect the OpenCV contrib module in configure? It would suck to get a bunch of compilation error reports if OpenCV is detected fine but the contrib module is not. If that is the case, then make this module disabled by default and require explicit enable.
On Tue, Sep 13, 2016 at 9:17 AM jb <j...@kdenlive.org> wrote: > Hi all, > > I'd finally like to merge my motion tracker module based on OpenCV. It > requires OpenCV >= 3.1 and its contrib module, which is sadly not yet > commonly available as distro package, so it won't compile unless you > have these dependencies. > > Is it ok if I merge on friday ? > > Source is here, I will fix the yml validation issue and review the code > once more before merging. > > https://github.com/j-b-m/mlt/tree/opencv/src/modules/opencv > > Thanks > > Jb > > > > ------------------------------------------------------------------------------ > _______________________________________________ > Mlt-devel mailing list > Mlt-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/mlt-devel >
------------------------------------------------------------------------------
_______________________________________________ Mlt-devel mailing list Mlt-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mlt-devel
