Merge lp://staging/~smspillaz/compiz-core/compiz-core.fix_964248 into lp://staging/compiz-core

Proposed by Sam Spilsbury
Status: Merged
Approved by: Alan Griffiths
Approved revision: 3064
Merged at revision: 3071
Proposed branch: lp://staging/~smspillaz/compiz-core/compiz-core.fix_964248
Merge into: lp://staging/compiz-core
Diff against target: 33 lines (+24/-0)
1 file modified
CMakeLists.txt (+24/-0)
To merge this branch: bzr merge lp://staging/~smspillaz/compiz-core/compiz-core.fix_964248
Reviewer Review Type Date Requested Status
Alan Griffiths Approve
Daniel van Vugt Approve
Review via email: mp+99193@code.staging.launchpad.net

Description of the change

Fix LP#964248 - Build GTest locally

== Problem ==

See LP#964248 - If libgtest isn't installed then FindGTest will fail, but Ubuntu doesn't ship the gtest library anymore so we need to compile it locally

== Solution ==

Added CMake commands to compile gtest locally, link to that

== Test ==

make

To post a comment you must log in.
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

Works OK, though possibly needs some more indentation inside if/endif.

review: Approve
Revision history for this message
Alan Griffiths (alan-griffiths) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches