Re: CVS commit: pkgsrc/x11/qt5-qtbase

[nia <nia%NetBSD.org@localhost>]

On Thu, May 30, 2024 at 11:17:52PM +0900, Ryo ONODERA wrote:
> Hi,
> 
> Greg Troxel <gdt%lexort.com@localhost> writes:
> 
> > If icu requires gnu++17 then presumably it probably should be setting
> > that in bl3 (plus it's broken but I know that isn't going to change :-)
> > rather than setting FORCE_CXX_STD in using packages.
> 
> Some direct consumers of textproc/icu, for example www/firefox and
> misc/libreoffice, dislike FORCE_CXX_STD because they use multiple -std=
> options in one package.
> 
> Defining FORCE_CXX_STD in textproc/icu/buildlink3.mk will break
> their builds.

Yes, FORCE_CXX_STD should never be used in bl3.
The purpose is really to set a maximum standard versions
which doesn't work for every dependency.