On Sat, Nov 23, 2024 at 09:15:17PM -0600, joshua stein wrote: > 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:
Will need a revision bump, otherwise ok > > > 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} > >