Merge branch 'patch-1' into 'master'
Fixed typo See merge request CLIUtils/modern-cmake!36
This commit is contained in:
commit
b0ed056e3f
@ -19,7 +19,7 @@ The next two lists are heavily based on the excellent gist [Effective Modern CMa
|
|||||||
* **Write a Config.cmake file**: This is what a library author should do to support clients.
|
* **Write a Config.cmake file**: This is what a library author should do to support clients.
|
||||||
* **Make ALIAS targets to keep usage consistent**: Using `add_subdirectory` and `find_package` should provide the same targets and namespaces.
|
* **Make ALIAS targets to keep usage consistent**: Using `add_subdirectory` and `find_package` should provide the same targets and namespaces.
|
||||||
* **Combine common functionality into clearly documented functions or macros**: Functions are better usually.
|
* **Combine common functionality into clearly documented functions or macros**: Functions are better usually.
|
||||||
* **Use lowercase function names**: CMake functions and macros can be called lower or upper case. Always user lower case. Upper case is for variables.
|
* **Use lowercase function names**: CMake functions and macros can be called lower or upper case. Always use lower case. Upper case is for variables.
|
||||||
* **Use `cmake_policy` and/or range of versions**: Policies change for a reason. Only piecemeal set OLD policies if you have to.
|
* **Use `cmake_policy` and/or range of versions**: Policies change for a reason. Only piecemeal set OLD policies if you have to.
|
||||||
|
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user