Re: [cmake-developers] [patch] fix Info.plist parser

2014-08-11 Thread Brad King
On 08/11/2014 12:54 PM, Ben Boeckel wrote: >> +string(REGEX REPLACE "\r" "${eol_char};" info_plist "${info_plist}") > > Do these need to be REGEX REPLACE? Any problem with straight REPLACE? There are two versions of the commit. One on master that uses straight REPLACE, and one backported to

Re: [cmake-developers] [patch] fix Info.plist parser

2014-08-11 Thread Ben Boeckel
On Thu, Aug 07, 2014 at 19:36:54 +0200, Tim Blechmann wrote: > diff --git a/Modules/BundleUtilities.cmake b/Modules/BundleUtilities.cmake > index b896de2..60d01b5 100644 > --- a/Modules/BundleUtilities.cmake > +++ b/Modules/BundleUtilities.cmake > @@ -237,6 +237,7 @@ function(get_bundle_main_execut

Re: [cmake-developers] Patch: Detect Fujitsu compilers

2014-08-11 Thread Erik Lindahl
Hi Bill, Would have loved to, but unfortunately we can't even test our own code (Gromacs) automatically on K. Still, we do test things every few months, so we'll contribute new patches if we find anything that breaks. ... and I just realized I have a patch for the openMP flag too that I'll post

Re: [cmake-developers] Patch: Detect Fujitsu compilers

2014-08-11 Thread Bill Hoffman
On 8/11/2014 10:39 AM, Brad King wrote: The attached patch provides detection for the Fujitsu C/C++ compilers Can you run a dashboard for this compilers? It would be good to make sure it keeps working in the future. http://www.cmake.org/Wiki/CMake/Git/Dashboard Thanks. -Bill -- Powered b

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] fix Info.plist parser

2014-08-11 Thread Brad King
On 08/07/2014 02:59 PM, Tim Blechmann wrote: >> Once it works there I will consider backporting it to 3.0. > > great! thanks a lot! Backported here: BundleUtilities: Allow Info.plist files which use CR line endings http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=00ad505b -Brad -- Powere

Re: [cmake-developers] Patch: Detect Fujitsu compilers

2014-08-11 Thread Brad King
On 08/10/2014 05:13 PM, Erik Lindahl wrote: > The attached patch provides detection for the Fujitsu C/C++ compilers Applied, thanks: Add Fujitsu compiler detection http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=0578c283 > I don't think the version information is very important, but if you

Re: [cmake-developers] [update]: CPack IFW generator

2014-08-11 Thread Brad King
On 08/06/2014 01:33 PM, Brad King wrote: > CPackIFW: Refactor and revise this generator > http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=194b412b After some fixes for nightly testing this is now in 'master' as: CPackIFW: Refactor and revise this generator http://cmake.org/gitweb?p=cmake.g