[cmake-developers] [CMake 0013379]: Toolchain prefix not set correctly for Fortran projects

2012-07-06 Thread Mantis Bug Tracker
The following issue has been SUBMITTED. == http://public.kitware.com/Bug/view.php?id=13379 == Reported By:Yngve Inntjore Levinsen Assigned

[cmake-developers] Ninja passes all tests on OS X

2012-07-06 Thread Nicolas Desprès
Hi, I finally found the time to fix the Ninja generator on OS X. Mainly I have fixed ExportImport and Qt4Deploy. I based my work on jkp's work (https://github.com/jkp/CMake/commits/jkp/ninja-generator). Although there are other patches in this branch, I just included commits fixing tests. We

Re: [cmake-developers] Ninja passes all tests on OS X

2012-07-06 Thread Nicolas Desprès
I forgot to say that most of the code is duplicated from the Makefile generator. I have left some todo but this code will need to be factored sooner or later. On Fri, Jul 6, 2012 at 12:54 PM, Nicolas Desprès nicolas.desp...@gmail.com wrote: Hi, I finally found the time to fix the Ninja

Re: [cmake-developers] Ninja passes all tests on OS X

2012-07-06 Thread David Cole
We should re-factor it sooner (not later). This is great, thank you very much! But we should definitely re-factor it before merging it to 'master' Thx, David On Fri, Jul 6, 2012 at 7:05 AM, Nicolas Desprès nicolas.desp...@gmail.com wrote: I forgot to say that most of the code is duplicated

Re: [cmake-developers] Ninja passes all tests on OS X

2012-07-06 Thread Peter Kümmel
On 06.07.2012 12:54, Nicolas Desprès wrote: Hi, I finally found the time to fix the Ninja generator on OS X. Mainly I have fixed ExportImport and Qt4Deploy. I based my work on jkp's work (https://github.com/jkp/CMake/commits/jkp/ninja-generator). Although there are other patches in this branch,

Re: [cmake-developers] Ninja passes all tests on OS X

2012-07-06 Thread Nicolas Desprès
I agree. Any idea how you would like to re-factor it? I am thinking about using multiple inheritance by introducing a cmOSXBundleGenerator class with CreateFramework and CreateAppBundle. WDYT? Cheers, Nico On Fri, Jul 6, 2012 at 1:13 PM, David Cole david.c...@kitware.com wrote: We should

Re: [cmake-developers] Ninja passes all tests on OS X

2012-07-06 Thread Nicolas Desprès
On Fri, Jul 6, 2012 at 1:29 PM, Peter Kümmel syntheti...@gmx.net wrote: On 06.07.2012 12:54, Nicolas Desprès wrote: Hi, I finally found the time to fix the Ninja generator on OS X. Mainly I have fixed ExportImport and Qt4Deploy. I based my work on jkp's work

[cmake-developers] [CMake 0013380]: Xcode paths generated incorrectly

2012-07-06 Thread Mantis Bug Tracker
The following issue has been SUBMITTED. == http://public.kitware.com/Bug/view.php?id=13380 == Reported By:Jay Sistar Assigned To:

Re: [cmake-developers] Ninja passes all tests on OS X

2012-07-06 Thread Bill Hoffman
On 7/6/2012 7:33 AM, Nicolas Desprès wrote: I agree. Any idea how you would like to re-factor it? I am thinking about using multiple inheritance by introducing a cmOSXBundleGenerator class with CreateFramework and CreateAppBundle. WDYT? Cheers, Nico Thanks, this is great. I am not a big fan

[cmake-developers] [CMake 0013381]: /usr/include ignored by code::Blocks generator

2012-07-06 Thread Mantis Bug Tracker
The following issue has been SUBMITTED. == http://public.kitware.com/Bug/view.php?id=13381 == Reported By:Nicholas Katzakis Assigned To:

Re: [cmake-developers] Ninja passes all tests on OS X

2012-07-06 Thread Nicolas Desprès
On Fri, Jul 6, 2012 at 3:52 PM, Bill Hoffman bill.hoff...@kitware.com wrote: On 7/6/2012 7:33 AM, Nicolas Desprès wrote: I agree. Any idea how you would like to re-factor it? I am thinking about using multiple inheritance by introducing a cmOSXBundleGenerator class with CreateFramework and

Re: [cmake-developers] Ninja passes all tests on OS X

2012-07-06 Thread Bill Hoffman
On 7/6/2012 12:45 PM, Nicolas Desprès wrote: Both work. Generally I use helper class with language not supporting multiple inheritance but if you prefer not to use it that's not a problem. I'll do the patch. Thanks for answering. Great, thanks for doing the work. Lets go with the helper class,