[cmake-developers] Topic "add-opencl-imported-target" good to merge?

2016-05-31 Thread Matthäus G . Chajdas
Hi, I've just pushed "add-opencl-imported-target" which adds an imported target to FindOpenCL. The whole change is rather small: https://cmake.org/gitweb?p=stage/cmake.git;a=commitdiff;h=6c53137a19e482140db3dc97b626af38348f2c71 Good to merge this to next for testing? Cheers, Matthäus -- Pow

Re: [cmake-developers] Topic "add-opencl-imported-target" good to merge?

2016-06-01 Thread Matthäus G . Chajdas
Hi Brad, updated to latest master and pushed again (I merged latest master into this - is that fine or does it have to be a rebase? In that case I'll redo.) Cheers, Matthäus Am 01.06.2016 um 17:18 schrieb Brad King: > On 05/31/2016 03:17 PM, Matthäus G. Chajdas wrote: >> I'

Re: [cmake-developers] Topic "add-opencl-imported-target" good to merge?

2016-06-01 Thread Matthäus G . Chajdas
Hopefully done - I'm not the biggest git expert but the history looks like I'd expect it to look like :) Cheers, Matthäus Am 01.06.2016 um 19:57 schrieb Brad King: > On 06/01/2016 01:53 PM, Matthäus G. Chajdas wrote: >> updated to latest master and pushed again (I merged

Re: [cmake-developers] Topic "add-opencl-imported-target" good to merge?

2016-06-01 Thread Matthäus G . Chajdas
, 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 wit

Re: [cmake-developers] Topic "add-opencl-imported-target" good to merge?

2016-06-02 Thread Matthäus G . Chajdas
Thanks! Cheers, Matthäus Am 02.06.2016 um 15:44 schrieb Brad King: > On 06/01/2016 03:00 PM, Matthäus G. Chajdas wrote: >> 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

[cmake-developers] FindVulkan.cmake

2016-06-04 Thread Matthäus G . Chajdas
Hi, I've pushed an add-FindVulkan topic branch which adds a module to search for the Vulkan graphics API (https://www.khronos.org/vulkan/). I'm also happy to maintain this going forward. The complete diff can be found here: https://cmake.org/gitweb?p=stage/cmake.git;a=commitdiff;h=276fa1497efd3

Re: [cmake-developers] FindVulkan.cmake

2016-06-06 Thread Matthäus G . Chajdas
Hi Brad, thanks a lot. Is this make test or is there something special needed to get the ModuleNotices? make test is super-slow, I wonder if there's a faster way to get to the relevant test. Cheers, Matthäus Am 06.06.2016 um 15:36 schrieb Brad King: > On 06/04/2016 02:54 PM, Ma

Re: [cmake-developers] FindVulkan.cmake

2016-06-06 Thread Matthäus G . Chajdas
Thanks a lot! I've pushed a squashed commit which does everything in one go, and merged it into next. Cheers, Matthäus Am 06.06.2016 um 17:22 schrieb Brad King: > On 06/06/2016 11:09 AM, Matthäus G. Chajdas wrote: >> thanks a lot. Is this make test or is there something spe

Re: [cmake-developers] [PATCH] Fix FindOpenCL on Mac OS

2016-08-01 Thread Matthäus G . Chajdas
Hi Jerry, thanks for giving it a spin :) I don't have a Mac to test myself - as your changes are confined to macOS, they look safe to me. The version is not listed as a required variable, so that's why it passes through. If you pass in a version into your find_package call, it should bail out if

Re: [cmake-developers] [PATCH] Fix FindOpenCL on Mac OS

2016-08-05 Thread Matthäus G . Chajdas
hanged something on macOS? > You are right _DIR and _DIRS are working. > I think I only changed it because for example FindGLUT uses it, for the > library the singular variant LIBRARY is used and I > don't understand the difference of _DIR and _DIRS :) > > Jerry > >

Re: [cmake-developers] [PATCH] Fix FindOpenCL on Mac OS

2016-08-07 Thread Matthäus G . Chajdas
Here we go: https://cmake.org/gitweb?p=stage/cmake.git;a=commit;h=48dc6343bba3b3f296d35ab060681c50f0eb8cde Thanks again for the patch! Cheers, Matthäus Am 05.08.2016 um 22:09 schrieb Matthäus G. Chajdas: > Hi, > > all right. Yes, that sounds like a pro

[cmake-developers] [New Module] FindOpenCL, FindHg

2014-02-08 Thread Matthäus G. Chajdas
Hi, I would like to propose two new modules for inclusion in CMake: FindOpenCL to find OpenCL and FindHg for Mercurial (see attached.) FindOpenCL is written in similar spirit to FindOpenGL, while FindHg is basically the equivalent of FindSubversion for Mercurial. The modules have been tested on W

Re: [cmake-developers] [New Module] FindOpenCL, FindHg

2014-02-08 Thread Matthäus G. Chajdas
nteru/findhgcl Cheers, Matthäus On 08.02.2014 14:59, Rolf Eike Beer wrote: > Am Samstag, 8. Februar 2014, 14:28:37 schrieb Matthäus G. Chajdas: >> Hi, >> >> I would like to propose two new modules for inclusion in CMake: >> FindOpenCL to find OpenCL and FindHg for Mercurial

Re: [cmake-developers] [New Module] FindOpenCL, FindHg

2014-02-12 Thread Matthäus G. Chajdas
t; Am Samstag, 8. Februar 2014, 16:38:02 schrieb Matthäus G. Chajdas: >> Hi Eike, >> >> thanks for taking a look. I've changed the module accordingly. Is there >> a recommendation whether to use HG_ or Hg_ as the prefix for the variables? >> >> As I guess t

Re: [cmake-developers] [New Module] FindOpenCL, FindHg

2014-02-16 Thread Matthäus G. Chajdas
Beer wrote: > Am Samstag, 15. Februar 2014, 18:54:47 schrieb Stephen Kelly: >> Rolf Eike Beer wrote: >>> Matthäus G. Chajdas wrote: >>>> Hi Eike, >>>> >>>> all right, then Hg, as it's FindHg, unless there is a naming policy I'm >>&g

Re: [cmake-developers] [New Module] FindOpenCL, FindHg

2014-02-16 Thread Matthäus G. Chajdas
Hi, couldn't find it in the documentation either :( I've changed it to OPENCL_VERSION_STRING. Should I apply for commit access now? Cheers, Matthäus On 16.02.2014 20:10, Rolf Eike Beer wrote: > Am Sonntag, 16. Februar 2014, 18:43:01 schrieb Matthäus G. Chajdas: >> Hi Eik

Re: [cmake-developers] [New Module] FindOpenCL, FindHg

2014-02-16 Thread Matthäus G. Chajdas
olf Eike Beer wrote: > Am Sonntag, 16. Februar 2014, 20:26:22 schrieb Matthäus G. Chajdas: >> Hi, >> >> couldn't find it in the documentation either :( > > Help/manual/cmake-developer.7.rst > >> I've changed it to OPENCL_VERSION_STRING. > > Shou

Re: [cmake-developers] [New Module] FindOpenCL, FindHg

2014-02-17 Thread Matthäus G. Chajdas
dules ready for acceptance, and if so, who is the person to ask for commit access, or is there something left to fix? Cheers, Matthäus On 16.02.2014 22:11, Stephen Kelly wrote: > Rolf Eike Beer wrote: > >> Am Sonntag, 16. Februar 2014, 18:43:01 schrieb Matthäus G. Chajdas: >>> H

Re: [cmake-developers] [New Module] FindOpenCL, FindHg

2014-02-19 Thread Matthäus G. Chajdas
no problem waiting longer, but it would be definitely helpful for first-time contributers like me to have a rough indication how fast the process goes. Is there something else I can do to speed up the process? Cheers, Matthäus On 19.02.2014 16:30, Brad King wrote: > On 02/17/2014 02:27 PM, &quo

Re: [cmake-developers] [New Module] FindOpenCL, FindHg

2014-02-23 Thread Matthäus G. Chajdas
Hi, > So, I'm very sorry to come up with those things now and not having catched > them earlier. If you now do no problem, I should have also checked what I was adding with git :/ I think I fixed everything now in my latest commit. At least `git diff origin/master Modules/FindHg.cmake` looks rea

Re: [cmake-developers] [New Module] FindOpenCL, FindHg

2014-02-25 Thread Matthäus G. Chajdas
Hi Brad, thanks for splitting the patches & the review. I've just pushed the requested changes to the add-FindOpenCL topic. If there is still something missing, please drop me a line; I appreciate your patience with a new contributor! Cheers, Matthäus On 25.02.2014 17:36, Brad King wrote: > H

[cmake-developers] CMake 3.0 find_patch changes?

2014-06-17 Thread Matthäus G. Chajdas
Hi, I've just tried my FindOpenCL Module in cmake-next, and with 3.0, it doesn't work as expected (on Windows) Before I submit the fix, I wanted to check back that I was indeed doing it wrong before. In my find_path, I used the following call: find_path(OpenCL_INCLUDE_DIR NAMES CL/cl.h Op