Building Brewtarget Still Problematic on Slackware

Bug #1109493 reported by mike_622 on 2011-11-05
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Brewtarget
High
Philip

Bug Description

Building Brewtarget on Slackware64 13.37 still has a few issues due to Phonon being packaged separately from Qt. Most of the difficulty from 1.2.3 has been alleviated, but a few small issues remain.
The suggestion in the cmake output to use -DCMAKE_PREFIX_PATH and set it to the path of Phonon allows the build process to find the Phonon headers and libraries. However, linking still fails because the build process is not linking against the Phonon libraries.
I addressed this in a previous patch I posted on the Homebrewtalk forums, but it seems to have been overlooked. The TARGET_LINK_LIBRARIES statement in src/CMakeLists.txt is formatted incorrectly. The attached patch fixes it (at least for me).

mike_622 (mike-622) wrote :

The file phonon_fix.patch was added: Address typos in src/CMakeLists.txt

Philip (rocketman768) wrote :

Hm, I don't know why putting a semicolon between the two variables would fix the problem since Cmake automatically inserts semicolons if you simply put a space between the variables, but I'll check it out.

Philip (rocketman768) wrote :

Will check this issue near 2.0 release.

Philip (rocketman768) on 2013-01-30
Changed in brewtarget:
importance: Low → High
status: New → Triaged
assignee: Philip (rocketman768) → nobody
Philip (rocketman768) on 2013-03-06
Changed in brewtarget:
milestone: none → 2.0.2
Philip (rocketman768) wrote :

Should be fixed in commit 8be034f.

Changed in brewtarget:
status: Triaged → Fix Committed
assignee: nobody → Philip (rocketman768)
Philip (rocketman768) on 2014-02-02
Changed in brewtarget:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers