tech-pkg archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: icu update to 77.1
Thomas Klausner <wiz%NetBSD.org@localhost> writes:
> On Fri, Apr 04, 2025 at 07:28:57AM +0200, Martin Husemann wrote:
>> On Thu, Apr 03, 2025 at 09:44:33PM +0200, Thomas Klausner wrote:
>> > The previous version was already marked as needing C++17, so no
>> > change.
>>
>> Yes, but the compile errors you quoted come from g++ invocations using
>> older standards.
>>
>> And one *could* make a distinction between the library needing C++$something
>> to build and the API in the public headers requiring C++$somethingelse.
>
> I tried two things:
>
> - Setting USE_CXX_FEATURES+=c++17 in icu/buildlink3.mk - with the same
> bulk build result, since qt5-qtwebkit uses -std=c++11
>
> - Setting USE_CXX_FEATURES+=c++17 and FORCE_CXX_STD=c++17 directly in
> qt5-qtwebkit, which lead to it failing differently.
So that means
icu (newly?) requires --std=c++17 (perhaps implicitly) to process
headers?
qt5-qtwebkit requires -std=c++11 to build
and
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?
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)?
Home |
Main Index |
Thread Index |
Old Index