qt58-qtbase: Feature 'icu' fails, but it is required

[nilFinx]

Relevant lines in the logs:

:info:configure > In file included from icu.cpp:40:
:info:configure > In file included from /opt/local/include/unicode/utypes.h:39:
:info:configure > /opt/local/include/unicode/uversion.h:167:11: error: nested namespace definition must define each namespace separately
:info:configure > namespace U_HEADER_ONLY_NAMESPACE {}
:info:configure >           ^
:info:configure > /opt/local/include/unicode/uversion.h:165:48: note: expanded from macro 'U_HEADER_ONLY_NAMESPACE'
:info:configure > #define U_HEADER_ONLY_NAMESPACE U_ICU_NAMESPACE::U_HEADER_NESTED_NAMESPACE
:info:configure >                                                ^
:info:configure > In file included from icu.cpp:41:
:info:configure > In file included from /opt/local/include/unicode/ucol.h:17:
:info:configure > In file included from /opt/local/include/unicode/unorm.h:25:
:info:configure > In file included from /opt/local/include/unicode/unorm2.h:35:
:info:configure > /opt/local/include/unicode/uset.h:36:10: fatal error: 'string_view' file not found
:info:configure > #include <string_view>
:info:configure >          ^
:info:configure > 2 errors generated.
:info:configure > make: *** [icu.o] Error 1
:info:configure  => source failed verification.

[ryandesign (Ryan Carsten Schmidt)]

string_view is new in C++17. qt58 only requests a C++11 compiler. Presumably the C++ standard required increased when icu was updated.

[nilFinx]

I've disabled icu by replacing -icu with -no-icu, and commented out two icu related patches - and build worked fine. Probably not a very useful information, but I figured that I should note it.