Summary: | dev-qt/qtwebengine-5.7.0 with dev-libs/icu-58: error: static assertion failed: breakAllLineBreakClassTable should be consistent | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Lars Wendler (Polynomial-C) (RETIRED) <polynomial-c> |
Component: | Current packages | Assignee: | Qt Bug Alias <qt> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | ab4bd, ao, bjh-gentoobt, creffett, david.narvaez, dschridde+gentoobugs, eugene.shalygin, iivanich, josef64, kuba.iluvatar, mpagano, oleg.hoefling, panard, paolo.pedroni, till2.schaefer, ua_gentoo_bugzilla |
Priority: | Normal | Keywords: | PATCH |
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
See Also: |
https://bugs.gentoo.org/show_bug.cgi?id=594494 https://bugs.gentoo.org/show_bug.cgi?id=589814 |
||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 586612 | ||
Attachments: |
build.log.xz
Fix compilation with icu 58.1 |
Description
Lars Wendler (Polynomial-C) (RETIRED)
2016-11-07 00:29:36 UTC
Created attachment 452556 [details]
build.log.xz
Created attachment 452782 [details, diff] Fix compilation with icu 58.1 Based on https://chromium.googlesource.com/chromium/src/+/e60b571faa3f14dd9119a6792dccf12f8bf80192%5E%21/#F0 I suspect that the fix in chromium will not make its way into Qt until at least 5.9, so I think we have no option but to carry this patch for now. (In reply to Michael Palimaka (kensington) from comment #3) > I suspect that the fix in chromium will not make its way into Qt until at > least 5.9, so I think we have no option but to carry this patch for now. They might backport it to 5.8.0 if they're aware of the problem. In any case, we should apply it to all affected in-tree versions. Btw, the upstream commit msg says "Prepare to upgrade ICU to 58 part 2"... is there a part 1? Part 1 seems to be: https://chromium.googlesource.com/chromium/src/+/424584b4984bde7c831f42e9fb47f1ad583a1c46%5E%21/ Chromium bug about update of ICU: https://bugs.chromium.org/p/chromium/issues/detail?id=637001 ICU 58.1 will probably go stable soon. Not blocking the sec bugs here since 5.7.0 is masked, but I've added them as "See Also". Will the attached patch be included in Gentoo? (In reply to Dennis Schridde from comment #8) > Will the attached patch be included in Gentoo? There is a pull request for this and some other issues that is currently under review: https://github.com/gentoo/gentoo/pull/2844 pesa just gave you (kensington) the go to merge. Please, let me get rid of using --keep-going :-). Thanks, fixed in git by Aric Belsito. https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=ed080e3f5696fcdc822ad60e95649176195bea9e |