Merge lp://qastaging/~compiz-team/compiz/compiz.fix_parsearguments_not_installed into lp://qastaging/compiz/0.9.8
Proposed by
Sam Spilsbury
| Status: | Merged |
|---|---|
| Approved by: | Daniel van Vugt |
| Approved revision: | 3300 |
| Merged at revision: | 3300 |
| Proposed branch: | lp://qastaging/~compiz-team/compiz/compiz.fix_parsearguments_not_installed |
| Merge into: | lp://qastaging/compiz/0.9.8 |
| Diff against target: |
141 lines (+54/-55) 3 files modified
cmake/CompizCommon.cmake (+53/-2) cmake/EnableCoverageReport.cmake (+1/-1) cmake/ParseArguments.cmake (+0/-52) |
| To merge this branch: | bzr merge lp://qastaging/~compiz-team/compiz/compiz.fix_parsearguments_not_installed |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Daniel van Vugt | Approve | ||
|
Review via email:
|
|||
Commit message
ParseArguments isn't installed, having it around is misleading, move its functionality to CompizCommon which is installed so that we don't cause cascading build failures everywhere else ...
Description of the change
ParseArguments isn't installed, having it around is misleading, move its functionality to CompizCommon which is installed so that we don't cause cascading build failures everywhere else ...
To post a comment you must log in.
Revision history for this message
| Daniel van Vugt (vanvugt) | # |
review:
Approve
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
