Merge lp://staging/~thomas-voss/xorg-gtest/cmake into lp://staging/~oif-team/xorg-gtest/trunk
Proposed by
Thomas Voß
Status: | Rejected |
---|---|
Rejected by: | Chase Douglas |
Proposed branch: | lp://staging/~thomas-voss/xorg-gtest/cmake |
Merge into: | lp://staging/~oif-team/xorg-gtest/trunk |
Diff against target: |
209 lines (+195/-0) 3 files modified
CMakeLists.txt (+152/-0) doc/CMakeLists.txt (+25/-0) examples/CMakeLists.txt (+18/-0) |
To merge this branch: | bzr merge lp://staging/~thomas-voss/xorg-gtest/cmake |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Chase Douglas (community) | Needs Fixing | ||
Thomas Voß (community) | Needs Resubmitting | ||
Review via email:
|
Description of the change
Added a cmake setup basically replicating the autotools setup. Adds support for inclusion of xorg-gtest with the help of cmake's add_subdirectory in accordance with Google Test's recommendation of bundling the test framework with the actual framework.
To post a comment you must log in.
Unmerged revisions
- 11. By Thomas Voß
-
Converted CMake directives to lower case and adjusted naming of cached-variable DUMMY_CONF_PATH -> DUMMY_XORG_
CONF_FILE. - 10. By Thomas Voß
-
ADDED: Installation of conf/dummy.conf.
- 9. By Thomas Voß
-
Added a cmake setup to support add_subdirectory scenarios.
Looks fine. Just a few niggles.
I don't like CMake's default style of writing function names in UPPERCASE. It just looks like SHOUTING.
DUMMY_CONF_PATH is not, in my mind, a path since it leads all the way to a file. A path variable to me points to a directory (like the path variable in a shell). I would call this DUMMY_XORG_ CONF_FILE to make it more self-documenting.