[cmake-developers] [PATCH] add debug glew to FindGLEW
Farbos a
farbos.arnaud at gmail.com
Mon Jun 20 10:52:43 EDT 2016
Brad,
Here is the patch fixed.
Thanks,
2016-06-20 16:17 GMT+02:00 Brad King <brad.king at kitware.com>:
> On 06/20/2016 07:55 AM, Farbos a wrote:
>> I added the Glew debug library to the FindGLEW.cmake, here is the patch.
>
> Good start. Here are some comments:
>
> * For compatibility with scripts that set GLEW_LIBRARY we need to
> honor the old name if it is set. See FindZLIB for an example.
> (Near the "Allow ZLIB_LIBRARY to be set manually" comment.)
>
> * The imported target needs to have each configuration added only
> if it exists. See FindZLIB for an example. IMPORTED_CONFIGURATIONS
> needs to list the available configurations.
>
> Thanks,
> -Brad
>
--
Arnaud FARBOS
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-add-debug-glew-to-FindGLEW.patch
Type: application/octet-stream
Size: 2544 bytes
Desc: not available
URL: <http://public.kitware.com/pipermail/cmake-developers/attachments/20160620/f37e7485/attachment.obj>
More information about the cmake-developers
mailing list