dev-qt/qtwebengine-5.7.1-r1:5[system-icu] fails to build with ICU >=59. [1139/11505] CXX obj/src/3rdparty/chromium/third_party/sfntly/src/cpp/src/sample/chromium/sfntly.subsetter_impl.o FAILED: obj/src/3rdparty/chromium/third_party/sfntly/src/cpp/src/sample/chromium/sfntly.subsetter_impl.o /usr/bin/x86_64-pc-linux-gnu-g++ -MMD -MF obj/src/3rdparty/chromium/third_party/sfntly/src/cpp/src/sample/chromium/sfntly.subsetter_impl.o.d -DV8_DEPRECATION_WARNINGS -DCLD_VERSION=2 -DTOOLKIT_QT -D_FILE_OFFSET_BITS=64 -DNO_TCMALLOC -DCHROMIUM_BUILD -DUI_COMPOSITOR_IMAGE_TRANSPORT -DUSE_AURA=1 -DUSE_DEFAULT_RENDER_THEME=1 -DUSE_LIBJPEG_TURBO=1 -DUSE_X11=1 -DENABLE_ONE_CLICK_SIGNIN -DENABLE_WEBRTC=1 -DENABLE_MEDIA_ROUTER=1 -DUSE_PROPRIETARY_CODECS -DENABLE_PEPPER_CDMS -DENABLE_CONFIGURATION_POLICY -DENABLE_NOTIFICATIONS -DENABLE_HIDPI=1 -DENABLE_TOPCHROME_MD=1 -DUSE_UDEV -DDONT_EMBED_BUILD_METADATA -DFIELDTRIAL_TESTING_ENABLED -DENABLE_TASK_MANAGER=1 -DENABLE_PDF=1 -DENABLE_PLUGINS=1 -DENABLE_SESSION_SERVICE=1 -DENABLE_THEMES=1 -DENABLE_AUTOFILL_DIALOG=1 -DENABLE_BACKGROUND=1 -DENABLE_PRINTING=1 -DENABLE_BASIC_PRINTING=1 -DENABLE_CAPTIVE_PORTAL_DETECTION=1 -DENABLE_APP_LIST=1 -DENABLE_SETTINGS_APP=1 -DENABLE_SUPERVISED_USERS=1 -DENABLE_MDNS=1 -DENABLE_SERVICE_DISCOVERY=1 -DFULL_SAFE_BROWSING -DSAFE_BROWSING_CSD -DSAFE_BROWSING_DB_LOCAL -DSFNTLY_NO_EXCEPTION -DU_USING_ICU_NAMESPACE=0 -DU_ENABLE_DYLOAD=0 -DUSE_LIBPCI=1 -DUSE_OPENSSL=1 -DUSE_GLIB=1 -DUSE_NSS_CERTS=1 -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -DNDEBUG -DNVALGRIND -DDYNAMIC_ANNOTATIONS_ENABLED=0 '-DQTWEBENGINEPROCESS_NAME="QtWebEngineProcess"' -Igen/shim_headers/icuuc/target -Igen -I/var/tmp/portage/dev-qt/qtwebengine-5.7.1-r1/work/qtwebengine-opensource-src-5.7.1/src/3rdparty/chromium/third_party/sfntly/src/cpp/src -fstack-protector --param=ssp-buffer-size=4 -pthread -fno-strict-aliasing -Wall -Wno-extra -Wno-unused-parameter -Wno-missing-field-initializers -fvisibility=hidden -pipe -fPIC -Wno-unused-local-typedefs -m64 -march=x86-64 -O2 -fno-ident -fdata-sections -ffunction-sections -funwind-tables -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 -march=native -O2 -fno-ident -frecord-gcc-switches -pipe -Wall -fno-exceptions -fno-rtti -fno-threadsafe-statics -fvisibility-inlines-hidden -fno-delete-null-pointer-checks -std=gnu++11 -Wno-narrowing -c /var/tmp/portage/dev-qt/qtwebengine-5.7.1-r1/work/qtwebengine-opensource-src-5.7.1/src/3rdparty/chromium/third_party/sfntly/src/cpp/src/sample/chromium/subsetter_impl.cc -o obj/src/3rdparty/chromium/third_party/sfntly/src/cpp/src/sample/chromium/sfntly.subsetter_impl.o /var/tmp/portage/dev-qt/qtwebengine-5.7.1-r1/work/qtwebengine-opensource-src-5.7.1/src/3rdparty/chromium/third_party/sfntly/src/cpp/src/sample/chromium/subsetter_impl.cc:51:38: error: ‘UnicodeString’ has not been declared void ConstructName(UChar* name_part, UnicodeString* name, int32_t name_id) { ^~~~~~~~~~~~~ /var/tmp/portage/dev-qt/qtwebengine-5.7.1-r1/work/qtwebengine-opensource-src-5.7.1/src/3rdparty/chromium/third_party/sfntly/src/cpp/src/sample/chromium/subsetter_impl.cc: In function ‘void {anonymous}::ConstructName(UChar*, int*, int32_t)’: /var/tmp/portage/dev-qt/qtwebengine-5.7.1-r1/work/qtwebengine-opensource-src-5.7.1/src/3rdparty/chromium/third_party/sfntly/src/cpp/src/sample/chromium/subsetter_impl.cc:54:15: error: invalid conversion from ‘UChar* {aka char16_t*}’ to ‘int’ [-fpermissive] *name = name_part; ^~~~~~~~~ ...
Created attachment 472782 [details] qtwebengine-5.7.1-r1.log
Created attachment 472884 [details, diff] qtwebengine-5.7.1-icu-59.patch This patch is incompatible with versions of ICU not using char16_t as UChar, so it should be applied conditionally.
Created attachment 475690 [details, diff] qtwebengine-5.7.1-icu-59.patch
Created attachment 475692 [details, diff] qtwebengine-5.9.0-icu-59.patch
Upstream fix: https://code.qt.io/cgit/qt/qtwebengine-chromium.git/commit/?id=363741cffb565dd6cfc2f95054c25a434b101dba https://code.qt.io/cgit/qt/qtwebengine.git/commit/?id=1eed5a436c4e9754d922f5ff6e4e4791a3fec27c
dup ?
Created attachment 477998 [details] emerge-info.txt
Created attachment 478000 [details] dev-qt:qtwebengine-5.7.1-r1:20170625-192626.log
Created attachment 478002 [details] emerge-history.txt
Created attachment 478004 [details] environment
Created attachment 478006 [details] etc.portage.tbz2
Created attachment 478008 [details] logs.tbz2
Created attachment 478010 [details] temp.tbz2
*** Bug 622732 has been marked as a duplicate of this bug. ***
Since the already fixed >=dev-qt/qtwebengine-5.9.1 is in the tree, I suggest that =dev-qt/qtwebengine-5.7* be changed to have the following dependency: system-icu? ( <dev-libs/icu-59:= )
But this is annoying since chromium (stable) require >=icu-59 to build with "system-icu".
Well this is annoying too, because now qtwebengine with system-icu fails to compile. If you want chromium-xx/firefox-57/thunderbird-yy/seamonkey-zz with system-icu, you either need to upgrade to qtwebengine-5.9.3 with system-icu, or just use -system-icu on qtwebengine-5.7.1. Either way emerge should not allow qtwebengine with system-icu when there's an incompatible version of icu installed. Because that will just fail anyway, and leave the user to wonder why. (and probably waste some time trying to determine why until he finds this bug report)
The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=c3f06fdd6dc71c5f125f9da80c5f4fcd7e53839f commit c3f06fdd6dc71c5f125f9da80c5f4fcd7e53839f Author: Andreas Sturmlechner <asturm@gentoo.org> AuthorDate: 2018-01-30 23:30:39 +0000 Commit: Andreas Sturmlechner <asturm@gentoo.org> CommitDate: 2018-01-31 00:45:46 +0000 dev-qt/qtwebengine: 5.7.1-r2: Switch off USE=system-icu, fix dep Fix build and ease dependency resolution with a future stable icu-60.2. Bug: https://bugs.gentoo.org/618648 Package-Manager: Portage-2.3.21, Repoman-2.3.6 dev-qt/qtwebengine/qtwebengine-5.7.1-r2.ebuild | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)}
Workaround applied to 5.7.1-r2, thanks.
Well, I never experienced this problem and now the build fails with USE=-system-icu. I'll file a bug later
Please disregard my previous comment.
