Merge pull request #204 from FeignClaims/fix/find_dependencies_macro

fix: `target_find_dependencies` should be a macro

Author: aminya

docs/src/target_find_dependencies.md

@@ -11,7 +11,7 @@ target_find_dependencies(<target_name>
 )
 ```
 
-This function calls `find_package(${dependency} [CONFIG] REQUIRED)` for all dependencies required and binds them to the target.
+This macro calls `find_package(${dependency} [CONFIG] REQUIRED)` for all dependencies required and binds them to the target.
 
 Properties named `PROJECT_OPTIONS_<PRIVATE|PUBLIC|INTERFACE>[_CONFIG]_DEPENDENCIES` will be created in `target_name` to represent corresponding dependencies.
 When adding the target to `package_project`, directories in this property will be automatically added.

src/PackageProject.cmake

@@ -325,7 +325,7 @@ endfunction()
    :parser: myst_parser.sphinx_
 
 #]]
-function(target_find_dependencies target)
+macro(target_find_dependencies target)
   set(_options)
   set(_oneValueArgs)
   set(_MultiValueArgs
@@ -357,10 +357,10 @@ function(target_find_dependencies target)
     set_or_append_target_property(${target} "PROJECT_OPTIONS_${property}_DEPENDENCIES" "${args_${property}}")
   endmacro()
 
-  _property_for(PRIVATE)
-  _property_for(PUBLIC)
-  _property_for(INTERFACE)
-  _property_for(PRIVATE_CONFIG)
-  _property_for(PUBLIC_CONFIG)
-  _property_for(INTERFACE_CONFIG)
-endfunction()
+  _Property_for(PRIVATE)
+  _Property_for(PUBLIC)
+  _Property_for(INTERFACE)
+  _Property_for(PRIVATE_CONFIG)
+  _Property_for(PUBLIC_CONFIG)
+  _Property_for(INTERFACE_CONFIG)
+endmacro()