Re: [cmake-developers] [PATCH] New module: FindIce.cmake

2014-08-27 Thread Roger Leigh
On Fri, Aug 22, 2014 at 11:17:05AM +0100, Roger Leigh wrote: > On Thu, Aug 21, 2014 at 02:47:24PM +0200, Rolf Eike Beer wrote: > > Roger Leigh wrote: > > > On Sun, Aug 17, 2014 at 05:22:38PM +0100, Roger Leigh wrote: > > > > On Sun, Aug 17, 2014 at 05:50:58PM +0200, Rolf Eike Beer wrote: > > > > >

Re: [cmake-developers] [PATCH] New module: FindIce.cmake

2014-08-22 Thread Roger Leigh
On Thu, Aug 21, 2014 at 02:47:24PM +0200, Rolf Eike Beer wrote: > Roger Leigh wrote: > > On Sun, Aug 17, 2014 at 05:22:38PM +0100, Roger Leigh wrote: > > > On Sun, Aug 17, 2014 at 05:50:58PM +0200, Rolf Eike Beer wrote: > > > > Am Sonntag, 17. August 2014, 16:21:24 schrieb Roger Leigh: > > > > > On

Re: [cmake-developers] [PATCH] New module: FindIce.cmake

2014-08-21 Thread Rolf Eike Beer
Roger Leigh wrote: > On Sun, Aug 17, 2014 at 05:22:38PM +0100, Roger Leigh wrote: > > On Sun, Aug 17, 2014 at 05:50:58PM +0200, Rolf Eike Beer wrote: > > > Am Sonntag, 17. August 2014, 16:21:24 schrieb Roger Leigh: > > > > On Fri, Aug 15, 2014 at 12:31:17AM +0100, Roger Leigh wrote: > > > > > OK.

Re: [cmake-developers] [PATCH] New module: FindIce.cmake

2014-08-18 Thread Roger Leigh
On Sun, Aug 17, 2014 at 05:22:38PM +0100, Roger Leigh wrote: > On Sun, Aug 17, 2014 at 05:50:58PM +0200, Rolf Eike Beer wrote: > > Am Sonntag, 17. August 2014, 16:21:24 schrieb Roger Leigh: > > > On Fri, Aug 15, 2014 at 12:31:17AM +0100, Roger Leigh wrote: > > > > OK. I'll have to read up on this

Re: [cmake-developers] [PATCH] New module: FindIce.cmake

2014-08-17 Thread Roger Leigh
On Sun, Aug 17, 2014 at 05:50:58PM +0200, Rolf Eike Beer wrote: > Am Sonntag, 17. August 2014, 16:21:24 schrieb Roger Leigh: > > On Fri, Aug 15, 2014 at 12:31:17AM +0100, Roger Leigh wrote: > > > OK. I'll have to read up on this and see what needs doing. > > > In the meantime, I've attached a revi

Re: [cmake-developers] [PATCH] New module: FindIce.cmake

2014-08-17 Thread Rolf Eike Beer
Am Sonntag, 17. August 2014, 16:21:24 schrieb Roger Leigh: > On Fri, Aug 15, 2014 at 12:31:17AM +0100, Roger Leigh wrote: > > OK. I'll have to read up on this and see what needs doing. > > In the meantime, I've attached a revised patch with all the > > above corrections included. > > Based on the

Re: [cmake-developers] [PATCH] New module: FindIce.cmake

2014-08-17 Thread Roger Leigh
On Fri, Aug 15, 2014 at 12:31:17AM +0100, Roger Leigh wrote: > OK. I'll have to read up on this and see what needs doing. > In the meantime, I've attached a revised patch with all the > above corrections included. Based on the comments made on my other FindXerces patch, I've updated this patch to

Re: [cmake-developers] [PATCH] New module: FindIce.cmake

2014-08-14 Thread Roger Leigh
On Wed, Aug 13, 2014 at 10:10:01AM -0400, Brad King wrote: > On 08/12/2014 05:59 PM, Roger Leigh wrote: > >> Regarding the Windows Registry, I've taken a look and it looks > >> like there might be some usable keys from the installer which > >> could be used, but I'll need to do further digging with

Re: [cmake-developers] [PATCH] New module: FindIce.cmake

2014-08-13 Thread Brad King
Hi Roger, Thanks for the revisions! On 08/12/2014 05:59 PM, Roger Leigh wrote: >> Regarding the Windows Registry, I've taken a look and it looks >> like there might be some usable keys from the installer which >> could be used, but I'll need to do further digging with all >> the different version

Re: [cmake-developers] [PATCH] New module: FindIce.cmake

2014-08-12 Thread Roger Leigh
On Tue, Aug 12, 2014 at 09:08:57PM +0100, Roger Leigh wrote: > On Mon, Aug 11, 2014 at 11:07:57AM -0400, Brad King wrote: > > On 08/08/2014 08:56 AM, Roger Leigh wrote: > > > On Thu, Aug 07, 2014 at 06:49:28PM +0100, Roger Leigh wrote: > > > 3. There is a lot of hard-coded version-specific informa

Re: [cmake-developers] [PATCH] New module: FindIce.cmake

2014-08-12 Thread Roger Leigh
On Mon, Aug 11, 2014 at 11:07:57AM -0400, Brad King wrote: > On 08/08/2014 08:56 AM, Roger Leigh wrote: > > On Thu, Aug 07, 2014 at 06:49:28PM +0100, Roger Leigh wrote: > >> Hi, > >> > >> I've written a module for finding the details of a ZeroC ICE installation, > >> attached, which I thought might

Re: [cmake-developers] [PATCH] New module: FindIce.cmake

2014-08-11 Thread Brad King
On 08/08/2014 08:56 AM, Roger Leigh wrote: > On Thu, Aug 07, 2014 at 06:49:28PM +0100, Roger Leigh wrote: >> Hi, >> >> I've written a module for finding the details of a ZeroC ICE installation, >> attached, which I thought might be of interest to a wider audience and be >> suitable for direct inclu

Re: [cmake-developers] [PATCH] New module: FindIce.cmake

2014-08-08 Thread Roger Leigh
On Thu, Aug 07, 2014 at 06:49:28PM +0100, Roger Leigh wrote: > Hi, > > I've written a module for finding the details of a ZeroC ICE installation, > attached, which I thought might be of interest to a wider audience and be > suitable for direct inclusion into cmake. I've attached the patch for thi

[cmake-developers] [PATCH] New module: FindIce.cmake

2014-08-07 Thread Roger Leigh
Hi, I've written a module for finding the details of a ZeroC ICE installation, attached, which I thought might be of interest to a wider audience and be suitable for direct inclusion into cmake. I've attached the patch for this. The docs should be correct, but I'm not yet totally familiar with th