Re: CMake: use ninja to build cmake itself

2013-06-28 Thread David Coppa
On Fri, Jun 28, 2013 at 6:37 PM, Christian Weisgerber wrote: > David Coppa wrote: > >> Tested. It works w/o problems. >> Opinions/OKs ? > > This finally gets the cmake build to honor MAKE_JOBS, so > DPB_PROPERTIES=parallel will actually make sense. The old build > used the setting only for the f

Re: CMake: use ninja to build cmake itself

2013-06-28 Thread Christian Weisgerber
David Coppa wrote: > Tested. It works w/o problems. > Opinions/OKs ? This finally gets the cmake build to honor MAKE_JOBS, so DPB_PROPERTIES=parallel will actually make sense. The old build used the setting only for the first bootstrap stage and the rest of the build ran one job at a time. mak

Re: CMake: use ninja to build cmake itself

2013-06-28 Thread Marc Espie
On Fri, Jun 28, 2013 at 02:57:04PM +0200, David Coppa wrote: > On Fri, Jun 28, 2013 at 2:52 PM, Stuart Henderson wrote: > > On 2013/06/28 06:03, David Coppa wrote: > >> > >> Tested. It works w/o problems. > >> > >> Opinions/OKs ? > > > > Makes sense, but I wonder if it might be better to move USE_

Re: CMake: use ninja to build cmake itself

2013-06-28 Thread David Coppa
On Fri, Jun 28, 2013 at 2:52 PM, Stuart Henderson wrote: > On 2013/06/28 06:03, David Coppa wrote: >> >> Tested. It works w/o problems. >> >> Opinions/OKs ? > > Makes sense, but I wonder if it might be better to move USE_NINJA to > bsd.port.mk to avoid duplication.. > Already requested to Marc ;)

Re: CMake: use ninja to build cmake itself

2013-06-28 Thread Stuart Henderson
On 2013/06/28 06:03, David Coppa wrote: > > Tested. It works w/o problems. > > Opinions/OKs ? Makes sense, but I wonder if it might be better to move USE_NINJA to bsd.port.mk to avoid duplication..

CMake: use ninja to build cmake itself

2013-06-28 Thread David Coppa
Tested. It works w/o problems. Opinions/OKs ? Index: Makefile === RCS file: /cvs/ports/devel/cmake/Makefile,v retrieving revision 1.84 diff -u -p -u -p -r1.84 Makefile --- Makefile25 Jun 2013 07:16:46 - 1.84 +++ Makefil