[Bug 239313] graphics/mesa-dri could use a test for the installed llvm

[bugzilla-noreply at freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=239313

dewayne at heuristicsystems.com.au changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |dewayne at heuristicsystems.co
                   |                            |m.au
           Severity|Affects Only Me             |Affects Some People

--- Comment #1 from dewayne at heuristicsystems.com.au ---
As I have clang 8.0.0 from the base os and gcc9 for those tricky ports, I find
that mesa-dri requires a build of llvm8.  Can I suggest that a test for the CC
be included and possibly wrapped around this segment from the Makefile.common

.if ${ARCH} == aarch64 || ${ARCH} == amd64 || ${ARCH:Marm*} || ${ARCH} == i386
|| ${ARCH:Mmi
BUILD_DEPENDS+= llvm${LLVM_DEFAULT}>=3.9.0_4:devel/llvm${LLVM_DEFAULT}
.if ${COMPONENT} != libs
RUN_DEPENDS+=   llvm${LLVM_DEFAULT}>=3.9.0_4:devel/llvm${LLVM_DEFAULT}
.endif
CONFIGURE_ENV+= LLVM_CONFIG=${LOCALBASE}/bin/llvm-config${LLVM_DEFAULT}
LDFLAGS+=       -Wl,-rpath=${LOCALBASE}/llvm${LLVM_DEFAULT}/lib
CONFIGURE_ARGS+=        --enable-llvm
.else
CONFIGURE_ARGS+=        --disable-llvm
.endif

This would reduce the unnecessary dependency and ongoing maintenance.  (as well
as the build time ;) )

My apologies if llvm is actually required regardless of the available compiler
on the build machine, but this is pulled in due to libepoxy even though I tell
qemu not to use x11.  A complex web!

-- 
You are receiving this mail because:
You are the assignee for the bug.