Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 10 Nov 2017 14:07:48 +0700
From:      Alexey Dokuchaev <danfe@nsu.ru>
To:        brooks@freebsd.org
Cc:        ports@freebsd.org
Subject:   RTTI support in devel/llvm40 (and maybe other llvm ports)
Message-ID:  <20171110070748.GA27570@regency.nsu.ru>

next in thread | raw e-mail | index | archive | help
Hi Brooks,

I've just found out that our `devel/llvm40' port comes without
-DLLVM_ENABLE_RTTI=ON on the CMAKE_ARGS.  This is a regression
from e.g. 3.4 times when it was enabled by default.

The problem is that RTTI support is required by some consumers,
e.g. `graphics/openshadinglanguage' and `graphics/appleseed'
(cf. https://github.com/appleseedhq/appleseed/issues/1625),
but I cannot enable RTTI in those ports unless I enable it in
LLVM port(s) first.

The patch is very simple (apart port revision bump):

@@ -39,7 +41,7 @@
 SUB_LIST=	LLVM_PREFIX="${LLVM_PREFIX}" LLVM_SUFFIX="${LLVM_SUFFIX}"
 CMAKE_INSTALL_PREFIX=  ${LLVM_PREFIX}
-CMAKE_ARGS=	-DLLVM_BUILD_LLVM_DYLIB=ON
+CMAKE_ARGS=	-DLLVM_BUILD_LLVM_DYLIB=ON -DLLVM_ENABLE_RTTI=ON

Could you review/commit it, and check if other LLVM ports could
also benefit from it?  Thanks,

./danfe



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20171110070748.GA27570>