Summary: | dev-libs/libdbusmenu-qt-0.6.3 from kde overlay doesn't compile | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Yörn Jünger <greggel> |
Component: | [OLD] Library | Assignee: | Gentoo KDE team <kde> |
Status: | RESOLVED FIXED | ||
Severity: | normal | ||
Priority: | High | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | https://bugs.launchpad.net/libdbusmenu-qt/+bug/646381 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
ebuild patch to libdbusmenu-qt-0.6.3-r1
patch to fix the source |
Description
Yörn Jünger
2010-09-26 21:54:07 UTC
Created attachment 248711 [details, diff]
ebuild patch to libdbusmenu-qt-0.6.3-r1
Created attachment 248713 [details, diff] patch to fix the source Patch taken from https://bugs.launchpad.net/libdbusmenu-qt/+bug/646381 The patch looks invalid... LDFLAGS is not for passing libraries, the order matters for LDFLAGS like -Wl,--as-needed. It's a workaround at most. Above patch looks more proper, cmake has ways to separate -L flags from -l flags when calling pkg-config, _LIBRARY_DIRS. Perhaps http://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/dev-libs/libdbusmenu-qt/files/libdbusmenu-qt-0.3.2-cmake.patch?hideattic=0&view=markup was dropped too soon? Yeah, this is a lazy peaople patch. Separating is always better. The patch works and lives in my local portage overlay till there is a "better" patch. I don't know that much about cmake ;) Here is another solution + link_directories( + ${QJSON_LIBRARY_DIRS} + ) add_executable(dbusmenubench-qtapp ${qtapp_SRCS}) should be fixed in 0.6.4, please reopen if it persists |