Download raw body.
BUG: we should add devel/llvm/17 to RUN_DEPENDS on lang/crystal
BUG: we should add devel/llvm/17 to RUN_DEPENDS on lang/crystal
On Sat, 23 Nov 2024 at 20:27:18 -0600, izzy Meyer wrote:
> Hello ports@
>
> Crystal is a compiled language, and needs to link against llvm--%17 as
> it currently stands. Most users who want to use this language will need
> to compile.
>
> If devel/llvm/17 isn't installed, it fails with:
> Dependencies are satisfied
> Building: whatever
> Error target whatever failed to compile:
> ld.so: crystal: can't load library 'libLLVM-17.so'
>
> Could we add llvm--%17 to the RUN_DEPENDS?
It used to be there before someone switched it to use the clang
module, but this should add it back:
diff --git lang/crystal/Makefile lang/crystal/Makefile
index 6d978c85add..81b166e535e 100644
--- lang/crystal/Makefile
+++ lang/crystal/Makefile
@@ -44,6 +44,7 @@ DISTFILES.c = crystal-molinillo-${V}{v${V_MOLINILLO}}.tar.gz
# environment.
MODULES = lang/clang
MODCLANG_VERSION = 17
+MODCLANG_RUNDEP= yes
COMPILER = ports-clang
LLVM_CONFIG = llvm-config-${MODCLANG_VERSION}
BUG: we should add devel/llvm/17 to RUN_DEPENDS on lang/crystal
BUG: we should add devel/llvm/17 to RUN_DEPENDS on lang/crystal