libdbusmenu-qt

libdbusmenu-qt fails to find -lQtCore

Bug #646381 reported by Peter Levine
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
libdbusmenu-qt
Fix Released
Medium
Aurélien Gâteau

Bug Description

The QJSON_LIBRARIES variable in tools/testapp/CMakeLists.txt adds "-lQtCore" from pkgconfig, but omits the
preceeding "-L/usr/lib/qt4" path, which causes the build to look in "/usr/lib"
only. This patch changes it to use the QJSON_LDFLAGS variable, which includes
both.

Also reported downstream at http://bugs.gentoo.org/show_bug.cgi?id=317389

Revision history for this message
Peter Levine (plevine457) wrote :
Revision history for this message
Aurélien Gâteau (agateau) wrote :

I just fixed it in -r186, but in a different way: the value of $QJSON_LIBRARIES was coming from pkg_check_module() instead of coming from find_package_handle_standard_args() so I prefixed the pkg_check_module() variable with "PC_".

Changed in libdbusmenu-qt:
assignee: nobody → Aurélien Gâteau (agateau)
importance: Undecided → Medium
status: New → Fix Committed
Revision history for this message
Corentin Chary (iksaif) wrote :

Using your fix, you have to edit the original FindQJSON.cmake.
Another solution is to add

A cleaner fix seems to add

link_directories(
  ${QJSON_LIBRARY_DIRS}
)

on top of

add_executable(dbusmenubench-qtapp ${qtapp_SRCS})

Aurélien Gâteau (agateau)
Changed in libdbusmenu-qt:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.