Re: icu update to 77.1

[Thomas Klausner <wiz%NetBSD.org@localhost>]

On Fri, Apr 04, 2025 at 09:03:00AM -0400, Greg Troxel wrote:
> So that means
> 
>   icu (newly?) requires --std=c++17 (perhaps implicitly) to process
>   headers?

icu already required c++17.
qt5-qtwebkit has a workaround in its Makefile:

# XXX Workaround for icu-76 whose header files require C++17 unless you
# define U_HIDE_DRAFT_API.  pkg/59028: qt5-qtwebkit doesn't build
CXXFLAGS+=              -DU_HIDE_DRAFT_API

that doesn't seem to be enough any longer with icu 77.

(I also tried turning off its new header-only APIs with
CXXFLAGS+=              -DU_SHOW_CPLUSPLUS_API=0
but that didn't fix qt5-qtwebkit.)

>   qt5-qtwebkit requires -std=c++11 to build

Looks that way.

>   nobody is going to think it reasonable to fix icu to permit using the
>   headers with only c++11 (even though as a library they should!)
> 
>   nobody is doing any kind of maintenance on qt5-qtwebkit
> 
> Is that how it is?

Yes.

> What if in icu bl3, instead of USE_CXX_FEATURES+=c++17, it is c++11?
> Or does icu document and truly require c++17 for header use (martin's
> 2nd question)?

Since icu documents it requires C++17 I'm not sure what this test is
useful for. icu/buildlink3.mk does not require any c++ standard right
now, just gcc 4.9+.

Btw, here is the changelog:
https://unicode-org.github.io/icu/download/77.html
 Thomas