Hi Rafael,
On Mon, Mar 11 2019, Rafael Sadowski <raf...@sizeofvoid.org> wrote:
> Update opencv to 3.4.2.
>
> Tested with upcoming nomacs update and digikam.
>
> Face detection and recognition works quite well with this version and
> upcoming digikam5.
Was there a reason for this change
> -MODULES = devel/cmake \
> +MODULES = lang/clang \
> + devel/cmake \
> lang/python
when
COMPILER = base-clang ports-gcc base-gcc
?
I see that there's some code in opencv that #includes <llvm/...>:
./3rdparty/ittnotify/include/llvm_jit_event_listener.hpp
but it doesn't appear to be built nor shipped.
If there's indeed no reason to use the clang module then I'd suggest the
patch below (build ongoing). No bump needed since only BUILD_DEPENDS
change.
Index: Makefile
===================================================================
RCS file: /cvs/ports/graphics/opencv/Makefile,v
retrieving revision 1.49
diff -u -p -r1.49 Makefile
--- Makefile 11 Jun 2019 00:38:35 -0000 1.49
+++ Makefile 1 Jul 2019 10:33:04 -0000
@@ -41,7 +41,7 @@ WANTLIB-java += opencv_videoio
WANTLIB-docs =
-COMPILER = base-clang ports-gcc base-gcc
+COMPILER = base-clang ports-gcc
MASTER_SITES = ${MASTER_SITE_SOURCEFORGE:=opencvlibrary/}
EXTRACT_SUFX = .zip
@@ -55,8 +55,7 @@ PERMIT_PACKAGE_CDROM = Yes
.include <bsd.port.arch.mk>
-MODULES = lang/clang \
- devel/cmake \
+MODULES = devel/cmake \
lang/python
BUILD_DEPENDS = devel/doxygen \
--
jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF DDCC 0DFA 74AE 1524 E7EE
