aboutsummaryrefslogtreecommitdiffstats
path: root/deskutils/nextcloudclient
diff options
context:
space:
mode:
authortijl <tijl@FreeBSD.org>2019-01-16 19:13:44 +0800
committertijl <tijl@FreeBSD.org>2019-01-16 19:13:44 +0800
commit0c123abc65c17a023a06fa03c2da7624f5a40a03 (patch)
tree127ae6686665fd0ff97a7bd5f27449299929734e /deskutils/nextcloudclient
parente64e67d7f17dd66dc29ae5c5ea52dd16964f47bd (diff)
downloadfreebsd-ports-gnome-0c123abc65c17a023a06fa03c2da7624f5a40a03.tar.gz
freebsd-ports-gnome-0c123abc65c17a023a06fa03c2da7624f5a40a03.tar.zst
freebsd-ports-gnome-0c123abc65c17a023a06fa03c2da7624f5a40a03.zip
Fix Qt5 symbol version scripts to put the catch-all clause first. When
a symbol matches multiple clauses the last one takes precedence. If the catch-all is last it captures everything. In the case of Qt5 libraries this caused all symbols to have a Qt_5 label while some should have Qt_5_PRIVATE_API. This only affects lld because GNU ld always gives the catch-all lowest priority. Older versions of Qt5Webengine exported some memory allocation symbols from the bundled Chromium. Version 5.9 stopped exporting these [1] but the symbols were kept as weak wrappers for the standard allocation functions to maintain binary compatibility. [2][3] The problem is that the call to the standard function in these weak wrappers is only resolved to the standard function if there's a call to this standard function in other parts of Qt5Webengine, because only then is there a non-weak symbol that takes precedence over the weak one. If there's no such non-weak symbol the call in the weak wrapper resolves to the weak wrapper itself creating an infinite call loop that overflows the stack and causes a crash. Some of the allocation functions are variants of C++ new and delete and it probably depends on the compiler whether these variants are used in other parts of Qt5Webengine. Remove the weak wrappers (make them Linux specific). This isn't binary compatible but we are already breaking that with the changes to the symbol versions. [1] https://github.com/qt/qtwebengine/commit/5c2cbfccf9aafb547b0b30914c4056abd25942a4 [2] https://github.com/qt/qtwebengine/commit/2ed5054e3a800fa97c2c9e920ba1e6ea4b6ef2a5 [3] https://github.com/qt/qtwebengine/commit/009f5ebb4bd6e50188671e0815a5dae6afe39db5 Bump all ports that depend on Qt5. PR: 234070 Exp-run by: antoine Approved by: kde (adridg)
Diffstat (limited to 'deskutils/nextcloudclient')
-rw-r--r--deskutils/nextcloudclient/Makefile1
1 files changed, 1 insertions, 0 deletions
diff --git a/deskutils/nextcloudclient/Makefile b/deskutils/nextcloudclient/Makefile
index 96fdd7e1f600..070e171d75b1 100644
--- a/deskutils/nextcloudclient/Makefile
+++ b/deskutils/nextcloudclient/Makefile
@@ -3,6 +3,7 @@
PORTNAME= nextcloudclient
PORTVERSION= 0.0.0.20181226
+PORTREVISION= 1
#PORTREVISION= 2
CATEGORIES= deskutils