On Wed, May 27, 2026 at 08:51:53PM +0200, Rafael Sadowski wrote:
> On Wed May 27, 2026 at 08:13:50PM +0200, Rafael Sadowski wrote:
> > On Wed May 27, 2026 at 04:36:40PM +0200, Christian Weisgerber wrote:
> > > http://build-failures.rhaalovely.net/amd64-clang/2026-05-26/
> > > 
> > > $ awk '/->/ { a[$NF]++ } END { for(i in a) printf "%3d %s\n", a[i], i }' \
> > >     summary.log | sort -nr
> > > 133 audio/openal
> > 
> > I think that is currently our best solution/pattern for the
> > "CMAKE_CXX_COMPILER_CLANG_SCAN_DEPS-NOTFOUND" error class:
> > 
> > MODULES +=  lang/clang
> > MODCLANG_COMPILER_LINKS = No
> > CONFIGURE_ARGS+=    
> > -DCMAKE_CXX_COMPILER_CLANG_SCAN_DEPS=${LOCALBASE}/bin/clang-scan-deps-${MODCLANG_VERSION}
> > 
> 
> Correction: the CONFIGURE_ARGS will be overwritten again by
> Modules/Compiler/Clang-FindBinUtils.cmake because
> MODCLANG_COMPILER_LINKS is set to "No" and the cmake module will force
> the compiler version.
> 
> Meaning:
> 
> Will fail if we switch base to 22 and ports stay 19:
> 
> > MODULES +=  lang/clang
> > MODCLANG_COMPILER_LINKS = No
> > CONFIGURE_ARGS+=    
> > -DCMAKE_CXX_COMPILER_CLANG_SCAN_DEPS=${LOCALBASE}/bin/clang-scan-deps-${MODCLANG_VERSION}
> 
> This should work for the migration now:
> 
> MODULES +=      lang/clang
> MODCLANG_COMPILER_LINKS = No
> MODCLANG_VERSION =      22
> CONFIGURE_ARGS+=      
> -DCMAKE_CXX_COMPILER_CLANG_SCAN_DEPS=${LOCALBASE}/bin/clang-scan-deps-${MODCLANG_VERSION}
> 
> # XXX remove after base has llvm/22
> BUILD_DEPENDS = devel/llvm/19
> 
> Tested with openal and it fallback to 19 for now and if
> MODCLANG_COMPILER_LINKS = Yes (tested) or llvm22 is in base (untested)
> it should pickup 22.
> 

What is in tree is the above without the CONFIGURE_ARGS magic. This works
because with llvmXX in base, clang-scan-deps-XX is selected and provided
by the relevant devel/llvm/XX port. This has been tested for poppler in
several bulks and I have also tested it for openal. I'm happy to switch
to your approach for poppler and openal once the dust of the compiler
switch has settled.

Reply via email to