Re: UPDATE: devel/netbeans

2016-11-23 Thread Daniel Dickman
On Wed, Nov 23, 2016 at 4:11 PM, Rafael Sadowski wrote: > Hi everybody, > > Giovanni Bechis asked me to submit the new netbenas diff to ports@. > better to put online and send a link instead of a huge inline diff. > So here we go, below you can find the new diff with many input from > Giovanni

Re: [NEW] textproc/distribution

2016-11-23 Thread Jeremie Courreges-Anglas
Taylor Stearns writes: > On Tue, Nov 15, 2016 at 06:56:59PM +0100, Rafael Sadowski wrote: >> On Mon Nov 14, 2016 at 03:54:30PM -0500, Taylor Stearns wrote: >> > On Fri, Nov 04, 2016 at 05:52:49PM -0400, Taylor Stearns wrote: >> > > On Thu, Jun 16, 2016 at 07:28:43PM -0400, Taylor Stearns wrote: >

Re: games/slash : choosing character 4 time won't let you play

2016-11-23 Thread Jeremie Courreges-Anglas
Solène Rapenne writes: > There is a bug in games/slash which can be annoying, I'm using -current > at the moment of this mail. When you start the game and you want to > choose your character, whatever your pick, the game ends with abort trap > like this (where I pick A) => > >>An Archeologist, a

[UPDATE] devel/p5-MooseX-Types-LoadableClass to 0.014

2016-11-23 Thread Andrew Fresh
Only real change is switching from Class::Load to Module::Runtime for loading modules. https://metacpan.org/changes/distribution/MooseX-Types-LoadableClass Tests all pass, and reverse depends don't seem any worse off. Catalyst could use some love, maybe eventually I'll get there. OK? Index: Ma

Re: UPDATE: net/lftp

2016-11-23 Thread Jeremie Courreges-Anglas
Rafael Sadowski writes: > On Wed Nov 23, 2016 at 12:24:01PM +, Stuart Henderson wrote: >> On 2016/11/22 04:40, Jeremie Courreges-Anglas wrote: >> > > # lftp needs add_history_time() >> > > LIB_DEPENDS=devel/libidn \ >> > > devel/readline>=6.1p2 \ >> > > + >> > > +RUN_DE

Re: Update: multimedia/mpv, feedback needed

2016-11-23 Thread Jeremie Courreges-Anglas
"Dmitrij D. Czarkoff" writes: > Jeremie Courreges-Anglas wrote: > >>I don't use mpv, but the fact that it uses libsmbclient puts mpv on my >>plate when I update samba. Last time I checked I didn't find the >>smbclient support particularly useful (access files on network >>filesystems that aren'

Re: [UPDATE] devel/p5-strictures

2016-11-23 Thread Andrew Fresh
On Wed, Nov 23, 2016 at 12:14:47PM +0300, Sergey Bronnikov wrote: > Ok? Looks OK to me. Tests all pass, as do those for reverse depends. https://metacpan.org/changes/distribution/strictures > diff --git a/devel/p5-strictures/Makefile b/devel/p5-strictures/Makefile > index 47fcfc0..8738127 1006

Re: [UPDATE] devel/p5-indirect

2016-11-23 Thread Andrew Fresh
On Wed, Nov 23, 2016 at 12:14:05PM +0300, Sergey Bronnikov wrote: > Ok? Looks OK to me. Tests all pass, as do those for reverse depends. Seems like mostly bugfixes and refactoring. https://metacpan.org/changes/distribution/indirect > diff --git a/devel/p5-indirect/Makefile b/devel/p5-indirec

Re: [UPDATE] devel/p5-Role-Tiny

2016-11-23 Thread Andrew Fresh
On Wed, Nov 23, 2016 at 12:13:26PM +0300, Sergey Bronnikov wrote: > Ok? Looks OK to me. Tests all pass, as do those for reverse depends. https://metacpan.org/changes/distribution/Role-Tiny > diff --git a/devel/p5-Role-Tiny/Makefile b/devel/p5-Role-Tiny/Makefile > index e3ca498..9c1dbae 100644

Re: [UPDATE] devel/p5-Import-Into

2016-11-23 Thread Andrew Fresh
On Wed, Nov 23, 2016 at 12:12:47PM +0300, Sergey Bronnikov wrote: > Ok? This one appears to need an additional RUN_DEPENDS to actually work. Other that that it seem OK. Seems like a good bugfix to have. https://metacpan.org/changes/distribution/Import-Into Index: Makefile =

Re: [UPDATE] devel/p5-IO-TieCombine

2016-11-23 Thread Andrew Fresh
On Wed, Nov 23, 2016 at 12:12:09PM +0300, Sergey Bronnikov wrote: > Ok? Looks OK to me. Tests all pass as do those for reverse depends. https://metacpan.org/changes/distribution/IO-TieCombine > diff --git a/devel/p5-IO-TieCombine/Makefile b/devel/p5-IO-TieCombine/Makefile > index 133c184..46a3a

Re: Update: multimedia/mpv, feedback needed

2016-11-23 Thread Christian Weisgerber
On 2016-11-23, "Dmitrij D. Czarkoff" wrote: > I was asked off-list to add a "lite" flavor for multimedia/mpv. This > diff introduces such version. This iteration disables the fllowing in > "lite" FLAVOR: -snip- Well, if I was using mpv (as espie urges me to), I would want some of the removed f

Re: [UPDATE] devel/p5-ExtUtils-InstallPaths

2016-11-23 Thread Andrew Fresh
On Wed, Nov 23, 2016 at 12:11:17PM +0300, Sergey Bronnikov wrote: > Ok? This looks OK to me, no changes we care about AFAICT. Tests all pass as do those for reverse depends. https://metacpan.org/changes/distribution/ExtUtils-InstallPaths > diff --git a/devel/p5-ExtUtils-InstallPaths/Makefile >

Re: [UPDATE] devel/p5-App-Cmd

2016-11-23 Thread Andrew Fresh
On Wed, Nov 23, 2016 at 12:10:27PM +0300, Sergey Bronnikov wrote: > Ok? This looks OK to me. Tests pass as do all the tests for reverse dependencies. Looks like mostly documentation changes. https://metacpan.org/changes/distribution/App-Cmd > diff --git a/devel/p5-App-Cmd/Makefile b/devel/p5-

games/slash : choosing character 4 time won't let you play

2016-11-23 Thread Solène Rapenne
There is a bug in games/slash which can be annoying, I'm using -current at the moment of this mail. When you start the game and you want to choose your character, whatever your pick, the game ends with abort trap like this (where I pick A) => >An Archeologist, a Barbarian, a Caveman, [..], or a Wi

need review for games/nethack36

2016-11-23 Thread Solène RAPENNE
Hello, nethack 3.6.0 has been released at the end of last year. Here is a port to bring it on OpenBSD. Marc Espie told me that we need to keep both games versions (if people have saves), so I propose to have games/nethack{34,36}, this requires to have games/hackdata{34,36} too, I don't know if it'

Re: UPDATE: net/lftp

2016-11-23 Thread Rafael Sadowski
On Wed Nov 23, 2016 at 12:24:01PM +, Stuart Henderson wrote: > On 2016/11/22 04:40, Jeremie Courreges-Anglas wrote: > > > # lftp needs add_history_time() > > > LIB_DEPENDS= devel/libidn \ > > > devel/readline>=6.1p2 \ > > > + > > > +RUN_DEPENDS= devel/gettext > > > > I do n

Re: Update: multimedia/mpv, feedback needed

2016-11-23 Thread Dmitrij D. Czarkoff
Jeremie Courreges-Anglas wrote: >I don't use mpv, but the fact that it uses libsmbclient puts mpv on my >plate when I update samba. Last time I checked I didn't find the >smbclient support particularly useful (access files on network >filesystems that aren't mounted locally). I'd be thankful if

Re: Update: multimedia/mpv, feedback needed

2016-11-23 Thread Antoine Jacoutot
On Wed, Nov 23, 2016 at 04:11:10PM +0100, Dmitrij D. Czarkoff wrote: > Antoine Jacoutot wrote: > > >>> --enable-foo needs to be explicitely set when the dependency \ > >>> is found by configure? > >> > >> And surely if that's the case for some foo, then --disable wouldn't be > >> needed for that

Re: Update: multimedia/mpv, feedback needed

2016-11-23 Thread Dmitrij D. Czarkoff
Antoine Jacoutot wrote: >>> --enable-foo needs to be explicitely set when the dependency \ >>> is found by configure? >> >> And surely if that's the case for some foo, then --disable wouldn't be >> needed for that foo.. > >--disable-foo is needed because in case a dependency is \ >there at build

Re: Update: multimedia/mpv, feedback needed

2016-11-23 Thread Antoine Jacoutot
On Wed, Nov 23, 2016 at 02:11:27PM +, Stuart Henderson wrote: > On 2016/11/23 14:30, Antoine Jacoutot wrote: > > On Wed, Nov 23, 2016 at 02:05:30PM +0100, Dmitrij D. Czarkoff wrote: > > > Antoine Jacoutot wrote: > > > > > > >> +.if ${FLAVOR:L:Mlite} > > > >> +TOGGLE =disable > > >

Re: Update: multimedia/mpv, feedback needed

2016-11-23 Thread Stuart Henderson
On 2016/11/23 14:30, Antoine Jacoutot wrote: > On Wed, Nov 23, 2016 at 02:05:30PM +0100, Dmitrij D. Czarkoff wrote: > > Antoine Jacoutot wrote: > > > > >> +.if ${FLAVOR:L:Mlite} > > >> +TOGGLE =disable > > >> +.else > > >> +TOGGLE =enable > > > > > >Why using a variable fo

Re: Update: multimedia/mpv, feedback needed

2016-11-23 Thread Antoine Jacoutot
On Wed, Nov 23, 2016 at 02:05:30PM +0100, Dmitrij D. Czarkoff wrote: > Antoine Jacoutot wrote: > > >> +.if ${FLAVOR:L:Mlite} > >> +TOGGLE =disable > >> +.else > >> +TOGGLE =enable > > > >Why using a variable for this? > >Wouldn't this work? > > > >.if ${FLAVOR:L:Mlite} > >

Re: Update: multimedia/mpv, feedback needed

2016-11-23 Thread Josh Grosse
On Wed, Nov 23, 2016 at 01:49:41AM +0100, Dmitrij D. Czarkoff wrote: > Hi! > > I was asked off-list to add a "lite" flavor for multimedia/mpv. This > diff introduces such version. This iteration disables the fllowing in > "lite" FLAVOR: > > * Lua UI and scripting > * Encoding > * DVD and Blu

Re: Update: multimedia/mpv, feedback needed

2016-11-23 Thread Dmitrij D. Czarkoff
Antoine Jacoutot wrote: >> +.if ${FLAVOR:L:Mlite} >> +TOGGLE =disable >> +.else >> +TOGGLE =enable > >Why using a variable for this? >Wouldn't this work? > >.if ${FLAVOR:L:Mlite} >CONFIGURE_ARGS += --disable-foo \ > --disable-bar >... Separate variable is supposed

Re: UPDATE: net/lftp

2016-11-23 Thread Stuart Henderson
On 2016/11/22 04:40, Jeremie Courreges-Anglas wrote: > > # lftp needs add_history_time() > > LIB_DEPENDS= devel/libidn \ > > devel/readline>=6.1p2 \ > > + > > +RUN_DEPENDS= devel/gettext > > I do not understand this. gettext could be part of LIB_DEPENDS, but why > would

Re: Another gstreamer security problem

2016-11-23 Thread David Coppa
On Wed, 23 Nov 2016, David Coppa wrote: > Another gstreamer exploit by Chris Evans: > > https://scarybeastsecurity.blogspot.it/2016/11/0day-exploit-advancing-exploitation.html > > This time, we have patches for both gstreamer-0.10 and gstreamer1 > (from upstream). > > Ok for the diff below? Fi

Re: Update: multimedia/mpv, feedback needed

2016-11-23 Thread Marc Espie
On Wed, Nov 23, 2016 at 01:49:41AM +0100, Dmitrij D. Czarkoff wrote: > Hi! > > I was asked off-list to add a "lite" flavor for multimedia/mpv. This > diff introduces such version. This iteration disables the fllowing in > "lite" FLAVOR: > > * Lua UI and scripting > * Encoding > * DVD and Blu

Another gstreamer security problem

2016-11-23 Thread David Coppa
Another gstreamer exploit by Chris Evans: https://scarybeastsecurity.blogspot.it/2016/11/0day-exploit-advancing-exploitation.html This time, we have patches for both gstreamer-0.10 and gstreamer1 (from upstream). Ok for the diff below? Index: gstreamer-0.10/plugins-good/Makefile ==

Re: Update: multimedia/mpv, feedback needed

2016-11-23 Thread Stuart Henderson
You have a number of libraries repeated in WANTLIB (i.e. once for "lite" and again for "no-lite"), could you remove the second one please as it will mean things may get missed when WANTLIB are bulk-synced in the future. Should *input.conf also go to PFRAG.no-lite? I think you disabled all the

[UPDATE] devel/p5-strictures

2016-11-23 Thread Sergey Bronnikov
Ok? diff --git a/devel/p5-strictures/Makefile b/devel/p5-strictures/Makefile index 47fcfc0..8738127 100644 --- a/devel/p5-strictures/Makefile +++ b/devel/p5-strictures/Makefile @@ -4,7 +4,7 @@ COMMENT = turn on strict and make most warnings fatal MODULES = cpan PKG_ARCH = * -DIS

[UPDATE] devel/p5-indirect

2016-11-23 Thread Sergey Bronnikov
Ok? diff --git a/devel/p5-indirect/Makefile b/devel/p5-indirect/Makefile index 1b77a0a..4f43e4a 100644 --- a/devel/p5-indirect/Makefile +++ b/devel/p5-indirect/Makefile @@ -3,7 +3,7 @@ COMMENT= lexically warn about using the indirect method call syntax MODULES= cpan -DISTNAME=

[UPDATE] devel/p5-Role-Tiny

2016-11-23 Thread Sergey Bronnikov
Ok? diff --git a/devel/p5-Role-Tiny/Makefile b/devel/p5-Role-Tiny/Makefile index e3ca498..9c1dbae 100644 --- a/devel/p5-Role-Tiny/Makefile +++ b/devel/p5-Role-Tiny/Makefile @@ -4,7 +4,7 @@ COMMENT = minimalist role composition tool MODULES = cpan PKG_ARCH = * -DISTNAME = Ro

[UPDATE] devel/p5-IO-TieCombine

2016-11-23 Thread Sergey Bronnikov
Ok? diff --git a/devel/p5-IO-TieCombine/Makefile b/devel/p5-IO-TieCombine/Makefile index 133c184..46a3a22 100644 --- a/devel/p5-IO-TieCombine/Makefile +++ b/devel/p5-IO-TieCombine/Makefile @@ -4,7 +4,7 @@ COMMENT = produce tied (and other) separate but combined variables MODULES = c

[UPDATE] devel/p5-Import-Into

2016-11-23 Thread Sergey Bronnikov
Ok? diff --git a/devel/p5-Import-Into/Makefile b/devel/p5-Import-Into/Makefile index 5acec73..d473ee5 100644 --- a/devel/p5-Import-Into/Makefile +++ b/devel/p5-Import-Into/Makefile @@ -4,10 +4,10 @@ COMMENT = import packages into other packages MODULES = cpan PKG_ARCH = * -DISTNAM

Re: Update: multimedia/mpv, feedback needed

2016-11-23 Thread Antoine Jacoutot
On Wed, Nov 23, 2016 at 01:49:41AM +0100, Dmitrij D. Czarkoff wrote: > Hi! > > I was asked off-list to add a "lite" flavor for multimedia/mpv. This > diff introduces such version. This iteration disables the fllowing in > "lite" FLAVOR: > > * Lua UI and scripting > * Encoding > * DVD and Blu

[UPDATE] devel/p5-ExtUtils-InstallPaths

2016-11-23 Thread Sergey Bronnikov
Ok? diff --git a/devel/p5-ExtUtils-InstallPaths/Makefile b/devel/p5-ExtUtils-InstallPaths/Makefile index 0a7cbd8..f96345c 100644 --- a/devel/p5-ExtUtils-InstallPaths/Makefile +++ b/devel/p5-ExtUtils-InstallPaths/Makefile @@ -4,7 +4,7 @@ COMMENT = Build.PL install path logic made easy MOD

[UPDATE] devel/p5-App-Cmd

2016-11-23 Thread Sergey Bronnikov
Ok? diff --git a/devel/p5-App-Cmd/Makefile b/devel/p5-App-Cmd/Makefile index b7cc25a..b6077f0 100644 --- a/devel/p5-App-Cmd/Makefile +++ b/devel/p5-App-Cmd/Makefile @@ -4,7 +4,7 @@ COMMENT = write command line apps with less suffering MODULES = cpan PKG_ARCH =