Index | Thread | Search

From:
Rafael Sadowski <rafael@sizeofvoid.org>
Subject:
Re: x11/qt6: force to use llvm19
To:
ports@openbsd.org
Date:
Sat, 13 Dec 2025 17:29:39 +0100

Download raw body.

Thread
On Sat Dec 13, 2025 at 03:47:19PM +0100, Kirill A. Korinsky wrote:
> On Sat, 13 Dec 2025 15:32:17 +0100,
> Rafael Sadowski <rafael@sizeofvoid.org> wrote:
> > 
> > @@ -0,0 +1,11 @@
> > +Index: src/qdoc/cmake/QDocConfiguration.cmake
> > +--- src/qdoc/cmake/QDocConfiguration.cmake.orig
> > ++++ src/qdoc/cmake/QDocConfiguration.cmake
> > +@@ -8,6 +8,6 @@ set(QDOC_MINIMUM_CLANG_VERSION "17")
> > + 
> > + # List of explicitly supported Clang versions for QDoc
> > + set(QDOC_SUPPORTED_CLANG_VERSIONS
> > +-    "21.1" "20.1" "19.1" "18.1" "17.0.6"
> > ++    "19.1" "18.1" "17.0.6"
> > + )
> > + 
> > 
> 
> Is it worth to keep only one 19.1 here? At least it will fail when clang is
> increased and will protect from linking with wrong version on bulk bild.
> 

Yes we can do it but there is no longer a <19 (expect llvm16 but the
list starts with 17), and if anything changes here, we will have
to adjust the patch anyway.