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.
No comments:
Post a Comment