On 03/04/2014 10:07 PM, Daniele E. Domenichelli wrote:
> Follow up to this thread:
>   http://thread.gmane.org/gmane.comp.programming.tools.cmake.devel/8369
> 
> Please review the topic FindPkgConfig_Extend-PKG_CONFIG_PATH.

Nice!  Please revise the documentation to use proper cross-reference
syntax to link to other variables instead of just inline literals:

 ``SOME_VARIABLE`` => :variable:`SOME_VARIABLE`

Also it looks like some logic is taken from GNUInstallDirs.  Is there
enough in common to try to factor that out into a helper module?

Thanks,
-Brad

-- 

Powered by www.kitware.com

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers

Reply via email to