Merge lp:~aacid/compiz/kde_needs_kdeworkspace into lp:compiz/0.9.8

Proposed by Albert Astals Cid on 2012-09-07
Status: Merged
Approved by: Sam Spilsbury on 2012-09-08
Approved revision: 3346
Merged at revision: 3355
Proposed branch: lp:~aacid/compiz/kde_needs_kdeworkspace
Merge into: lp:compiz/0.9.8
Diff against target: 25 lines (+4/-3)
1 file modified
kde/window-decorator-kde4/CMakeLists.txt (+4/-3)
To merge this branch: bzr merge lp:~aacid/compiz/kde_needs_kdeworkspace
Reviewer Review Type Date Requested Status
Sam Spilsbury 2012-09-07 Approve on 2012-09-08
jenkins (community) continuous-integration Approve on 2012-09-07
Review via email: mp+123278@code.launchpad.net

Commit Message

Also need kdeworkspace since kdecorationbridge.h is there

Description of the Change

Also need kdeworkspace since kdecorationbridge.h is there as you can see in http://packages.ubuntu.com/quantal/amd64/kde-workspace-dev/filelist

To post a comment you must log in.
jenkins (martin-mrazik+qa) wrote :
review: Needs Fixing (continuous-integration)
jenkins (martin-mrazik+qa) wrote :
review: Approve (continuous-integration)
Sam Spilsbury (smspillaz) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'kde/window-decorator-kde4/CMakeLists.txt'
2--- kde/window-decorator-kde4/CMakeLists.txt 2012-09-03 07:19:30 +0000
3+++ kde/window-decorator-kde4/CMakeLists.txt 2012-09-07 13:52:26 +0000
4@@ -7,8 +7,9 @@
5 if (USE_KDE4)
6
7 find_package(KDE4 4.3.0)
8+ find_package(KDE4Workspace)
9
10- if (KDE4_FOUND)
11+ if (KDE4_FOUND AND KDE4Workspace_FOUND)
12 include(KDE4Defaults)
13 include(FindPkgConfig)
14 include(MacroLibrary)
15@@ -71,8 +72,8 @@
16 DESTINATION ${exec_prefix}
17 )
18
19- else (KDE4_FOUND)
20+ else (KDE4_FOUND AND KDE4Workspace_FOUND)
21 compiz_set (USE_KDE4 0)
22- endif (KDE4_FOUND)
23+ endif (KDE4_FOUND AND KDE4Workspace_FOUND)
24
25 endif (USE_KDE4)

Subscribers

People subscribed via source and target branches