[Bug 273753] [exp-run] Against llvm-17-update branch on GitHub
- In reply to: bugzilla-noreply_a_freebsd.org: "[Bug 273753] [exp-run] Against llvm-17-update branch on GitHub"
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Sat, 04 Nov 2023 16:27:37 UTC
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=273753 --- Comment #56 from commit-hook@FreeBSD.org --- A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=3cb46a4465c6319ff9fcf7ac6d151ac884a515ee commit 3cb46a4465c6319ff9fcf7ac6d151ac884a515ee Author: Dimitry Andric <dim@FreeBSD.org> AuthorDate: 2023-11-04 15:51:40 +0000 Commit: Dimitry Andric <dim@FreeBSD.org> CommitDate: 2023-11-04 16:26:59 +0000 qt(-dist)?.mk: suppress linker version script errors with lld >= 17 (take 2) This amends commit 3e5360df35848a1ad8c5edae83a0a9cdfd6bb775: Qt uses generated linker version scripts which always have a qt_version_tag symbol, but that symbol is only defined in the main Qt shared library. For other Qt components, this leads to lld >= 17 erroring out due to the symbol being undefined. Suppress these errors from qt-dist.mk, so we don't have to patch each individual port. This affects at least: * devel/qt5-dbus * net/qt5-network * databases/qt5-sql which were the first ones to fall over with a clang+lld 17 poudriere bulk run. For reference, the error you would see looks like: ... c++ -Wl,--as-needed -fstack-protector-strong -Wl,--no-undefined -Wl,--version-script,QtDBus.version -pthread -Wl,-rpath,/usr/local/lib/qt5 -shared -Wl,-Bsymbolic-functions -Wl,-soname,libQt5DBus.so.5 -o libQt5DBus.so.5.15.8 .obj/qdbusconnection.o .obj/qdbusconnectioninterface.o .obj/qdbuserror.o .obj/qdbusintegrator.o .obj/qdbusmessage.o .obj/qdbusserver.o .obj/qdbusabstractinterface.o .obj/qdbusinterface.o .obj/qdbusxmlparser.o .obj/qdbusutil.o .obj/qdbusintrospection.o .obj/qdbusabstractadaptor.o .obj/qdbusinternalfilters.o .obj/qdbusmetaobject.o .obj/qdbusxmlgenerator.o .obj/qdbusmisc.o .obj/qdbusargument.o .obj/qdbusreply.o .obj/qdbusmetatype.o .obj/qdbusextratypes.o .obj/qdbuscontext.o .obj/qdbuspendingcall.o .obj/qdbuspendingreply.o .obj/qdbus_symbols.o .obj/qdbusservicewatcher.o .obj/qdbusunixfiledescriptor.o .obj/qdbusvirtualobject.o .obj/moc_qdbusconnectioninterface.o .obj/moc_qdbuserror.o .obj/moc_qdbusconnection.o .obj/moc_qdbusabstrac tadaptor.o .obj/moc_qdbusvirtualobject.o .obj/moc_qdbusconnection_p.o .obj/moc_qdbusconnectionmanager_p.o .obj/moc_qdbuspendingcall_p.o -L/wrkdirs/usr/ports/devel/qt5-dbus/work/kde-qtbase-5.15.8p157/lib /usr/local/lib/qt5/libQt5Core.so -L/usr/local/lib -ldbus-1 ld: error: version script assignment of 'Qt_5.15' to symbol 'qt_version_tag' failed: symbol not defined c++: error: linker command failed with exit code 1 (use -v to see invocation) Some qt ports, such as www/qt5-webkit, do not use qt-dist.mk but qt.mk, and qt-dist.mk includes qt.mk, so move the workaround that sets LDFLAGS to qt.mk instead. This should fix the problem for all qt ports. PR: 273992, 273753 Fixes: 3e5360df3584 MFH: 2023Q4 Mk/Uses/qt-dist.mk | 6 ------ Mk/Uses/qt.mk | 6 ++++++ 2 files changed, 6 insertions(+), 6 deletions(-) -- You are receiving this mail because: You are on the CC list for the bug.