[cmake-developers] Topic "add-opencl-imported-target" good to merge?
Matthäus G. Chajdas
cmake at anteru.net
Wed Jun 1 15:00:45 EDT 2016
Hi Brad,
done - I had to squash and force push once more, because I used the
wrong author in the first commit.
Everything is now in one commit here:
https://cmake.org/gitweb?p=stage/cmake.git;a=commitdiff;h=e95b62110715c06fb76b57fdfb13ea493a94c0c4
Thanks for the timely feedback!
Cheers,
Matthäus
Am 01.06.2016 um 20:22 schrieb Brad King:
> On 06/01/2016 02:15 PM, Matthäus G. Chajdas wrote:
>> Hopefully done
>
> The revised history looks good. The change itself looks good.
>
> Please also add a `Help/release/dev/FindOpenCL-imported-target.rst`
> file with a release note for the feature. Look at other files
> in that directory for a sample. See Help/release/*.rst for other
> examples.
>
> As part of the modernization of find modules we're also trying
> to add better testing for them. Please see Tests/FindPNG and the
> CMake_TEST_FindPNG code path in Tests/CMakeLists.txt and construct
> a similar test for the FindOpenCL module.
>
> Thanks,
> -Brad
>
More information about the cmake-developers
mailing list