Re: [NEW] www/p5-Apache-LogFormat-Compiler

2015-01-17 Thread Andrew Fresh
On Sun, Jan 04, 2015 at 10:27:25PM +0100, Cesare Gargano wrote:
 On 2015-01-04 22:04, Andrew Fresh wrote:
 On Sun, Jan 04, 2015 at 08:13:45PM +0100, Cesare Gargano wrote:
 Needed for upcoming update to www/p5-Plack (needed for upcoming
 www/p5-Dancer2).
 
 Tested on amd64.
 
 -
 pkg/DESCR:
 Compile a log format string to perl-code. For faster generation of
 access_log lines.
 
 This needs several more TEST_DEPENDS, other than that it looks fine.
 
 -TEST_DEPENDS = devel/p5-Test-MockTime
 +TEST_DEPENDS = devel/p5-Test-MockTime=0 \
 +   devel/p5-Test-Requires=0 \
 +   devel/p5-Try-Tiny=0.12 \
 +   www/p5-HTTP-Message=0 \
 +   www/p5-URI=1.60
 +
 
 l8rZ,
 
 Ok, attached is the right version.

OK afresh1@

I think CONFIGURE_STYLE and PERMIT_PACKAGE_CDROM are still on the wrong
sides of *_DEPENDS but it's a nice short Makefile so not a big deal to
me.



Re: [NEW] www/p5-Apache-LogFormat-Compiler

2015-01-17 Thread Benoit Lecocq
On 01/17/15 23:38, Andrew Fresh wrote:
 On Sun, Jan 04, 2015 at 10:27:25PM +0100, Cesare Gargano wrote:
 On 2015-01-04 22:04, Andrew Fresh wrote:
 On Sun, Jan 04, 2015 at 08:13:45PM +0100, Cesare Gargano wrote:
 Needed for upcoming update to www/p5-Plack (needed for upcoming
 www/p5-Dancer2).

 Tested on amd64.

 -
 pkg/DESCR:
 Compile a log format string to perl-code. For faster generation of
 access_log lines.

 This needs several more TEST_DEPENDS, other than that it looks fine.

 -TEST_DEPENDS = devel/p5-Test-MockTime
 +TEST_DEPENDS = devel/p5-Test-MockTime=0 \
 +   devel/p5-Test-Requires=0 \
 +   devel/p5-Try-Tiny=0.12 \
 +   www/p5-HTTP-Message=0 \
 +   www/p5-URI=1.60
 +

 l8rZ,

 Ok, attached is the right version.
 
 OK afresh1@
 
 I think CONFIGURE_STYLE and PERMIT_PACKAGE_CDROM are still on the wrong
 sides of *_DEPENDS but it's a nice short Makefile so not a big deal to
 me.
 

Committed, thanks !



Re: [NEW] www/p5-Apache-LogFormat-Compiler

2015-01-16 Thread Cesare Gargano
Ping

On Mon, Jan 5, 2015 at 5:02 PM, Andrew Fresh and...@afresh1.com wrote:
 On Mon, Jan 05, 2015 at 11:22:02AM +0100, Cesare Gargano wrote:
 PERMIT_PACKAGE* are _after_ *_DEPENDS. Am I wrong?

 As I understand, in general things should be in the order specified in
 Makefile.template.

 http://cvsweb.openbsd.org/cgi-bin/cvsweb/ports/infrastructure/templates/Makefile.template

 I have found places where I think the Makefile is clearer in a different
 order, but that is not frequently the case.

 (And I see you resubmitted, but seems good to answer this question for
 the archives)

 l8rZ,
 --
 andrew - http://afresh1.com

 Beta. Software undergoes beta testing shortly before it's released.
Beta is Latin for still doesn't work.




-- 
Cesare Gargano | +39 328 56 44 911
Plusiagamma - Web | Mobile | Embedded | Training
http://www.plusiagamma.com



Re: [NEW] www/p5-Apache-LogFormat-Compiler

2015-01-05 Thread Cesare Gargano
On Jan 5, 2015 5:55 AM, Andrew Fresh afre...@openbsd.org wrote:

 On Sun, Jan 04, 2015 at 10:27:25PM +0100, Cesare Gargano wrote: 
  On 2015-01-04 22:04, Andrew Fresh wrote: 
  On Sun, Jan 04, 2015 at 08:13:45PM +0100, Cesare Gargano wrote: 
  Needed for upcoming update to www/p5-Plack (needed for upcoming 
  www/p5-Dancer2). 
   
  Tested on amd64. 
   
  - 
  pkg/DESCR: 
  Compile a log format string to perl-code. For faster generation of 
  access_log lines. 
   
  This needs several more TEST_DEPENDS, other than that it looks fine. 
   
  -TEST_DEPENDS = devel/p5-Test-MockTime 
  +TEST_DEPENDS = devel/p5-Test-MockTime=0 \ 
  +   devel/p5-Test-Requires=0 \ 
  +   devel/p5-Try-Tiny=0.12 \ 
  +   www/p5-HTTP-Message=0 \ 
  +   www/p5-URI=1.60 
  + 
   
  l8rZ, 
  
  Ok, attached is the right version. 

 This is much better.  Not sure how much of a stickler sthen@ is 
 about PERMIT_PACKAGE* coming before *_DEPENDS but that's the only nit I 
 see left to pick. 

 l8rZ, 
 -- 
 andrew - http://afresh1.com 

 I wish life had an UNDO function. 


PERMIT_PACKAGE* are _after_ *_DEPENDS. Am I wrong? 



Re: [NEW] www/p5-Apache-LogFormat-Compiler

2015-01-05 Thread Cesare Gargano

On 2015-01-05 11:22, Cesare Gargano wrote:

On Jan 5, 2015 5:55 AM, Andrew Fresh afre...@openbsd.org wrote:


On Sun, Jan 04, 2015 at 10:27:25PM +0100, Cesare Gargano wrote:
 On 2015-01-04 22:04, Andrew Fresh wrote:
 On Sun, Jan 04, 2015 at 08:13:45PM +0100, Cesare Gargano wrote:
 Needed for upcoming update to www/p5-Plack (needed for upcoming
 www/p5-Dancer2).
 
 Tested on amd64.
 
 -
 pkg/DESCR:
 Compile a log format string to perl-code. For faster generation of
 access_log lines.
 
 This needs several more TEST_DEPENDS, other than that it looks fine.
 
 -TEST_DEPENDS = devel/p5-Test-MockTime
 +TEST_DEPENDS = devel/p5-Test-MockTime=0 \
 +   devel/p5-Test-Requires=0 \
 +   devel/p5-Try-Tiny=0.12 \
 +   www/p5-HTTP-Message=0 \
 +   www/p5-URI=1.60
 +
 
 l8rZ,

 Ok, attached is the right version.

This is much better.  Not sure how much of a stickler sthen@ is
about PERMIT_PACKAGE* coming before *_DEPENDS but that's the only nit 
I

see left to pick.

l8rZ,
--
andrew - http://afresh1.com

I wish life had an UNDO function.



PERMIT_PACKAGE* are _after_ *_DEPENDS. Am I wrong?


I updated PERMIT_* order to all posted ports as for
infrastructure/templates/Makefile.template

--
C.



Re: [NEW] www/p5-Apache-LogFormat-Compiler

2015-01-05 Thread Andrew Fresh
On Mon, Jan 05, 2015 at 11:22:02AM +0100, Cesare Gargano wrote:
 PERMIT_PACKAGE* are _after_ *_DEPENDS. Am I wrong? 

As I understand, in general things should be in the order specified in
Makefile.template.

http://cvsweb.openbsd.org/cgi-bin/cvsweb/ports/infrastructure/templates/Makefile.template

I have found places where I think the Makefile is clearer in a different
order, but that is not frequently the case.

(And I see you resubmitted, but seems good to answer this question for
the archives)

l8rZ,
-- 
andrew - http://afresh1.com

Beta. Software undergoes beta testing shortly before it's released.
   Beta is Latin for still doesn't work.



[NEW] www/p5-Apache-LogFormat-Compiler

2015-01-04 Thread Cesare Gargano

Hi ports@!

Needed for upcoming update to www/p5-Plack (needed for upcoming 
www/p5-Dancer2).


Tested on amd64.

-
pkg/DESCR:
Compile a log format string to perl-code. For faster generation of 
access_log lines.


--
C.

p5-Apache-LogFormat-Compiler.tgz
Description: GNU Zip compressed data


Re: [NEW] www/p5-Apache-LogFormat-Compiler

2015-01-04 Thread Andrew Fresh
On Sun, Jan 04, 2015 at 08:13:45PM +0100, Cesare Gargano wrote:
 Needed for upcoming update to www/p5-Plack (needed for upcoming
 www/p5-Dancer2).
 
 Tested on amd64.
 
 -
 pkg/DESCR:
 Compile a log format string to perl-code. For faster generation of
 access_log lines.

This needs several more TEST_DEPENDS, other than that it looks fine.

-TEST_DEPENDS = devel/p5-Test-MockTime
+TEST_DEPENDS = devel/p5-Test-MockTime=0 \
+   devel/p5-Test-Requires=0 \
+   devel/p5-Try-Tiny=0.12 \
+   www/p5-HTTP-Message=0 \
+   www/p5-URI=1.60
+

l8rZ,
-- 
andrew - http://afresh1.com

The power The nightmares The strange medication



Re: [NEW] www/p5-Apache-LogFormat-Compiler

2015-01-04 Thread Cesare Gargano

On 2015-01-04 22:04, Andrew Fresh wrote:

On Sun, Jan 04, 2015 at 08:13:45PM +0100, Cesare Gargano wrote:

Needed for upcoming update to www/p5-Plack (needed for upcoming
www/p5-Dancer2).

Tested on amd64.

-
pkg/DESCR:
Compile a log format string to perl-code. For faster generation of
access_log lines.


This needs several more TEST_DEPENDS, other than that it looks fine.

-TEST_DEPENDS = devel/p5-Test-MockTime
+TEST_DEPENDS = devel/p5-Test-MockTime=0 \
+   devel/p5-Test-Requires=0 \
+   devel/p5-Try-Tiny=0.12 \
+   www/p5-HTTP-Message=0 \
+   www/p5-URI=1.60
+

l8rZ,


Ok, attached is the right version.

Thanks!

--
C.

p5-Apache-LogFormat-Compiler.tgz
Description: GNU Zip compressed data


Re: [NEW] www/p5-Apache-LogFormat-Compiler

2015-01-04 Thread Andrew Fresh
On Sun, Jan 04, 2015 at 10:27:25PM +0100, Cesare Gargano wrote:
 On 2015-01-04 22:04, Andrew Fresh wrote:
 On Sun, Jan 04, 2015 at 08:13:45PM +0100, Cesare Gargano wrote:
 Needed for upcoming update to www/p5-Plack (needed for upcoming
 www/p5-Dancer2).
 
 Tested on amd64.
 
 -
 pkg/DESCR:
 Compile a log format string to perl-code. For faster generation of
 access_log lines.
 
 This needs several more TEST_DEPENDS, other than that it looks fine.
 
 -TEST_DEPENDS = devel/p5-Test-MockTime
 +TEST_DEPENDS = devel/p5-Test-MockTime=0 \
 +   devel/p5-Test-Requires=0 \
 +   devel/p5-Try-Tiny=0.12 \
 +   www/p5-HTTP-Message=0 \
 +   www/p5-URI=1.60
 +
 
 l8rZ,
 
 Ok, attached is the right version.

This is much better.  Not sure how much of a stickler sthen@ is
about PERMIT_PACKAGE* coming before *_DEPENDS but that's the only nit I
see left to pick.

l8rZ,
-- 
andrew - http://afresh1.com

I wish life had an UNDO function.