Re: UPDATE: /devel/ninja 1.7.1 -> 1.7.2

2016-11-19 Thread Stuart Henderson 
On 2016/11/19 16:53, David Coppa wrote:
> On Sat, Nov 19, 2016 at 1:45 PM, Stuart Henderson  
> wrote:
> > On 2016/11/18 19:25, Rafael Sadowski wrote:
> >> > To save time for testing, which ports did you try it with? (It will
> >> > need testing with at least a selection of cmake things, plus one of
> >> > chromium/iridium).
> >>
> >> Successful chromium-54.0.2840.100 build with ninja 1.7.2.
> >
> > Thanks, I think this is enough.
> 
> It works for cmake too.
> 
> > I just noticed this though:
> >
> > On 2016/11/13 20:57, Rafael Sadowski wrote:
> >> - add post-install to compile ninja_syntax.py
> >
> > I don't think this is useful, pyc files are only for things which
> > are imported.
> 
> Agreed.
> 
> I think this update can be committed, without the post-install, as Stuart 
> said.
> 
> Thanks!
> David

Thanks, committed.

Re: UPDATE: /devel/ninja 1.7.1 -> 1.7.2

2016-11-19 Thread David Coppa 
On Sat, Nov 19, 2016 at 1:45 PM, Stuart Henderson  wrote:
> On 2016/11/18 19:25, Rafael Sadowski wrote:
>> > To save time for testing, which ports did you try it with? (It will
>> > need testing with at least a selection of cmake things, plus one of
>> > chromium/iridium).
>>
>> Successful chromium-54.0.2840.100 build with ninja 1.7.2.
>
> Thanks, I think this is enough.

It works for cmake too.

> I just noticed this though:
>
> On 2016/11/13 20:57, Rafael Sadowski wrote:
>> - add post-install to compile ninja_syntax.py
>
> I don't think this is useful, pyc files are only for things which
> are imported.

Agreed.

I think this update can be committed, without the post-install, as Stuart said.

Thanks!
David



-- 
"If you try a few times and give up, you'll never get there. But if
you keep at it... There's a lot of problems in the world which can
really be solved by applying two or three times the persistence that
other people will."
-- Stewart Nelson

Re: UPDATE: /devel/ninja 1.7.1 -> 1.7.2

2016-11-19 Thread Stuart Henderson 
On 2016/11/18 19:25, Rafael Sadowski wrote:
> > To save time for testing, which ports did you try it with? (It will
> > need testing with at least a selection of cmake things, plus one of
> > chromium/iridium).
> 
> Successful chromium-54.0.2840.100 build with ninja 1.7.2.

Thanks, I think this is enough.

I just noticed this though:

On 2016/11/13 20:57, Rafael Sadowski wrote:
> - add post-install to compile ninja_syntax.py

I don't think this is useful, pyc files are only for things which
are imported.

Re: UPDATE: /devel/ninja 1.7.1 -> 1.7.2

2016-11-18 Thread Rafael Sadowski 
On Sun Nov 13, 2016 at 09:25:33PM +, Stuart Henderson wrote:
> On 2016/11/13 20:57, Rafael Sadowski wrote:
> > Hi ports@, Hi Matthew,
> > 
> > simple ninja update. Tested @amd64 with some ports in the last days.
> 
> To save time for testing, which ports did you try it with? (It will
> need testing with at least a selection of cmake things, plus one of
> chromium/iridium).

Successful chromium-54.0.2840.100 build with ninja 1.7.2.

Re: UPDATE: /devel/ninja 1.7.1 -> 1.7.2

2016-11-13 Thread Rafael Sadowski 
On Sun Nov 13, 2016 at 09:25:33PM +, Stuart Henderson wrote:
> On 2016/11/13 20:57, Rafael Sadowski wrote:
> > Hi ports@, Hi Matthew,
> > 
> > simple ninja update. Tested @amd64 with some ports in the last days.
> 
> To save time for testing, which ports did you try it with? (It will
> need testing with at least a selection of cmake things, plus one of
> chromium/iridium).

I know, devel/llvm, x11/tellico-kde4, graphics/opencv, devel/dlib
(openbsd-wip), devel/doxygen and more out of my scope are tested.

Re: UPDATE: /devel/ninja 1.7.1 -> 1.7.2

2016-11-13 Thread Stuart Henderson 
On 2016/11/13 20:57, Rafael Sadowski wrote:
> Hi ports@, Hi Matthew,
> 
> simple ninja update. Tested @amd64 with some ports in the last days.

To save time for testing, which ports did you try it with? (It will
need testing with at least a selection of cmake things, plus one of
chromium/iridium).

Re: UPDATE: /devel/ninja 1.7.1 -> 1.7.2

2016-11-13 Thread David Coppa 
On Sun, Nov 13, 2016 at 8:57 PM, Rafael Sadowski  wrote:
> Hi ports@, Hi Matthew,
>
> simple ninja update. Tested @amd64 with some ports in the last days.
>
> Port highlights:
>
> - switch to new github repo (official)
> - new HOMEPAGE
> - add post-install to compile ninja_syntax.py
> - cleanup do-install tabs
>
> Kind regards,
>
> Rafael Sadowski

Thanks, I'll have a look at it tomorrow.

Ciao!
David

UPDATE: /devel/ninja 1.7.1 -> 1.7.2

2016-11-13 Thread Rafael Sadowski 
Hi ports@, Hi Matthew,

simple ninja update. Tested @amd64 with some ports in the last days.

Port highlights:

- switch to new github repo (official)
- new HOMEPAGE
- add post-install to compile ninja_syntax.py
- cleanup do-install tabs

Kind regards,

Rafael Sadowski


Index: Makefile
===
RCS file: /cvs/ports/devel/ninja/Makefile,v
retrieving revision 1.19
diff -u -p -u -p -r1.19 Makefile
--- Makefile12 May 2016 07:14:15 -  1.19
+++ Makefile13 Nov 2016 19:50:57 -
@@ -7,14 +7,12 @@ NOT_FOR_ARCHS=m88k
 
 COMMENT =  small build system with a focus on speed
 
-V =1.7.1
-GH_ACCOUNT =   martine
+GH_ACCOUNT =   ninja-build
 GH_PROJECT =   ninja
-GH_TAGNAME =   v$V
-DISTNAME = ${GH_PROJECT}-$V
+GH_TAGNAME =   v1.7.2
 
 CATEGORIES =   devel
-HOMEPAGE = http://martine.github.io/ninja/
+HOMEPAGE = https://ninja-build.org/
 MAINTAINER =   Matthew Dempsky 
 
 # Apache License v2.0
@@ -39,8 +37,8 @@ MAKE_ENV =CXX="${CXX}"
 
 do-build:
@cd ${WRKSRC} && ${SETENV} ${MAKE_ENV} \
-   ${MODPY_BIN} ./configure.py --bootstrap \
-   --verbose --with-python=${MODPY_BIN}
+   ${MODPY_BIN} ./configure.py --bootstrap \
+   --verbose --with-python=${MODPY_BIN}
 
 do-install:
${INSTALL_PROGRAM} ${WRKBUILD}/ninja ${PREFIX}/bin
@@ -52,5 +50,9 @@ do-install:
${INSTALL_DATA} ${WRKSRC}/misc/ninja.vim ${PREFIX}/share/ninja
${INSTALL_DATA} ${WRKSRC}/misc/ninja_syntax.py ${PREFIX}/share/ninja
${INSTALL_DATA} ${WRKSRC}/misc/zsh-completion ${PREFIX}/share/ninja
+
+post-install:
+   ${MODPY_BIN} ${MODPY_LIBDIR}/compileall.py \
+   ${PREFIX}/share/ninja/ninja_syntax.py
 
 .include 
Index: distinfo
===
RCS file: /cvs/ports/devel/ninja/distinfo,v
retrieving revision 1.8
diff -u -p -u -p -r1.8 distinfo
--- distinfo12 May 2016 07:14:15 -  1.8
+++ distinfo13 Nov 2016 19:50:57 -
@@ -1,2 +1,2 @@
-SHA256 (ninja-1.7.1.tar.gz) = UVgd5Tz0cFuJ62sUqFuqcyiK0Iv/JW59MNUpFVgTvhk=
-SIZE (ninja-1.7.1.tar.gz) = 179717
+SHA256 (ninja-1.7.2.tar.gz) = Lt2gpUIazjz0KDCSEScHct01qRr2DJb5P5Dfa8QbFtk=
+SIZE (ninja-1.7.2.tar.gz) = 180611
Index: pkg/PLIST
===
RCS file: /cvs/ports/devel/ninja/pkg/PLIST,v
retrieving revision 1.2
diff -u -p -u -p -r1.2 PLIST
--- pkg/PLIST   14 Jun 2013 16:39:39 -  1.2
+++ pkg/PLIST   13 Nov 2016 19:50:57 -
@@ -7,4 +7,5 @@ share/ninja/bash-completion
 share/ninja/ninja-mode.el
 share/ninja/ninja.vim
 share/ninja/ninja_syntax.py
+share/ninja/ninja_syntax.pyc
 share/ninja/zsh-completion